From: Jan Schmidt <jan@centricular.com>
Date: Wed, 1 Apr 2020 16:27:26 +0000 (+1100)
Subject: rmdemux: Don't ignore the return value of gst_segment_do_seek()
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ea945eb9c6af89a1ab70ad254fd6f9523eb5861d;p=platform%2Fupstream%2Fgst-plugins-ugly.git

rmdemux: Don't ignore the return value of gst_segment_do_seek()

If gst_segment_do_seek() fails, fail the seek.

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

diff --git a/gst/realmedia/rmdemux.c b/gst/realmedia/rmdemux.c
index 6634c225..6cc659a1 100644
--- a/gst/realmedia/rmdemux.c
+++ b/gst/realmedia/rmdemux.c
@@ -527,8 +527,11 @@ gst_rmdemux_perform_seek (GstRMDemux * rmdemux, GstEvent * event)
   GST_LOG_OBJECT (rmdemux, "Took streamlock");
 
   if (event) {
-    gst_segment_do_seek (&rmdemux->segment, rate, format, flags,
-        cur_type, cur, stop_type, stop, &update);
+    if (!gst_segment_do_seek (&rmdemux->segment, rate, format, flags,
+            cur_type, cur, stop_type, stop, &update)) {
+      ret = FALSE;
+      goto done;
+    }
   }
 
   GST_DEBUG_OBJECT (rmdemux, "segment positions set to %" GST_TIME_FORMAT "-%"