gst/gstpadtemplate.c: The old behaviour was that gst_pad_template_new() takes ownersh...

Original commit message from CVS:
* gst/gstpadtemplate.c:
The old behaviour was that gst_pad_template_new() takes ownership of
the caps. As we now call g_object_new() which calls g_object_set() and
which copies the caps, we have to unref them to not leak them. Fixes
make valgrid for me.

diff --git a/ChangeLog b/ChangeLog
index d898834..81873f7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,13 @@
 2008-08-28  Stefan Kost  <ensonic@users.sf.net>
 
+       * gst/gstpadtemplate.c:
+         The old behaviour was that gst_pad_template_new() takes ownership of
+         the caps. As we now call g_object_new() which calls g_object_set() and
+         which copies the caps, we have to unref them to not leak them. Fixes
+         make valgrid for me.
+
+2008-08-28  Stefan Kost  <ensonic@users.sf.net>
+
        * gst/gsturi.c:
          Don't segfault on input like "tel:+1-123-555-1234".
 

diff --git a/gst/gstpadtemplate.c b/gst/gstpadtemplate.c
index 9a60be3..b0f00f0 100644 (file)
--- a/gst/gstpadtemplate.c
+++ b/gst/gstpadtemplate.c
@@ -388,6 +388,9 @@ gst_pad_template_new (const gchar * name_template,
       "name", name_template, "name-template", name_template,
       "direction", direction, "presence", presence, "caps", caps, NULL);
 
+  if (caps)
+    gst_caps_unref (caps);
+
   return new;
 }