id3v2: explicitly seek to the end of the tag after reading
authorAnton Khirnov <anton@khirnov.net>
Thu, 17 Mar 2011 06:10:23 +0000 (07:10 +0100)
committerRonald S. Bultje <rsbultje@gmail.com>
Mon, 21 Mar 2011 18:56:59 +0000 (14:56 -0400)
Current code might stop in the middle of an invalid tag.

fixes issue2650

Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>
libavformat/id3v2.c

index bb36020..fb4c761 100644 (file)
@@ -186,7 +186,7 @@ static void ff_id3v2_parse(AVFormatContext *s, int len, uint8_t version, uint8_t
 {
     int isv34, tlen, unsync;
     char tag[5];
-    int64_t next;
+    int64_t next, end = avio_tell(s->pb) + len;
     int taghdrlen;
     const char *reason;
     AVIOContext pb;
@@ -284,8 +284,9 @@ static void ff_id3v2_parse(AVFormatContext *s, int len, uint8_t version, uint8_t
         avio_skip(s->pb, len);
     }
     if (version == 4 && flags & 0x10) /* Footer preset, always 10 bytes, skip over it */
-        avio_skip(s->pb, 10);
+        end += 10;
 
+    avio_seek(s->pb, end, SEEK_SET);
     av_free(buffer);
     return;