pad: don't try to pretty-print event after we've given away ownership
authorOlivier Crête <olivier.crete@collabora.com>
Fri, 14 Sep 2012 00:28:46 +0000 (01:28 +0100)
committerTim-Philipp Müller <tim@centricular.net>
Fri, 14 Sep 2012 00:28:46 +0000 (01:28 +0100)
Might cause crashes with debug logging enabled.

https://bugzilla.gnome.org/show_bug.cgi?id=683996

gst/gstpad.c

index b07fc4b..60f1d6e 100644 (file)
@@ -4480,9 +4480,8 @@ gst_pad_push_event_unchecked (GstPad * pad, GstEvent * event,
 
   /* Note: we gave away ownership of the event at this point but we can still
    * print the old pointer */
-  GST_LOG_OBJECT (pad,
-      "sent event %" GST_PTR_FORMAT " to peerpad %"
-      GST_PTR_FORMAT ", ret %s", event, peerpad, gst_flow_get_name (ret));
+  GST_LOG_OBJECT (pad, "sent event %p to peerpad %" GST_PTR_FORMAT ", ret %s",
+      event, peerpad, gst_flow_get_name (ret));
 
   gst_object_unref (peerpad);