diff options
author | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2016-11-08 00:42:23 +0100 |
---|---|---|
committer | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2016-11-08 20:17:31 +0100 |
commit | ff100c9dd97d2f1f456ff38b192edf84f9744738 (patch) | |
tree | a489a72642fc613a807152bdac5f0a50921d12ee | |
parent | 0a245875887430fbdfa9cbd19eb4d337dae93d2e (diff) | |
download | ffmpeg-ff100c9dd97d2f1f456ff38b192edf84f9744738.tar.gz |
matroskadec: fix NULL pointer dereference in webm_dash_manifest_read_header
The code assumes that s->streams[0] is valid.
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
-rw-r--r-- | libavformat/matroskadec.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index d2a691bc06..f79511e4f1 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -3776,6 +3776,11 @@ static int webm_dash_manifest_read_header(AVFormatContext *s) av_log(s, AV_LOG_ERROR, "Failed to read file headers\n"); return -1; } + if (!s->nb_streams) { + matroska_read_close(s); + av_log(s, AV_LOG_ERROR, "No streams found\n"); + return AVERROR_INVALIDDATA; + } if (!matroska->is_live) { buf = av_asprintf("%g", matroska->duration); |