matroskadec: fix crash when parsing invalid mkv
authorThomas Guillem <thomas@gllm.fr>
Fri, 10 Apr 2015 17:04:51 +0000 (19:04 +0200)
committerAnton Khirnov <anton@khirnov.net>
Sat, 18 Apr 2015 14:49:25 +0000 (16:49 +0200)
CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
libavformat/matroskadec.c

index 7dac7be..93cd4bb 100644 (file)
@@ -1827,8 +1827,12 @@ static int matroska_read_header(AVFormatContext *s)
     matroska->ctx = s;
 
     /* First read the EBML header. */
-    if (ebml_parse(matroska, ebml_syntax, &ebml) ||
-        ebml.version         > EBML_VERSION      ||
+    if (ebml_parse(matroska, ebml_syntax, &ebml) || !ebml.doctype) {
+        av_log(matroska->ctx, AV_LOG_ERROR, "EBML header parsing failed\n");
+        ebml_free(ebml_syntax, &ebml);
+        return AVERROR_INVALIDDATA;
+    }
+    if (ebml.version         > EBML_VERSION      ||
         ebml.max_size        > sizeof(uint64_t)  ||
         ebml.id_length       > sizeof(uint32_t)  ||
         ebml.doctype_version > 3) {