plugins/elements/gstinputselector.c: Gracefully handle the cases when we dont' have...
authorEdward Hervey <bilboed@bilboed.com>
Wed, 15 Oct 2008 17:45:37 +0000 (17:45 +0000)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Fri, 31 Dec 2010 00:53:47 +0000 (00:53 +0000)
Original commit message from CVS:
* plugins/elements/gstinputselector.c: (gst_input_selector_event),
(gst_input_selector_query):
Gracefully handle the cases when we dont' have otherpad.
Fixes #556430

plugins/elements/gstinputselector.c

index 45f2af9..c175a5e 100644 (file)
@@ -1017,15 +1017,16 @@ gst_input_selector_get_linked_pad (GstPad * pad, gboolean strict)
 static gboolean
 gst_input_selector_event (GstPad * pad, GstEvent * event)
 {
-  gboolean res;
+  gboolean res = FALSE;
   GstPad *otherpad;
 
   otherpad = gst_input_selector_get_linked_pad (pad, TRUE);
 
-  res = gst_pad_push_event (otherpad, event);
-
-  gst_object_unref (otherpad);
+  if (otherpad) {
+    res = gst_pad_push_event (otherpad, event);
 
+    gst_object_unref (otherpad);
+  }
   return res;
 }
 
@@ -1034,7 +1035,7 @@ gst_input_selector_event (GstPad * pad, GstEvent * event)
 static gboolean
 gst_input_selector_query (GstPad * pad, GstQuery * query)
 {
-  gboolean res;
+  gboolean res = TRUE;
   GstInputSelector *sel;
   GstPad *otherpad;
 
@@ -1101,10 +1102,12 @@ gst_input_selector_query (GstPad * pad, GstQuery * query)
       break;
     }
     default:
-      res = gst_pad_peer_query (otherpad, query);
+      if (otherpad)
+        res = gst_pad_peer_query (otherpad, query);
       break;
   }
-  gst_object_unref (otherpad);
+  if (otherpad)
+    gst_object_unref (otherpad);
   gst_object_unref (sel);
 
   return res;