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

changes since this is not currently supported by the decoder.

Originally committed as revision 18157 to svn://svn.ffmpeg.org/ffmpeg/trunk
release/0.6
Justin Ruggles 16 years ago
parent f91eaf5deb
commit e2a38af96e
  1. 19
      libavcodec/flacdec.c

@ -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)) {

Loading…
Cancel
Save