GNOME Bugzilla – Bug 309967
Memory leak in /gimp/plug-ins/common/gauss.c ?
Last modified: 2005-07-11 15:39:07 UTC
Please describe the problem: Can't find g_free(sum) and g_free(curve). Is this a possible memory leak? Steps to reproduce: Inspecting CVS source http://cvs.gnome.org/viewcvs/gimp/plug-ins/common/gauss.c? rev=1.23&view=markup Actual results: Expected results: Does this happen every time? Other information:
Memory leaks in plug-ins aren't really bothersome since the plug-in runs in a separate process. But of course it would be nicer if the plug-in didn't leak. This does indeed look like a memleak to me. If you or someone else wants to attempt a fix, patches are welcome.
Had a look at it myself. Fixed in CVS HEAD; probably not worth backporting to 2.2: 2005-07-11 Sven Neumann <sven@gimp.org> * plug-ins/common/gauss.c: plugged memory leaks (bug #309967). Invalidate the preview when the algorithm is changed; default to RLE since it appears to be faster for the cases I tried.
Just a followup note on the factors that affect speed: a look at the algorithms indicates that RLE gains an advantage when there are lots of places where neighborig pixels have the same color; IIR gains an advantage when the blur radius is large. But the differences should not be huge in either case.