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

fenrir at elivagar.org fenrir at elivagar.org
Sun Sep 25 00:38:23 CEST 2011


From: Laurent Aimar <fenrir at videolan.org>

---
 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[] = {
-- 
1.7.2.5



More information about the ffmpeg-devel mailing list