[Bug 757951] New: adaptivedemux: set src element to ready before flushing the input pad

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 757951
           Summary: adaptivedemux: set src element to ready before
                    flushing the input pad
    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: ---

Since commit ccff3be3ab2e5bffcefc12c80a5edb225801f1b9 introduced asynchronous
input to adaptivedemux (by using a queue2 element that intermediates data
generated by uri handler) the adaptive demux incorrectly shuts down the uri
handler. 

Previously, after the uri handler generated an error, it stopped sending data,
so when adaptive demux sees the error message, no more data will be received.
Now, due to the queue2 element, adaptive demux can see the error message and
then receive some more data. 

During the uri handler shut down sequence, adaptive demux will send a flush
start event, change the state of src to ready and then send the flush stop
event. But after the send start event, the queue2 element can try to push new
data and this will fail because no segment start event was sent. Adaptive demux
should first set the src element to ready and only then send the flush start
and flush stop events.

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