[Bug 743326] New: flvdemux: create pads as soon as we can.

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Jan 21 16:04:32 PST 2015


https://bugzilla.gnome.org/show_bug.cgi?id=743326
  GStreamer | gst-plugins-good | unspecified

           Summary: flvdemux: create pads as soon as we can.
    Classification: Platform
           Product: GStreamer
           Version: unspecified
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: mathieu.duponchelle at epitech.eu
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Otherwise the flow combiner could get confused thus:

1) Have an application that only links video pads
2) The demuxer sees audio tags first
3) It creates the pad and adds it to the flow combiner
4) Pushing returns NOT_LINKED
5) The flow combiner only knows the audio pad, returns NOT_LINKED too
6) Streaming stops, internal data error yada yada

Reproduce with:

ges-launch-1.0 /path/to/flv 0 5

(GES creates one uridecodebin per stream type)

This patch creates the pads when parsing the header, but as it seems
some flv files might have headers that lie with respect to the
streams they contain, we still make sure to create the pad
in parse_tag_video and parse_tag_audio.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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