diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-11-14 01:35:08 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-11-19 03:51:39 +0100 |
| commit | 896c8e38915cc452556c50c7c2f58d72b898f2fc (patch) | |
| tree | 246cc3cdcc9a2b6434a8c378ae7c971dcad52b7e | |
| parent | 53869b687c4ec13aa534a147fb3a3648fe72b191 (diff) | |
| download | ffmpeg-896c8e38915cc452556c50c7c2f58d72b898f2fc.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 3c75df753a..f7d049b459 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -3395,7 +3395,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: |