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 691230 - GeglTileBackendRam leaks memory when destroyed
GeglTileBackendRam leaks memory when destroyed
Status: RESOLVED FIXED
Product: GEGL
Classification: Other
Component: core
git master
Other All
: Normal normal
: ---
Assigned To: Default Gegl Component Owner
Default Gegl Component Owner
Depends on:
Blocks:
 
 
Reported: 2013-01-06 16:10 UTC by Daniel Sabo
Modified: 2013-01-10 03:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gcc -Wall -g -o leak_test leak_test.c `pkg-config --cflags --libs gegl-0.2` (1.30 KB, text/x-csrc)
2013-01-06 16:10 UTC, Daniel Sabo
  Details
Free all buffers when GeglTileBackendRam is finalized (1.16 KB, patch)
2013-01-06 16:11 UTC, Daniel Sabo
none Details | Review

Description Daniel Sabo 2013-01-06 16:10:59 UTC
Created attachment 232863 [details]
gcc -Wall -g -o leak_test leak_test.c `pkg-config --cflags --libs gegl-0.2`

GeglTileBackendRam leaks all of its buffers when destroyed.
Comment 1 Daniel Sabo 2013-01-06 16:11:49 UTC
Created attachment 232864 [details] [review]
Free all buffers when GeglTileBackendRam is finalized
Comment 2 Michael Natterer 2013-01-06 17:31:44 UTC
That looks right, just add the space before the ( and wait for your
git account :)
Comment 3 Daniel Sabo 2013-01-10 03:02:19 UTC
commit a8030a513acbdfe2465d390d23ef62f6e86efcff
Author: Daniel Sabo <DanielSabo@gmail.com>
Date:   Sun Jan 6 00:55:14 2013 -0800

    GeglTileBackendRam: Free all buffers in finalize()

 gegl/buffer/gegl-tile-backend-ram.c |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)