Crashes on trying to open document properties or preferences

Bug #1125620 reported by Dru
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
High
jazzynico

Bug Description

Hi! This is my first post here (I think!) so bear with any gaffes I might make. Hi ~suv!
The problem is simple to describe. Open Inkscape. Choose either preferences (editmenu) or document properties (filemenu). Inkscape immediately puts up the box 'will close now'. Shortly followed by the sorry for the inconvenience box.

The last build by Uwe Scholer for windows that this did not happen on was inkscape_r12088-201302020916. To be very clear - this build has no problem. (At least this one.)

The last build that I tried it on and crashed was inkscape_r12122-201302131948. I downloaded and ran the dbg for this build. Somewhat clumsily because this was my first time and I am not deft with command line.

To avoid any possible interference from preferences or default files, I just removed my Inkscape folder from my application data folder entirely. Inkscape did create a new folder before crashing. This was to eliminate the possibility that I had a profile attached to my default that was a problem, as someone suggested. Although I did not remove all color profiles from my system.

I have attached the output from dbg as a text file. If I was meant to insert the text here I'm sorry and perhaps someone could point that out for next time. It's a rather small file - I deleted my tedious and repetitive help requests and lists of commands.

Just to give you a preview of what's in it I have excerpted a few lines here that i thought might be important but the entire output is in the file.

(gdb) bt
#0 0x705c7554 in _cmsSearchTag (Profile=0x6148218, sig=icSigProfileDescriptionTag,

lSignalError=0) at cmsio0.c:89
#1 0x705c75b7 in cmsIsTag (hProfile=0x6148218, sig=icSigProfileDescriptionTag) at

cmsio0.c:89
#2 0x705cde46 in cmsTakeProductDesc (hProfile=0x6148218) at cmsio1.c:170
#3 0x005abd07 in ?? ()
(gdb) continue
Continuing.

Program received signal SIGSEGV, Segmentation fault.
0x705c7554 in _cmsSearchTag (Profile=0x6148218, sig=icSigProfileDescriptionTag,

lSignalError=0) at cmsio0.c:89
89 in cmsio0.c
(gdb) kill

Thanks for your help. Sorry for breaking your program!

Revision history for this message
Dru (drubanerjee) wrote :
su_v (suv-lp)
tags: added: color crash preferences win32
Changed in inkscape:
importance: Undecided → High
tags: added: regression
Changed in inkscape:
milestone: none → 0.49
Revision history for this message
su_v (suv-lp) wrote :

Possibly related changes in Windows devlibs (*): revision 42/43
<http://bazaar.launchpad.net/~inkscape.dev/inkscape-devlibs/trunk/revision/42>
<http://bazaar.launchpad.net/~inkscape.dev/inkscape-devlibs/trunk/revision/43>
which added lcms2 for libcdr (inkscape trunk r12091):
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/12091>

(*) based on the build date of the last development snapshot (r12088) not exposing this issue (I have no way to verify when UweSch updated the devlibs, but I'd be inclined to assume it was for the one after that (r12093), because libcdr support for Windows was added in r12091)

Revision history for this message
su_v (suv-lp) wrote :

~suv wrote:
> which added lcms2 (…)

Mentioned in parts because of this older report about conflicts due to having both lcms and lcms2 as dynamic dependencies (on linux though):
- Bug #682357 “Crash on startup while reading ICC profile”
  <https://bugs.launchpad.net/inkscape/+bug/682357>

Revision history for this message
Dru (drubanerjee) wrote :

After some experimentation I decided that

1. I had several profiles that were a problem.
2. I did not need any profiles older than 8 years.

I do think that just having an old profile should not cause the program to crash - perhaps a safeguard could be written in.

To that end I am attaching a few profiles that cause the crash, perhaps they will help you fix the problem.

Thanks for your help! Keep up the good work!

Revision history for this message
Dru (drubanerjee) wrote :

Could not find an edit button so am adding this in another comment, sorry.
I have to point out that the problem is entirely ameliorated by removing offending profiles from this location:
%systemroot%\system32\spool\drivers\color
In case it's a problem for anyone else.

Revision history for this message
su_v (suv-lp) wrote :

Crash reproduced (for each of the attached color profiles) with trunk builds of one specific build environments with broken color management (see bug #1024344 for details). Unlike on Dru's system, the crash here also occurs with trunk builds older than revision 12088.

Builds with newer (Quartz-based) newer dependencies (different build environment) don't crash (they only recognize 3 of the 5 profiles though), nor do stable 0.48.4 builds with either build environment.

Full backtrace attached, crash also occurs in 'lcms' (installed version: 0.19).

Revision history for this message
su_v (suv-lp) wrote :

@JazzyNico - if you find the time, could you test this issue (using one or more ICM profiles attached to comment #4) with your own Inkscape Windows builds?
1) latest trunk version built with older devlibs (i.e. without lcms2 included, <= r41)
2) current trunk version built with current devlibs (inlcudes lcms as well as lcms2)

Revision history for this message
jazzynico (jazzynico) wrote :

Confirmed with the new devlibs (with lcms2).
Not reproduced with the old devlibs.

Changed in inkscape:
status: New → Triaged
Revision history for this message
jazzynico (jazzynico) wrote :

Also reproduced with the experimental cross-compiled devlibs.

Revision history for this message
John Smith (john-smithi) wrote :

Reproduced with Windows 7, devlibs r44, current trunk (r12132).

Revision history for this message
jazzynico (jazzynico) wrote :

Patch in progress.

Some preliminary tests seems to show that completely replacing lcms with lcms2 fixes the issue, and works perfectly. The ICM profiles attached comment #4 can be linked to the document and used on objects.

Changed in inkscape:
assignee: nobody → JazzyNico (jazzynico)
status: Triaged → In Progress
Revision history for this message
jazzynico (jazzynico) wrote :

Patch attached (in DOS mode).
Tested successfully on Windows XP, Inkscape trunk revision 12131 (official and experimental devlibs).

Note that since the patch modify build.xml, you need to touch color-profile-cms-fns.h, color-profile.cpp and sp-image.cpp or start a clean compilation to take the changes into account.

Revision history for this message
jazzynico (jazzynico) wrote :

Fixed revision 12136.

Changed in inkscape:
milestone: 0.49 → none
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.