From: Vincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Date: Fri, 13 Jan 2012 16:52:23 +0000 (+0000)
Subject: xvimagesink: fix leak when images are freed after the X context
X-Git-Tag: RELEASE-0.11.2~2^2~39
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a8a3e3f7d58492f9e47dfeee94b7af4679737207;p=platform%2Fupstream%2Fgst-plugins-base.git

xvimagesink: fix leak when images are freed after the X context

I'm not 100% sure this is valid on any other X server than mine,
but since the XFree call does not take the context as a parameter,
it seems pretty certain it's the right thing to do, but I'll put
this caveat here in case someone checks in the future.
---

diff --git a/sys/xvimage/xvimagesink.c b/sys/xvimage/xvimagesink.c
index 8a7913b..0c386c1 100644
--- a/sys/xvimage/xvimagesink.c
+++ b/sys/xvimage/xvimagesink.c
@@ -244,6 +244,8 @@ gst_xvimage_buffer_destroy (GstXvImageBuffer * xvimage)
       shmdt (xvimage->SHMInfo.shmaddr);
     }
 #endif
+    if (xvimage->xvimage)
+      XFree (xvimage->xvimage);
     goto beach;
   }