[FFmpeg-cvslog] Fixed NULL pointer dereference in id3v2 parser.

Laurent Aimar git at videolan.org
Sun Sep 25 23:06:19 CEST 2011


ffmpeg | branch: master | Laurent Aimar <fenrir at videolan.org> | Sun Sep 25 00:38:23 2011 +0200| [f57b00e89749b559da7cd99a4b630c90617e17d4] | committer: Michael Niedermayer

Fixed NULL pointer dereference in id3v2 parser.

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

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

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

diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c
index 29ba1ab..dfa9a71 100644
--- a/libavformat/id3v2.c
+++ b/libavformat/id3v2.c
@@ -348,7 +348,7 @@ static const ID3v2EMFunc *get_extra_meta_func(const char *tag, int isv34)
             return &ff_id3v2_extra_meta_funcs[i];
         i++;
     }
-    return NULL;
+    return &ff_id3v2_extra_meta_funcs[i];
 }
 
 static void ff_id3v2_parse(AVFormatContext *s, int len, uint8_t version, uint8_t flags, ID3v2ExtraMeta **extra_meta)
@@ -531,7 +531,7 @@ void ff_id3v2_free_extra_meta(ID3v2ExtraMeta **extra_meta)
 
 const ID3v2EMFunc ff_id3v2_extra_meta_funcs[] = {
     { "GEO", "GEOB", read_geobtag, free_geobtag },
-    { NULL }
+    { NULL,  NULL,   NULL,         NULL }
 };
 
 const AVMetadataConv ff_id3v2_34_metadata_conv[] = {



More information about the ffmpeg-cvslog mailing list