[vlc-commits] Skip all id3tag in the demuxer, not just the first one.

Luc Saillard git at videolan.org
Mon Jul 18 16:57:38 CEST 2011


vlc | branch: master | Luc Saillard <luc.saillard at sfr.com> | Mon Jul 18 10:29:19 2011 +0200| [ab6aece328aa884cdfde44b0081d3d252db325e1] | committer: Rémi Denis-Courmont

Skip all id3tag in the demuxer, not just the first one.

Some audio file can contains severall id3tag. Apple use it to insert a
jpeg in an audio only stream to display an image. So just iterate others
all id3tag, and discard them.

Signed-off-by: Rémi Denis-Courmont <remi at remlab.net>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=ab6aece328aa884cdfde44b0081d3d252db325e1
---

 src/input/demux.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/input/demux.c b/src/input/demux.c
index 117b5ef..6b357c0 100644
--- a/src/input/demux.c
+++ b/src/input/demux.c
@@ -181,8 +181,9 @@ demux_t *demux_New( vlc_object_t *p_obj, input_thread_t *p_parent_input,
         /* ID3/APE tags will mess-up demuxer probing so we skip it here.
          * ID3/APE parsers will called later on in the demuxer to access the
          * skipped info. */
-        if( !SkipID3Tag( p_demux ) )
-            SkipAPETag( p_demux );
+        while (SkipID3Tag( p_demux ))
+          ;
+        SkipAPETag( p_demux );
 
         p_demux->p_module =
             module_need( p_demux, "demux", psz_module,



More information about the vlc-commits mailing list