[Bug 771040] New: tsdemux: program change causes pad name clash

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 771040
           Summary: tsdemux: program change causes pad name clash
    Classification: Platform
           Product: GStreamer
           Version: unspecified
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: knorr.jesper at gmail.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Created attachment 335069
  --> https://bugzilla.gnome.org/attachment.cgi?id=335069&action=edit
[PATCH] tsdemux: Remove old pads before adding new pads

It seems there is a wrong order in how things are done in
gst_ts_demux_program_started.

1. If there was a previous program, the streams are drained, which is all good.
2. Next the new stream pads are created and activated.
3. If there was a previous program, it is deactivated and freed

If the programs shares streams, step no 2 will try to create a pad with a name
that already exists.

The attached patch changes the order of 2 and 3, such that the old program is
freed before we create the streams for the new program.

This might be related to https://bugzilla.gnome.org/show_bug.cgi?id=730960

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