[FFmpeg-cvslog] asfdec: ensure that the whole tag is read.

[Anton Khirnov]

ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Thu Jan 27 10:41:49 2011 +0100| [827048268163bfe41438e0abbb2f8b131d3aa085] | committer: Michael Niedermayer

asfdec: ensure that the whole tag is read.

Signed-off-by: Janne Grunau <janne-ffmpeg at jannau.net>
(cherry picked from commit 042950542d6ee70fde01e4edbb6e6da38ebbaf27)

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

 libavformat/asfdec.c |    9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c
index 229f8ba..eafb71d 100644
--- a/libavformat/asfdec.c
+++ b/libavformat/asfdec.c
@@ -125,13 +125,14 @@ static int get_value(ByteIOContext *pb, int type){
 static void get_tag(AVFormatContext *s, const char *key, int type, int len)
 {
     char *value;
+    int64_t off = url_ftell(s->pb);
 
     if ((unsigned)len >= (UINT_MAX - 1)/2)
         return;
 
     value = av_malloc(2*len+1);
     if (!value)
-        return;
+        goto finish;
 
     if (type == 0) {         // UTF16-LE
         avio_get_str16le(s->pb, len, value, 2*len + 1);
@@ -139,13 +140,13 @@ static void get_tag(AVFormatContext *s, const char *key, int type, int len)
         uint64_t num = get_value(s->pb, type);
         snprintf(value, len, "%"PRIu64, num);
     } else {
-        url_fskip(s->pb, len);
-        av_freep(&value);
         av_log(s, AV_LOG_DEBUG, "Unsupported value type %d in tag %s.\n", type, key);
-        return;
+        goto finish;
     }
     av_metadata_set2(&s->metadata, key, value, 0);
+finish:
     av_freep(&value);
+    url_fseek(s->pb, off + len, SEEK_SET);
 }
 
 static int asf_read_header(AVFormatContext *s, AVFormatParameters *ap)