diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2017-07-12 16:24:18 +0200 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2017-07-12 16:32:27 +0200 |
commit | c28f648b19dd36ff9bc869ad527a1569a0b623e2 (patch) | |
tree | 1a591c5777ccb2fa94847ee2ce09d6804291512f /libavcodec/mjpegdec.c | |
parent | 80b9e40b6f1e15db9f36c195e7375e65f6b4924f (diff) | |
download | ffmpeg-c28f648b19dd36ff9bc869ad527a1569a0b623e2.tar.gz |
avcodec/mjpegdec: Clip DC also on the negative side.
Fixes: runtime error: signed integer overflow: -16711425 + -2130772346 cannot be represented in type 'int'
Fixes: 2533/clusterfuzz-testcase-minimized-5372857678823424
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/mjpegdec.c')
-rw-r--r-- | libavcodec/mjpegdec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index e4e99e04e2..387ceadf55 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -716,7 +716,7 @@ static int decode_block(MJpegDecodeContext *s, int16_t *block, int component, return AVERROR_INVALIDDATA; } val = val * quant_matrix[0] + s->last_dc[component]; - val = FFMIN(val, 32767); + val = av_clip_int16(val); s->last_dc[component] = val; block[0] = val; /* AC coefs */ |