aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2017-08-15 03:32:43 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2017-08-17 00:27:31 +0200
commit9bfa8b692ebd3718e1ba6f67bfc61a29fc3f1b7d (patch)
tree45acbeee7021eabbebc4f6bc0babeca668071d40
parente154826a2f020f966a677cfbff4a97d7a0a94d5f (diff)
downloadffmpeg-9bfa8b692ebd3718e1ba6f67bfc61a29fc3f1b7d.tar.gz
avcodec/diracdec: Check perspective_exp and zrs_exp.
Fixes: undefined shift Fixes: runtime error: shift exponent 264 is too large for 32-bit type 'int' Fixes: 2860/clusterfuzz-testcase-minimized-4672811689836544 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit 1e6cab874512070b36267a5a53fd053f90072fa2) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/diracdec.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec.c
index 450a5bb91e..80c4e66eed 100644
--- a/libavcodec/diracdec.c
+++ b/libavcodec/diracdec.c
@@ -1082,6 +1082,10 @@ static int dirac_unpack_prediction_parameters(DiracContext *s)
s->globalmc[ref].perspective[0] = dirac_get_se_golomb(gb);
s->globalmc[ref].perspective[1] = dirac_get_se_golomb(gb);
}
+ if (s->globalmc[ref].perspective_exp + (uint64_t)s->globalmc[ref].zrs_exp > 30) {
+ return AVERROR_INVALIDDATA;
+ }
+
}
}