id3v2: fix skipping extended header in id3v2.4
authorAnton Khirnov <anton@khirnov.net>
Sat, 31 Mar 2012 05:52:42 +0000 (07:52 +0200)
committerAnton Khirnov <anton@khirnov.net>
Sun, 1 Apr 2012 07:02:24 +0000 (09:02 +0200)
In v2.4, the length includes the length field itself.

libavformat/id3v2.c

index 9488fd8..3891897 100644 (file)
@@ -561,8 +561,17 @@ static void ff_id3v2_parse(AVFormatContext *s, int len, uint8_t version, uint8_t
 
     unsync = flags & 0x80;
 
-    if (isv34 && flags & 0x40) /* Extended header present, just skip over it */
-        avio_skip(s->pb, get_size(s->pb, 4));
+    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;
+        }
+        avio_skip(s->pb, extlen);
+    }
 
     while (len >= taghdrlen) {
         unsigned int tflags = 0;