twinvqdec: Check the ibps parameter separately
authorMartin Storsjö <martin@martin.st>
Tue, 17 Sep 2013 16:33:48 +0000 (19:33 +0300)
committerMartin Storsjö <martin@martin.st>
Tue, 17 Sep 2013 20:22:46 +0000 (23:22 +0300)
This is required, since invalid parameters actually could
pass the switch check below.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
libavcodec/twinvqdec.c

index a88b6ff..580c1f2 100644 (file)
@@ -356,6 +356,10 @@ static av_cold int twinvq_decode_init(AVCodecContext *avctx)
                                                  : AV_CH_LAYOUT_STEREO;
 
     ibps = avctx->bit_rate / (1000 * avctx->channels);
+    if (ibps < 8 || ibps > 48) {
+        av_log(avctx, AV_LOG_ERROR, "Bad bitrate per channel value %d\n", ibps);
+        return AVERROR_INVALIDDATA;
+    }
 
     switch ((isampf << 8) + ibps) {
     case (8 << 8) + 8: