diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2019-11-21 23:02:56 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2019-12-15 01:01:22 +0100 |
| commit | 9fe07908c3f67d59cf4db5668d61b34506189590 (patch) | |
| tree | 334dc80f71d81f851c8a7b6166bf7691d35bf3bc | |
| parent | 01593278cef06dbb4491d50d03b72198d2848adf (diff) | |
| download | ffmpeg-9fe07908c3f67d59cf4db5668d61b34506189590.tar.gz | |
avcodec/adpcm: Clip predictor for APC
Fixes: signed integer overflow: -2147483648 - 13 cannot be represented in type 'int'
Fixes: 18893/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ADPCM_IMA_APC_fuzzer-5630760442920960
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/adpcm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c index 73beb5050e..b99a21002c 100644 --- a/libavcodec/adpcm.c +++ b/libavcodec/adpcm.c @@ -140,8 +140,8 @@ static av_cold int adpcm_decode_init(AVCodecContext * avctx) break; case AV_CODEC_ID_ADPCM_IMA_APC: if (avctx->extradata && avctx->extradata_size >= 8) { - c->status[0].predictor = AV_RL32(avctx->extradata); - c->status[1].predictor = AV_RL32(avctx->extradata + 4); + c->status[0].predictor = av_clip_intp2(AV_RL32(avctx->extradata ), 18); + c->status[1].predictor = av_clip_intp2(AV_RL32(avctx->extradata + 4), 18); } break; case AV_CODEC_ID_ADPCM_IMA_WS: |