After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 623383 - [PATCH] crash on rsvg-gobject.c:141, in instance_dispose function
[PATCH] crash on rsvg-gobject.c:141, in instance_dispose function
Status: RESOLVED FIXED
Product: librsvg
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: librsvg maintainers
librsvg maintainers
Depends on:
Blocks:
 
 
Reported: 2010-07-02 12:55 UTC by Adam Tkac
Modified: 2010-07-02 13:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Test file (14.74 KB, image/svg+xml)
2010-07-02 12:56 UTC, Adam Tkac
  Details
Proposed patch (957 bytes, patch)
2010-07-02 12:57 UTC, Adam Tkac
committed Details | Review

Description Adam Tkac 2010-07-02 12:55:26 UTC
This issue is present in current origin/master branch.

This issue is reproducible with the following command:

convert -background transparent "tigervnc_16.svg" "tigervnc_16.png"

Backtrace:

(gdb) bt
  • #0 g_object_unref
    from /lib64/libgobject-2.0.so.0
  • #1 instance_dispose
    at rsvg-gobject.c line 141
  • #2 g_object_unref
    from /lib64/libgobject-2.0.so.0
  • #3 ReadSVGImage
    at coders/svg.c line 2841
  • #4 ReadImage
    at magick/constitute.c line 578
  • #5 ReadImages
    at magick/constitute.c line 861
  • #6 ConvertImageCommand
    at wand/convert.c line 581
  • #7 MagickCommandGenesis
    at wand/mogrify.c line 165
  • #8 main
    at utilities/convert.c line 80

I will attach the "tigervnc_16.svg" for testing purposes and also proposed patch. Note the "convert" command comes from ImageMagick.
Comment 1 Adam Tkac 2010-07-02 12:56:18 UTC
Created attachment 165104 [details]
Test file
Comment 2 Adam Tkac 2010-07-02 12:57:21 UTC
Created attachment 165106 [details] [review]
Proposed patch

This simple patch fixes the issue in my case.
Comment 3 Hiroyuki Ikezoe 2010-07-02 13:42:55 UTC
Review of attachment 165106 [details] [review]:

Thank you! The patch is absolutely right.