From: sjoerd@luon.net <sjoerd@luon.net>
Date: Tue, 5 Aug 2008 11:30:57 +0000 (+0000)
Subject: [MOVED FROM GST-P-FARSIGHT] Fix refcounting issues in prepare_output_buffer
X-Git-Tag: 1.19.3~507^2~19435
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d2a43636b9b6cd9d328469bc028d3b3169d52d6c;p=platform%2Fupstream%2Fgstreamer.git

[MOVED FROM GST-P-FARSIGHT] Fix refcounting issues in prepare_output_buffer

20080805113057-be0f2-9dc270781f0a0f21c616ed11dbd1f198fd1b326e.gz
---

diff --git a/gst/valve/gstvalve.c b/gst/valve/gstvalve.c
index c0f4163..d70f8ba 100644
--- a/gst/valve/gstvalve.c
+++ b/gst/valve/gstvalve.c
@@ -212,18 +212,25 @@ gst_valve_prepare_output_buffer (GstBaseTransform *trans, GstBuffer * in_buf,
   }
   else
   {
+    /* Always return a reffed buffer, which in case of returning the input
+     * buffer means adding an extra ref to it */
     if (valve->discont)
     {
-      *out_buf = gst_buffer_make_metadata_writable (in_buf);
+      if (gst_buffer_is_metadata_writable (in_buf))
+      {
+        *out_buf = gst_buffer_ref (in_buf);
+      }
+      else
+      {
+        *out_buf = gst_buffer_create_sub (in_buf, 0, GST_BUFFER_SIZE (in_buf));
+      }
       GST_BUFFER_FLAG_SET (*out_buf, GST_BUFFER_FLAG_DISCONT);
       valve->discont = FALSE;
-
     }
     else
     {
-      *out_buf = in_buf;
+      *out_buf = gst_buffer_ref(in_buf);
     }
-    gst_buffer_ref (*out_buf);
   }
   GST_OBJECT_UNLOCK (GST_OBJECT (trans));