trunk: crash on releasing groups clipped/masked by non-SPLPEItems, revision >= 13418

Bug #1333020 reported by Liam P. White
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
High
Liam P. White

Bug Description

Attaching minimal test case. Just ungroup the object.

This crash occurs because the code assumes that any object that serves as a clip-path or mask can have an LPE applied to it, when this is actually not the case.

Steps to reproduce:

1. Draw a rectangle.
2. Clone the rectangle and move it off to the side.
3. Group the original object.
4. Clip the group with the clone
5. Ungroup

--> crash

Revision history for this message
Liam P. White (liampwhite) wrote :
Revision history for this message
su_v (suv-lp) wrote :

Reproduced with r13435 on OS X 10.7.5

Based on tests with archived builds:
- not reproduced with rev <= 13417
- reproduced with rev >= 13418
the regression was introduced in revision 13418:
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/13418>

Changed in inkscape:
importance: Undecided → Critical
milestone: none → 0.91
status: New → Triaged
importance: Critical → High
Changed in inkscape:
assignee: nobody → Liam P. White (inkscapebrony)
Revision history for this message
Johan Engelen (johanengelen) wrote :

Sorry for not seeing this during code review.
Simple NULL check fixes it?

Revision history for this message
Liam P. White (liampwhite) wrote :

@Johan — correct.

Changed in inkscape:
status: Triaged → Fix Committed
Revision history for this message
Liam P. White (liampwhite) wrote :

Fix committed in:

lp:inkscape, revision 13436
lp:inkscape/experimental, revision 13407

Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → 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.