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 89411 - gdk-pixbuf-csource leaks memory
gdk-pixbuf-csource leaks memory
Status: RESOLVED FIXED
Product: gdk-pixbuf
Classification: Platform
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2002-07-30 12:42 UTC by Sven Neumann
Modified: 2010-07-10 04:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch that plugs the reported memory leak (3.72 KB, patch)
2002-07-30 12:44 UTC, Sven Neumann
none Details | Review

Description Sven Neumann 2002-07-30 12:42:51 UTC
Unless I missed something obvious, gdk-pixbuf-csource leaks a good amount of
memory in gdk_pixdata_to_csource(). In line 727 the function
gdk_pixdata_serialize() is called which returns a newly allocated character
array. This array is never freed. I'll attach a patch that fixes the problem.
Comment 1 Sven Neumann 2002-07-30 12:44:43 UTC
Created attachment 10118 [details] [review]
patch that plugs the reported memory leak
Comment 2 Owen Taylor 2002-07-30 18:12:24 UTC
How does stream get initialized if cdata.dump_stream is FALSE?
Comment 3 Sven Neumann 2002-07-30 20:38:19 UTC
By the very first lines of the patch. The stream variable is
unconditionally initialized to NULL.
Comment 4 Owen Taylor 2002-07-30 22:00:07 UTC
Doh :-)

Go ahead and commit to both branch.

(Though I'm not all that worried by this memory leak.)
Comment 5 Sven Neumann 2002-07-31 08:18:06 UTC
OK, fixed in both branches.