From: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date: Thu, 2 Jun 2011 08:41:52 +0000 (+0200)
Subject: souphttpsink: Fix refcounting of the "session" property
X-Git-Tag: RELEASE-0.11.1~7^2~439
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e376913fc3e914b5fe940ceb4a84357ff9334779;p=platform%2Fupstream%2Fgst-plugins-good.git

souphttpsink: Fix refcounting of the "session" property

Properties should never take ownership of the values
passed to them.
---

diff --git a/ext/soup/gstsouphttpsink.c b/ext/soup/gstsouphttpsink.c
index 509d689..04af372 100644
--- a/ext/soup/gstsouphttpsink.c
+++ b/ext/soup/gstsouphttpsink.c
@@ -249,7 +249,7 @@ gst_soup_http_sink_set_property (GObject * object, guint property_id,
       if (souphttpsink->prop_session) {
         g_object_unref (souphttpsink->prop_session);
       }
-      souphttpsink->prop_session = g_value_get_object (value);
+      souphttpsink->prop_session = g_value_dup_object (value);
       break;
     case PROP_LOCATION:
       g_free (souphttpsink->location);
@@ -327,9 +327,12 @@ gst_soup_http_sink_get_property (GObject * object, guint property_id,
 void
 gst_soup_http_sink_dispose (GObject * object)
 {
-  /* GstSoupHttpSink *souphttpsink = GST_SOUP_HTTP_SINK (object); */
+  GstSoupHttpSink *souphttpsink = GST_SOUP_HTTP_SINK (object);
 
   /* clean up as possible.  may be called multiple times */
+  if (souphttpsink->prop_session)
+    g_object_unref (souphttpsink->prop_session);
+  souphttpsink->prop_session = NULL;
 
   G_OBJECT_CLASS (parent_class)->dispose (object);
 }