Inkscape

Memory access error (crash) on undo of Pattern along Path

Bug #1716926 reported by Stephen Michael Gombas
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Jabiertxof
Inkscape 0.92.3 "0.92.3"

Bug Description

Upon updating to 0.92.2, I found that if I use the ellipse shape on the bezier tool, then move the shape with the node editor tool, then try to undo the path, Inkscape immediately crashes without fail. When I open the autosave file it creates upon crashing and click on the line that was there with the node selector tool, it crashes again. It has to be selected with the object selector tool and deleted to not crash. This has been reproduced by two others on windows, and once on Linux Mint 18.2. This bug appears on a fresh install as well.

To better exemplify what how this bug behaves, I've made a video. It shows the original crash, me trying to click on it, and then I recreate the bug again. I then show you my resource allocation for inkscape, system specs, and inkscape version.

https://www.youtube.com/watch?v=YE-ahcmeAFY

Windows 10 Home 64x
Inkscape 0.92.2 - 8 threads, 4096 MB RAM allocated

See original description
Stephen Michael Gombas (tubahorse)
description: updated
Revision history for this message
Jabiertxof (jabiertxof) wrote :

Fixed in trunk in r.58ef38f81c

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Fixed in 0.92.x in r.194e3ff648

Revision history for this message
Jabiertxof (jabiertxof) wrote :

Thanks for fill the bug @tubahorse

Revision history for this message
Stephen Michael Gombas (tubahorse) wrote :

Excellent! Does this mean it's fixed? Where can I download the fixed version?

Revision history for this message
Patrick Storz (ede123) wrote :

Both issue and fix confirmed.

@Jabiertxof: Thanks for the fix!
One other thing (not meant to blame, just a suggestion on how to achieve clearer commit messages others can use more efficiently):

You put "Fixes for bug #1716926. Consider backport".
- For the "Consider backport" part:
  We have the "backport-proposed" tag in Launchpad, so you can omit
  it from the commit message (were it does not belong and is likely
  to be overlooked anyway; certainly it's not suitable to track
  commits that should be considered for backporting reliably)
- For the "Fixes for bug #1716926." part:
  While this additional information is helpful and should be part
  of the commit message it does not really describe what your
  commit was about (unless one digs through the bug itself which
  is tedious and often impractical, e.g. while preparing release
  notes).
  I suggest you put a very short (but very precise) summary of the
  commit into the first line of the commit message, e.g.
  "Fix crash when removing a path with ellipse shape via undo"
  in this case.

Changed in inkscape:
status: New → Fix Committed
importance: Undecided → Medium
assignee: nobody → Jabiertxof (jabiertxof)
milestone: none → 0.92.3
tags: added: livepatheffects undo
Revision history for this message
Jabiertxof (jabiertxof) wrote :

Thanks @eduard-braun2 I do as it in the future!

Revision history for this message
Jabiertxof (jabiertxof) wrote :

There is only source code to compile it @tubahorse.

The code for the next release "0.92.3" is in the git branch "0.92.x" on our Inkscape repository over gitlab.
https://gitlab.com/inkscape/inkscape

Cheers, Jabier.

Revision history for this message
Stephen Michael Gombas (tubahorse) wrote :

My mistake! Again, thank you for fixing this bug @jabiertxof.

Revision history for this message
Patrick Storz (ede123) wrote :

Actually we have CI builds available for Windows!

At [1] you'll find the latest builds, the one for Jabier's commit would be [2].
If you follow one of the links (MINGW32 for 32-bit build, MINGW32 for the 64-bit build) you can download a complete package (7-zip archive) from the "Artifacts" tab.

I admit it's a bit hidden but should work flawlessly.

[1] https://ci.appveyor.com/project/inkscape/inkscape/history
[2] https://ci.appveyor.com/project/inkscape/inkscape/build/0.92.x-482

Revision history for this message
Jabiertxof (jabiertxof) wrote :

you are welcome @tubahorse

Revision history for this message
Stephen Michael Gombas (tubahorse) wrote :

Hello! I see 0.92.3 has been released. Was this bug fixed in the update?

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.
Subscribing...

Other bug subscribers

Remote bug watches

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