rtspsrc: Don't return TRUE for unhandled query
authorSeungha Yang <seungha@centricular.com>
Tue, 16 Jun 2020 10:34:01 +0000 (19:34 +0900)
committerSeungha Yang <seungha@centricular.com>
Tue, 16 Jun 2020 10:35:30 +0000 (19:35 +0900)
Expected return value for unhandled query is FALSE

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/merge_requests/629>

gst/rtsp/gstrtspsrc.c

index 1f91bbf..603bdf0 100644 (file)
@@ -3057,7 +3057,7 @@ gst_rtspsrc_handle_internal_src_query (GstPad * pad, GstObject * parent,
     GstQuery * query)
 {
   GstRTSPSrc *src;
-  gboolean res = TRUE;
+  gboolean res = FALSE;
 
   src = GST_RTSPSRC_CAST (gst_pad_get_element_private (pad));
 
@@ -3079,9 +3079,9 @@ gst_rtspsrc_handle_internal_src_query (GstPad * pad, GstObject * parent,
       switch (format) {
         case GST_FORMAT_TIME:
           gst_query_set_duration (query, format, src->segment.duration);
+          res = TRUE;
           break;
         default:
-          res = FALSE;
           break;
       }
       break;
@@ -3091,6 +3091,7 @@ gst_rtspsrc_handle_internal_src_query (GstPad * pad, GstObject * parent,
       /* we are live with a min latency of 0 and unlimited max latency, this
        * result will be updated by the session manager if there is any. */
       gst_query_set_latency (query, src->is_live, 0, -1);
+      res = TRUE;
       break;
     }
     default: