[Bug 770075] New: adaptivedemux: fix stream exposure condition

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Aug 18 08:32:01 UTC 2016


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

            Bug ID: 770075
           Summary: adaptivedemux: fix stream exposure condition
    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: m.olbrich at pengutronix.de
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Created attachment 333548
  --> https://bugzilla.gnome.org/attachment.cgi?id=333548&action=edit
adaptivedemux: fix stream exposure condition

The new streams should not be exposed until all streams are done with the
current fragment. The old code is incorrect and actually only checked the
current stream. Fix this by properly checking all streams.

Also, ignore the current stream. The code is only reached when the current
stream finished downloading and since
65f2ce3c2f1f26b30bbf807b35b1faf23504fe7c ("adaptivedemux: On EOS, handle it
before waking download loop") download_finished is set after
gst_adaptive_demux_stream_advance_fragment_unlocked() is called.

Without this HLS playback with multiple streams is broken, because the new
streams are never exposed.

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


More information about the gstreamer-bugs mailing list