Crashes when moving an unconnected object with inkscape:connector-type attribute

Bug #478597 reported by Sergey Nizovtsev
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
High
Michael Wybrow
inkscape (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: inkscape

Steps to reproduce:
1. Open "inkscape_crash.svg"
2. Choose "Edit paths by nodes" tool
3. Select curve at the center
4. Choose "Connector" tool

ProblemType: Crash
Architecture: i386
AssertionMessage: ERROR:connector-context.cpp:1299:bool cc_item_is_connector(SPItem*): assertion failed: (!(SP_SHAPE(item)->curve->is_closed()))
Date: Sun Nov 8 21:46:03 2009
DistroRelease: Ubuntu 9.10
ExecutablePath: /usr/bin/inkscape
Package: inkscape 0.47~pre4-0ubuntu1
ProcCmdline: inkscape
ProcEnviron:
 LANG=ru_RU.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-14.48-generic
Signal: 6
SourcePackage: inkscape
StacktraceTop:
 __kernel_vsyscall ()
 raise () from /lib/tls/i686/cmov/libc.so.6
 abort () from /lib/tls/i686/cmov/libc.so.6
 g_assertion_message () from /lib/libglib-2.0.so.0
 g_assertion_message_expr () from /lib/libglib-2.0.so.0
Title: inkscape assert failure: ERROR:connector-context.cpp:1299:bool cc_item_is_connector(SPItem*): assertion failed: (!(SP_SHAPE(item)->curve->is_closed()))
Uname: Linux 2.6.31-14-generic i686
UserGroups: adm admin audio cdrom dialout dip fax fuse lpadmin netdev plugdev sambashare tape video

Revision history for this message
Sergey Nizovtsev (snizovtsev) wrote :
visibility: private → public
Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt (retraced)

StacktraceTop:__kernel_vsyscall ()
*__GI_raise (sig=6)
*__GI_abort () at abort.c:121
g_assertion_message () from /lib/libglib-2.0.so.0
g_assertion_message_expr () from /lib/libglib-2.0.so.0

Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt (retraced)
Changed in inkscape (Ubuntu):
importance: Undecided → Medium
tags: removed: need-i386-retrace
Revision history for this message
Alex Valavanis (valavanisalex) wrote : Re: inkscape assert failure: ERROR:connector-context.cpp:1299:bool cc_item_is_connector(SPItem*): assertion failed: (!(SP_SHAPE(item)->curve->is_closed()))

Confirmed on karmic with inkscape 0.47~pre4-0ubuntu1

Changed in inkscape (Ubuntu):
status: New → Confirmed
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Added more complete backtrace

tags: removed: apport-crash i386
Changed in inkscape:
status: New → Confirmed
su_v (suv-lp)
tags: added: connectors crash
Revision history for this message
su_v (suv-lp) wrote :

reproduced with Inkscape 0.46-2 and Inkscape 0.47-1 on OS X 10.5.8.

Revision history for this message
sas (sas-sas) wrote :

I can reproduce this with Inkscape 0.47 on Windows XP.

The following also crashes Inkscape:
 1. Open "inkscape_crash.svg"
 2. With the Selector tool, grab the curve and move it.
(The crash occurs on letting go, and there's the same assertion failure as above.)

The path has an inkscape:connector-type attribute (though it's not connected to anything). If this attribute is removed (using the XML editor), then neither crash occurs.

summary: - inkscape assert failure: ERROR:connector-context.cpp:1299:bool
- cc_item_is_connector(SPItem*): assertion failed:
- (!(SP_SHAPE(item)->curve->is_closed()))
+ Crashes when moving an unconnected object with inkscape:connector-type
+ attribute
Changed in inkscape:
importance: Undecided → High
Changed in inkscape (Ubuntu):
assignee: nobody → Michael Wybrow (mjwybrow)
Changed in inkscape:
assignee: nobody → Michael Wybrow (mjwybrow)
Revision history for this message
Michael Wybrow (mjwybrow) wrote :

Fixed in r9708 of the trunk.

The connector context was asserting that paths marked with connector attributes were always open. Now it just treats them as
connectors for editing purposes if they are open, or normal objects otherwise.

Changed in inkscape:
status: Confirmed → Fix Committed
Revision history for this message
Michael Wybrow (mjwybrow) wrote :

Fixed in r9708 of the lp:inkscape trunk.

The connector context was asserting that paths marked with connector attributes were always open. Now it just treats them as
connectors for editing purposes if they are open, or normal objects otherwise.

Changed in inkscape (Ubuntu):
status: Confirmed → Fix Committed
Changed in inkscape (Ubuntu):
status: Fix Committed → Confirmed
Changed in inkscape (Ubuntu):
status: Confirmed → Fix Committed
status: Fix Committed → Confirmed
Revision history for this message
Alex Valavanis (valavanisalex) wrote :

Adding upstream patch for testing with Ubuntu

Changed in inkscape:
milestone: none → 0.49
Changed in inkscape (Ubuntu):
assignee: Michael Wybrow (mjwybrow) → Alex Valavanis (valavanisalex)
assignee: Alex Valavanis (valavanisalex) → nobody
tags: added: patch
Revision history for this message
su_v (suv-lp) wrote :

Setting milestone to 0.48: the fix (r9708 in trunk) had been backported to 0.48 in r9654 (last revision before the release)

<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/9708>
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/RELEASE_0_48_BRANCH/revision/9654>

Changed in inkscape:
milestone: 0.49 → 0.48
status: Fix Committed → Fix Released
Changed in inkscape (Ubuntu):
status: Confirmed → 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.