ext/ogg/gstoggdemux.c: Don't add pads and activate them for skeleton streams. These...

Original commit message from CVS:
Patch by: Alessandro Dessina <alessandro nnva org>
* ext/ogg/gstoggdemux.c: (gst_ogg_demux_deactivate_current_chain),
(gst_ogg_demux_activate_chain):
Don't add pads and activate them for skeleton streams. These are already
handled inside oggdemux. Fixes bug #537599.

diff --git a/ChangeLog b/ChangeLog
index 4897d5d8e24915d2644a18282d5b998fbce2f06f..993599d5a4df68d5d7137f3273b69fe52b89c30b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2008-08-23  Sebastian Dröge  <sebastian.droege@collabora.co.uk>
+
+       Patch by: Alessandro Dessina <alessandro nnva org>
+
+       * ext/ogg/gstoggdemux.c: (gst_ogg_demux_deactivate_current_chain),
+       (gst_ogg_demux_activate_chain):
+       Don't add pads and activate them for skeleton streams. These are already
+       handled inside oggdemux. Fixes bug #537599.
+
 2008-08-22  Wim Taymans  <wim.taymans@collabora.co.uk>
 
        * ext/vorbis/vorbisdec.c: (vorbis_dec_change_state):

diff --git a/ext/ogg/gstoggdemux.c b/ext/ogg/gstoggdemux.c
index bb059e848d623e3f4a17493a984999bb1238b12c..469a18a7a205bbea90a1e2afecc95bbadae9f2a5 100644 (file)
--- a/ext/ogg/gstoggdemux.c
+++ b/ext/ogg/gstoggdemux.c
@@ -1746,6 +1746,9 @@ gst_ogg_demux_deactivate_current_chain (GstOggDemux * ogg)
   for (i = 0; i < chain->streams->len; i++) {
     GstOggPad *pad = g_array_index (chain->streams, GstOggPad *, i);
 
+    if (pad->is_skeleton)
+      continue;
+
     gst_pad_push_event (GST_PAD_CAST (pad), gst_event_new_eos ());
 
     GST_DEBUG_OBJECT (ogg, "removing pad %" GST_PTR_FORMAT, pad);
@@ -1786,6 +1789,10 @@ gst_ogg_demux_activate_chain (GstOggDemux * ogg, GstOggChain * chain,
       GstOggPad *pad;
 
       pad = g_array_index (chain->streams, GstOggPad *, i);
+
+      if (pad->is_skeleton)
+        continue;
+
       GST_DEBUG_OBJECT (ogg, "adding pad %" GST_PTR_FORMAT, pad);
 
       /* mark discont */