diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2019-10-04 00:46:08 +0200 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2019-10-20 19:57:51 +0200 |
commit | 315a4459334fdc1e6b18733fb63439e9d090cebd (patch) | |
tree | 653fafd362276b96acf11dfa084b8a4636903972 /libavcodec/agm.c | |
parent | a6ad328256fe6a6ace7d1e15f3515afccf1247fc (diff) |
avcodec/agm: Check for reference frame earlier
Fixes: Timeout (14sec - 120ms)
Fixes: 17824/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AGM_fuzzer-5639825457152000
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/agm.c')
-rw-r--r-- | libavcodec/agm.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/libavcodec/agm.c b/libavcodec/agm.c index 2c4c9805e9..3b2c21e9a7 100644 --- a/libavcodec/agm.c +++ b/libavcodec/agm.c @@ -1117,6 +1117,13 @@ static int decode_frame(AVCodecContext *avctx, void *data, frame->key_frame = s->key_frame; frame->pict_type = s->key_frame ? AV_PICTURE_TYPE_I : AV_PICTURE_TYPE_P; + if (!s->key_frame) { + if (!s->prev_frame->data[0]) { + av_log(avctx, AV_LOG_ERROR, "Missing reference frame.\n"); + return AVERROR_INVALIDDATA; + } + } + if (header) { if (avctx->codec_tag == MKTAG('A', 'G', 'M', '0') || avctx->codec_tag == MKTAG('A', 'G', 'M', '1')) @@ -1186,10 +1193,6 @@ static int decode_frame(AVCodecContext *avctx, void *data, else ret = decode_intra(avctx, gb, frame); } else { - if (!s->prev_frame->data[0]) { - av_log(avctx, AV_LOG_ERROR, "Missing reference frame.\n"); - return AVERROR_INVALIDDATA; - } if (s->prev_frame-> width != frame->width || s->prev_frame->height != frame->height) return AVERROR_INVALIDDATA; |