[FFmpeg-cvslog] apetag: fix the amount of data read from binary tags.

Anton Khirnov git at videolan.org
Mon Jul 2 21:24:14 CEST 2012


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Tue Jun 26 18:54:00 2012 +0200| [b72767df8e4f496da36138aa7127e408c1cde7f8] | committer: Anton Khirnov

apetag: fix the amount of data read from binary tags.

Substract the filename size from the data size.

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

 libavformat/apetag.c |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/libavformat/apetag.c b/libavformat/apetag.c
index e5c839e..f8653d3 100644
--- a/libavformat/apetag.c
+++ b/libavformat/apetag.c
@@ -59,7 +59,12 @@ static int ape_tag_read_field(AVFormatContext *s)
         AVStream *st = avformat_new_stream(s, NULL);
         if (!st)
             return AVERROR(ENOMEM);
-        avio_get_str(pb, size, filename, sizeof(filename));
+
+        size -= avio_get_str(pb, size, filename, sizeof(filename));
+        if (size <= 0) {
+            av_log(s, AV_LOG_WARNING, "Skipping binary tag '%s'.\n", key);
+            return 0;
+        }
         st->codec->extradata = av_malloc(size + FF_INPUT_BUFFER_PADDING_SIZE);
         if (!st->codec->extradata)
             return AVERROR(ENOMEM);



More information about the ffmpeg-cvslog mailing list