[Bug 751850] New: dashdemux: gst_mpd_client_advance_segment should return GST_FLOW_EOS when index is out of range

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 751850
           Summary: dashdemux: gst_mpd_client_advance_segment should
                    return GST_FLOW_EOS when index is out of range
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: florin.apostol at oregan.net
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

The gst_mpd_client_advance_segment function attempts to increase the
stream->segment_index. But if the updated index is out of range (>=
segments_count) the function will return GST_FLOW_OK. It will return
GST_FLOW_EOS only when the index is out of range BEFORE being incremented.

The current behavior leads to situations when gst_mpd_client_advance_segment
will return GST_FLOW_OK but gst_mpd_client_get_next_fragment will return false.

I believe gst_mpd_client_advance_segment function should check the updated
value of index and return GST_FLOW_EOS if it is out of range.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.