From fc6f314838dc2f192bd42d6bcf67aa9d5280908f Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Wed, 11 May 2016 00:00:52 +0200 Subject: [PATCH] avformat/utils: Check bps before using it in a shift in ff_get_pcm_codec_id() Fixes undefined shift Fixes: usan_shift Found-by: Thomas Guilbert Signed-off-by: Michael Niedermayer (cherry picked from commit ea791c080dd5494b3bee0c618a3f52e371b5f320) Signed-off-by: Michael Niedermayer --- libavformat/utils.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index 5ee4436c71..d40ae64411 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2785,6 +2785,9 @@ enum AVCodecID ff_codec_get_id(const AVCodecTag *tags, unsigned int tag) enum AVCodecID ff_get_pcm_codec_id(int bps, int flt, int be, int sflags) { + if (bps > 64U) + return AV_CODEC_ID_NONE; + if (flt) { switch (bps) { case 32: