[FFmpeg-cvslog] id3v2: don't discard the whole tag when encountering empty frames.

[Anton Khirnov]

ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Sun Oct  2 07:16:09 2011 +0200| [1e18d32d01fc0ac784a0d592b46215bfbdcc579d] | committer: Anton Khirnov

id3v2: don't discard the whole tag when encountering empty frames.

While they're technically invalid, it's better to skip them and try to
read the rest of the tag.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1e18d32d01fc0ac784a0d592b46215bfbdcc579d
---

 libavformat/id3v2.c |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c
index 68c1709..a3d7ae6 100644
--- a/libavformat/id3v2.c
+++ b/libavformat/id3v2.c
@@ -401,13 +401,19 @@ static void ff_id3v2_parse(AVFormatContext *s, int len, uint8_t version, uint8_t
             tag[3] = 0;
             tlen = avio_rb24(s->pb);
         }
-        if (tlen <= 0 || tlen > len - taghdrlen) {
+        if (tlen < 0 || tlen > len - taghdrlen) {
             av_log(s, AV_LOG_WARNING, "Invalid size in frame %s, skipping the rest of tag.\n", tag);
             break;
         }
         len -= taghdrlen + tlen;
         next = avio_tell(s->pb) + tlen;
 
+        if (!tlen) {
+            if (tag[0])
+                av_log(s, AV_LOG_DEBUG, "Invalid empty frame %s, skipping.\n", tag);
+            continue;
+        }
+
         if (tflags & ID3v2_FLAG_DATALEN) {
             avio_rb32(s->pb);
             tlen -= 4;