aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2015-05-15 17:31:58 +0200
committerMichael Niedermayer <michaelni@gmx.at>2015-05-15 17:31:58 +0200
commitade8a46154cb45c88b1cb5c616eaa6320c941187 (patch)
treeaa0ad9c9290a1f5fa35415f76fcfa24b8f718927
parentc131a9fead5bf63215b6e1172b3c5c183cf90b85 (diff)
downloadffmpeg-ade8a46154cb45c88b1cb5c616eaa6320c941187.tar.gz
avcodec/sonic: More completely check sample_rate_index and channels
Fixes CID1271783 Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/sonic.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/libavcodec/sonic.c b/libavcodec/sonic.c
index 81fe1ef5c8..3db77f30a3 100644
--- a/libavcodec/sonic.c
+++ b/libavcodec/sonic.c
@@ -883,13 +883,19 @@ static av_cold int sonic_decode_init(AVCodecContext *avctx)
if (s->version >= 1)
{
+ int sample_rate_index;
s->channels = get_bits(&gb, 2);
- s->samplerate = samplerate_table[get_bits(&gb, 4)];
+ sample_rate_index = get_bits(&gb, 4);
+ if (sample_rate_index >= FF_ARRAY_ELEMS(samplerate_table)) {
+ av_log(avctx, AV_LOG_ERROR, "Invalid sample_rate_index %d\n", sample_rate_index);
+ return AVERROR_INVALIDDATA;
+ }
+ s->samplerate = samplerate_table[sample_rate_index];
av_log(avctx, AV_LOG_INFO, "Sonicv2 chans: %d samprate: %d\n",
s->channels, s->samplerate);
}
- if (s->channels > MAX_CHANNELS)
+ if (s->channels > MAX_CHANNELS || s->channels < 1)
{
av_log(avctx, AV_LOG_ERROR, "Only mono and stereo streams are supported by now\n");
return AVERROR_INVALIDDATA;