Merge remote-tracking branch 'qatar/release/0.8' into release/0.10
authorMichael Niedermayer <michaelni@gmx.at>
Sun, 1 Apr 2012 22:52:23 +0000 (00:52 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Sun, 1 Apr 2012 22:52:23 +0000 (00:52 +0200)
* qatar/release/0.8:
  id3v2: fix skipping extended header in id3v2.4

Conflicts:
libavformat/id3v2.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavformat/id3v2.c

@@@ -464,21 -448,16 +464,21 @@@ static void ff_id3v2_parse(AVFormatCont
  
      unsync = flags & 0x80;
  
-     /* Extended header present, just skip over it */
-     if (isv34 && flags & 0x40) {
-         int size = get_size(s->pb, 4);
-         if (size < 6) {
-             reason = "extended header too short.";
+     if (isv34 && flags & 0x40) { /* Extended header present, just skip over it */
+         int extlen = get_size(s->pb, 4);
+         if (version == 4)
+             extlen -= 4;     // in v2.4 the length includes the length field we just read
+         if (extlen < 0) {
+             reason = "invalid extended header length";
              goto error;
          }
-         len -= size;
+         avio_skip(s->pb, extlen);
++        len -= extlen + 4;
 +        if (len < 0) {
 +            reason = "extended header too long.";
 +            goto error;
 +        }
-         /* already seeked past size, skip the reset */
-         size -= 4;
-         avio_skip(s->pb, size);
      }
  
      while (len >= taghdrlen) {