diff options
| author | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2015-12-19 23:44:53 +0100 |
|---|---|---|
| committer | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2015-12-20 11:57:54 +0100 |
| commit | 9fcfe4a3cdf9a5af0c37758b178965b7b99582d4 (patch) | |
| tree | c754488ea25dd7353fa616e5168740c173fe3d1f /libavformat/mlvdec.c | |
| parent | 70f13abb4f9a376ddc0d2c566739bc3c6a0c47e7 (diff) | |
| download | ffmpeg-9fcfe4a3cdf9a5af0c37758b178965b7b99582d4.tar.gz | |
mlvdec: check that index_entries exist
This fixes NULL pointer dereferencing.
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Diffstat (limited to 'libavformat/mlvdec.c')
| -rw-r--r-- | libavformat/mlvdec.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libavformat/mlvdec.c b/libavformat/mlvdec.c index 48a429eb23..4b3bdc1eca 100644 --- a/libavformat/mlvdec.c +++ b/libavformat/mlvdec.c @@ -365,6 +365,11 @@ static int read_header(AVFormatContext *avctx) if (ast) ast->duration = ast->nb_index_entries; + if ((vst && !vst->nb_index_entries) || (ast && !ast->nb_index_entries)) { + av_log(avctx, AV_LOG_ERROR, "no index entries found\n"); + return AVERROR_INVALIDDATA; + } + if (vst && ast) avio_seek(pb, FFMIN(vst->index_entries[0].pos, ast->index_entries[0].pos), SEEK_SET); else if (vst) |