summaryrefslogtreecommitdiffstats
path: root/libavcodec/h263dec.c
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2018-04-22 21:07:45 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2018-04-25 23:09:47 +0200
commitde841fbea7655b74a9663001e01008a86c88779a (patch)
tree9dce211f02512de62bf22c15aa3ecc977198d9fe /libavcodec/h263dec.c
parent7562567f41aa61860ed5ab03dbb11c1316a34021 (diff)
downloadffmpeg-streaming-de841fbea7655b74a9663001e01008a86c88779a.zip
ffmpeg-streaming-de841fbea7655b74a9663001e01008a86c88779a.tar.gz
avcodec/h263dec: Check slice_ret in mspeg4 slice loop
Fixes infinite loop Fixes: 6858/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_MSMPEG4V3_fuzzer-4681563766784000 Fixes: 6890/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_WMV1_fuzzer-4756103142309888 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/h263dec.c')
-rw-r--r--libavcodec/h263dec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
index 484bf39..eae29fa 100644
--- a/libavcodec/h263dec.c
+++ b/libavcodec/h263dec.c
@@ -641,7 +641,7 @@ retry:
slice_ret = decode_slice(s);
while (s->mb_y < s->mb_height) {
if (s->msmpeg4_version) {
- if (s->slice_height == 0 || s->mb_x != 0 ||
+ if (s->slice_height == 0 || s->mb_x != 0 || slice_ret < 0 ||
(s->mb_y % s->slice_height) != 0 || get_bits_left(&s->gb) < 0)
break;
} else {
OpenPOWER on IntegriCloud