diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2017-03-01 04:28:23 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2017-05-17 20:35:19 +0200 |
| commit | f6908d087e6bd7b10a2f6d4e9be8cb0faf9fe785 (patch) | |
| tree | 69719c9cb038ee88e5b394090788ebde60117c35 /libavcodec | |
| parent | 3c8c0c0162d2910a19bd2cc38af429e2004b84df (diff) | |
| download | ffmpeg-f6908d087e6bd7b10a2f6d4e9be8cb0faf9fe785.tar.gz | |
avcodec/vp56: Clear dimensions in case of failure in the middle of a resolution change
Similar code is used elsewhere in vp56 to force a more complete reinit in the future.
Fixes null pointer dereference
Fixes: 707/clusterfuzz-testcase-4717453097566208
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 4bed06637729ab000b79250c67d53078300e37c4)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/vp56.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c index b36c99fd33..52f2a7dae8 100644 --- a/libavcodec/vp56.c +++ b/libavcodec/vp56.c @@ -603,13 +603,18 @@ int ff_vp56_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, } ret = ff_get_buffer(avctx, p, AV_GET_BUFFER_FLAG_REF); - if (ret < 0) + if (ret < 0) { + if (res == VP56_SIZE_CHANGE) + ff_set_dimensions(avctx, 0, 0); return ret; + } if (avctx->pix_fmt == AV_PIX_FMT_YUVA420P) { av_frame_unref(s->alpha_context->frames[VP56_FRAME_CURRENT]); if ((ret = av_frame_ref(s->alpha_context->frames[VP56_FRAME_CURRENT], p)) < 0) { av_frame_unref(p); + if (res == VP56_SIZE_CHANGE) + ff_set_dimensions(avctx, 0, 0); return ret; } } |