[gstreamer-bugs] [Bug 586491] Memory leak in Buffer.Data

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=586491

  GStreamer | gst-sharp | Ver: git

Maarten Bosmans changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |NOTABUG




------- Comment #2 from Maarten Bosmans  2009-06-20 23:01 UTC -------
Fortunately, it turned out the memory leak wasn't in gstreamer-sharp.

It is in Mono.Cairo or the sample app, however you want to see it. The
Cairo.Surface needs to be explicitly destroyed.

On top of the previouse patch, apply:
-               return img.Data;
+               byte[] data = img.Data;
+               img.Destroy();
+               return data;

The testcase now runs smoothly, without eating up all your GBs of ram.

Please do add the corrected sample to the git tree, as it is a nice sample app.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=586491.