[gst-cvs] gstreamer: capsfilter: fix refcounting problem

[Wim Taymans]

Module: gstreamer
Branch: master
Commit: 04f3f096c36a97961d52dd9b2042c810e297215b
URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=04f3f096c36a97961d52dd9b2042c810e297215b

Author: Wim Taymans <wim.taymans at collabora.co.uk>
Date:   Wed Aug  5 13:46:42 2009 +0200

capsfilter: fix refcounting problem

Make sure the metadata is writable before setting the caps on a buffer.

---

 plugins/elements/gstcapsfilter.c |   12 +++++++++---
 1 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/plugins/elements/gstcapsfilter.c b/plugins/elements/gstcapsfilter.c
index 1a02b84..9d7a81a 100644
--- a/plugins/elements/gstcapsfilter.c
+++ b/plugins/elements/gstcapsfilter.c
@@ -306,9 +306,15 @@ gst_capsfilter_prepare_buf (GstBaseTransform * trans, GstBuffer * input,
     /* FIXME : Move this behaviour to basetransform. The given caps are the ones
      * of the source pad, therefore our outgoing buffers should always have
      * those caps. */
-    gst_buffer_set_caps (input, caps);
-    gst_buffer_ref (input);
-    *buf = input;
+    if (gst_buffer_is_metadata_writable (input)) {
+      *buf = input;
+      gst_buffer_set_caps (*buf, caps);
+      gst_buffer_ref (input);
+    } else {
+      GST_DEBUG_OBJECT (trans, "Creating sub-buffer and setting caps");
+      *buf = gst_buffer_create_sub (input, 0, GST_BUFFER_SIZE (input));
+      gst_buffer_set_caps (*buf, caps);
+    }
   } else {
     /* Buffer has no caps. See if the output pad only supports fixed caps */
     GstCaps *out_caps;