[PATCH 07/14] winegstreamer: Fix buffer leak in error path

Maarten Lankhorst m.b.lankhorst at gmail.com
Tue Nov 9 16:42:44 CST 2010


---
 dlls/winegstreamer/gstdemux.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c
index d26f853..a6f72c4 100644
--- a/dlls/winegstreamer/gstdemux.c
+++ b/dlls/winegstreamer/gstdemux.c
@@ -517,10 +517,12 @@ static GstFlowReturn got_data_sink(GstPad *pad, GstBuffer *buf) {
     } else {
         BYTE *ptr = NULL;
         hr = BaseOutputPinImpl_GetDeliveryBuffer(&pin->pin, &sample, NULL, NULL, 0);
-        freeSamp = TRUE;
-        if (hr == VFW_E_NOT_CONNECTED)
+        if (hr == VFW_E_NOT_CONNECTED) {
+            gst_buffer_unref(buf);
             return GST_FLOW_NOT_LINKED;
+        }
         if (FAILED(hr)) {
+            gst_buffer_unref(buf);
             ERR("Didn't get a GST_APP_BUFFER, and could not get a delivery buffer (%x), returning GST_FLOW_WRONG_STATE\n", hr);
             return GST_FLOW_WRONG_STATE;
         }
-- 
1.7.1




More information about the wine-patches mailing list