dashdemux: Fix period selection for live streams
authorDavid Waring <david.waring@rd.bbc.co.uk>
Tue, 2 Dec 2014 10:06:00 +0000 (10:06 +0000)
committerThiago Santos <thiagoss@osg.samsung.com>
Tue, 2 Dec 2014 16:24:52 +0000 (13:24 -0300)
Fix period selection and properly error out when update cannot be done

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

ext/dash/gstdashdemux.c

index 5e09e00a5b080cd9a8960ca4b17f3f00aeff8dd0..4ec68236d26c3797c0ecbf2246a3a072e97e4b53 100644 (file)
@@ -951,7 +951,7 @@ gst_dash_demux_update_manifest (GstAdaptiveDemux * demux, GstBuffer * buffer)
       /* TODO */
     }
 
-    if (period_idx) {
+    if (period_id) {
       if (!gst_mpd_client_set_period_id (new_client, period_id)) {
         GST_DEBUG_OBJECT (demux, "Error setting up the updated manifest file");
         return GST_FLOW_EOS;
@@ -1005,6 +1005,7 @@ gst_dash_demux_update_manifest (GstAdaptiveDemux * demux, GstBuffer * buffer)
      * the manifest */
     GST_WARNING_OBJECT (demux, "Error parsing the manifest.");
     gst_buffer_unmap (buffer, &mapinfo);
+    return GST_FLOW_ERROR;
   }
 
   gst_buffer_unmap (buffer, &mapinfo);