diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-11-14 01:35:08 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-12-06 12:40:49 +0100 |
| commit | 00dc345a83b882e3e86b76d37c11d8ad1257609b (patch) | |
| tree | fecbe7988c814f3741281981ffb9cbab3eb78235 | |
| parent | ac302efb9151addfc9d45495d56592ba6fd384b2 (diff) | |
| download | ffmpeg-00dc345a83b882e3e86b76d37c11d8ad1257609b.tar.gz | |
avcodec/utils: Better check for channels in av_get_audio_frame_duration()
Fixes integer overflow
Fixes: 0c2625f236ced104d402b4a03c0d65c7/asan_generic_274e1ce_5990_9314e7a67c26aecf011b178ade9f217c.avi
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 4e16ad2868a1819de6680fc355a8eb20164adaea)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 0901e75d85..d249507c5f 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -3263,7 +3263,7 @@ int av_get_audio_frame_duration(AVCodecContext *avctx, int frame_bytes) return frame_bytes * 8 / bps; } - if (ch > 0) { + if (ch > 0 && ch < INT_MAX/16) { /* calc from frame_bytes and channels */ switch (id) { case AV_CODEC_ID_ADPCM_AFC: |