summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorXavier Claessens <xavier.claessens@collabora.com>2016-09-07 10:12:18 -0400
committerSebastian Dröge <sebastian@centricular.com>2016-09-07 18:40:57 +0300
commitf90ab92547dcc2034744ed2abab2077be804b5c6 (patch)
tree2ea2b925333cdb5e90da92830b54fe829a5a5441
parentd33eca6156fde0dbc303615da278c098cdb1d0f0 (diff)
stream: Fix leaked joined_bin
There is no need to keep a strong ref on it, and _leave_bin() was setting it to NULL before calling g_clear_object() so it was leaked. https://bugzilla.gnome.org/show_bug.cgi?id=766612
m---------common0
-rw-r--r--gst/rtsp-server/rtsp-stream.c3
2 files changed, 1 insertions, 2 deletions
diff --git a/common b/common
-Subproject f49c55ecd35a7436194d28297f6d6f20eb6a66f
+Subproject 6f2d2093e84cc0eb99b634fa281822ebb950728
diff --git a/gst/rtsp-server/rtsp-stream.c b/gst/rtsp-server/rtsp-stream.c
index c57d13d..61ffa3e 100644
--- a/gst/rtsp-server/rtsp-stream.c
+++ b/gst/rtsp-server/rtsp-stream.c
@@ -2768,7 +2768,7 @@ gst_rtsp_stream_join_bin (GstRTSPStream * stream, GstBin * bin,
(GCallback) caps_notify, stream);
}
- priv->joined_bin = gst_object_ref (bin);
+ priv->joined_bin = bin;
g_mutex_unlock (&priv->lock);
return TRUE;
@@ -2920,7 +2920,6 @@ gst_rtsp_stream_leave_bin (GstRTSPStream * stream, GstBin * bin,
gst_rtsp_address_free (priv->server_addr_v6);
priv->server_addr_v6 = NULL;
- g_clear_object (&priv->joined_bin);
g_mutex_unlock (&priv->lock);
return TRUE;