Commit 1f0ad331 authored by jbr's avatar jbr

flacdec: change frame bps validation to return an error value if bps

changes since this is not currently supported by the decoder.


git-svn-id: file:///var/local/repositories/ffmpeg/trunk@18157 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b
parent 61cdb0d9
...@@ -512,25 +512,26 @@ static int decode_frame(FLACContext *s) ...@@ -512,25 +512,26 @@ static int decode_frame(FLACContext *s)
/* bits per sample */ /* bits per sample */
bps_code = get_bits(gb, 3); bps_code = get_bits(gb, 3);
if (bps_code == 0) { if (bps_code == 3 || bps_code == 7) {
bps= s->bps;
} else if ((bps_code != 3) && (bps_code != 7)) {
bps = sample_size_table[bps_code];
} else {
av_log(s->avctx, AV_LOG_ERROR, "invalid sample size code (%d)\n", av_log(s->avctx, AV_LOG_ERROR, "invalid sample size code (%d)\n",
bps_code); bps_code);
return -1; return -1;
} }
if (bps > 16) { bps = sample_size_table[bps_code];
if (bps && bps != s->bps) {
av_log(s->avctx, AV_LOG_ERROR, "switching bps mid-stream is not "
"supported\n");
return -1;
}
if (s->bps > 16) {
s->avctx->sample_fmt = SAMPLE_FMT_S32; s->avctx->sample_fmt = SAMPLE_FMT_S32;
s->sample_shift = 32 - bps; s->sample_shift = 32 - s->bps;
s->is32 = 1; s->is32 = 1;
} else { } else {
s->avctx->sample_fmt = SAMPLE_FMT_S16; s->avctx->sample_fmt = SAMPLE_FMT_S16;
s->sample_shift = 16 - bps; s->sample_shift = 16 - s->bps;
s->is32 = 0; s->is32 = 0;
} }
s->bps = s->avctx->bits_per_raw_sample = bps;
/* reserved bit */ /* reserved bit */
if (get_bits1(gb)) { if (get_bits1(gb)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment