rtpsession: Ensure MT safe event handling and plug event leak.
authorHaakon Sporsheim <haakon.sporsheim@gmail.com>
Thu, 28 Apr 2011 20:59:28 +0000 (22:59 +0200)
committerOlivier CrĂȘte <olivier.crete@collabora.com>
Mon, 28 Jan 2013 22:44:31 +0000 (17:44 -0500)
https://bugzilla.gnome.org/show_bug.cgi?id=667826

gst/rtpmanager/gstrtpsession.c

index 78007f5..f09a3fb 100644 (file)
@@ -1435,8 +1435,21 @@ gst_rtp_session_event_recv_rtp_src (GstPad * pad, GstObject * parent,
       break;
   }
 
-  if (forward)
-    ret = gst_pad_push_event (rtpsession->recv_rtp_sink, event);
+  if (forward) {
+    GstPad *recv_rtp_sink = NULL;
+    GST_RTP_SESSION_LOCK (rtpsession);
+    if (rtpsession->recv_rtp_sink)
+      recv_rtp_sink = gst_object_ref (rtpsession->recv_rtp_sink);
+    GST_RTP_SESSION_UNLOCK (rtpsession);
+
+    if (recv_rtp_sink) {
+      ret = gst_pad_push_event (recv_rtp_sink, event);
+      gst_object_unref (recv_rtp_sink);
+    } else
+      gst_event_unref (event);
+  } else {
+    gst_event_unref (event);
+  }
 
   return ret;
 }