diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2017-06-06 16:01:16 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2017-06-06 16:56:23 +0200 |
| commit | 07944df9a7d76a212ca898e37d91a660cd4ee0e7 (patch) | |
| tree | e8dbc4cb7f4957a44fd022c093b53c0e9f2beab4 | |
| parent | 34887d091d53edb530aca66de98ce545c4fcea5a (diff) | |
| download | ffmpeg-07944df9a7d76a212ca898e37d91a660cd4ee0e7.tar.gz | |
avcodec/takdec: Fix multiple runtime error: signed integer overflow: 637072 * 4096 cannot be represented in type 'int'
Fixes: 2079/clusterfuzz-testcase-minimized-5345861779324928
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 e4efd41b83e78c7f2ee3e74bee90226110743a8e)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/takdec.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/takdec.c b/libavcodec/takdec.c index 85ef56644b..dbcc11747d 100644 --- a/libavcodec/takdec.c +++ b/libavcodec/takdec.c @@ -862,7 +862,7 @@ static int tak_decode_frame(AVCodecContext *avctx, void *data, if (s->sample_shift[chan] > 0) for (i = 0; i < s->nb_samples; i++) - decoded[i] *= 1 << s->sample_shift[chan]; + decoded[i] *= 1U << s->sample_shift[chan]; } } @@ -904,7 +904,7 @@ static int tak_decode_frame(AVCodecContext *avctx, void *data, for (chan = 0; chan < avctx->channels; chan++) { int32_t *samples = (int32_t *)frame->extended_data[chan]; for (i = 0; i < s->nb_samples; i++) - samples[i] *= 1 << 8; + samples[i] *= 1U << 8; } break; } |