Commit 3c795698 authored by Michael Chinen's avatar Michael Chinen Committed by Justin Ruggles

Add log_level_offset parameter to ff_flac_decode_frame_header(). It will be used

to optionally silence the error messages.
Patch by Michael Chinen [mchinen at gmail]

Originally committed as revision 25912 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent 71b6fc21
...@@ -33,13 +33,13 @@ static int64_t get_utf8(GetBitContext *gb) ...@@ -33,13 +33,13 @@ static int64_t get_utf8(GetBitContext *gb)
} }
int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
FLACFrameInfo *fi) FLACFrameInfo *fi, int log_level_offset)
{ {
int bs_code, sr_code, bps_code; int bs_code, sr_code, bps_code;
/* frame sync code */ /* frame sync code */
if ((get_bits(gb, 15) & 0x7FFF) != 0x7FFC) { if ((get_bits(gb, 15) & 0x7FFF) != 0x7FFC) {
av_log(avctx, AV_LOG_ERROR, "invalid sync code\n"); av_log(avctx, AV_LOG_ERROR + log_level_offset, "invalid sync code\n");
return -1; return -1;
} }
...@@ -58,14 +58,14 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, ...@@ -58,14 +58,14 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
} else if (fi->ch_mode <= FLAC_CHMODE_MID_SIDE) { } else if (fi->ch_mode <= FLAC_CHMODE_MID_SIDE) {
fi->channels = 2; fi->channels = 2;
} else { } else {
av_log(avctx, AV_LOG_ERROR, "invalid channel mode: %d\n", fi->ch_mode); av_log(avctx, AV_LOG_ERROR + log_level_offset, "invalid channel mode: %d\n", fi->ch_mode);
return -1; return -1;
} }
/* bits per sample */ /* bits per sample */
bps_code = get_bits(gb, 3); bps_code = get_bits(gb, 3);
if (bps_code == 3 || bps_code == 7) { if (bps_code == 3 || bps_code == 7) {
av_log(avctx, AV_LOG_ERROR, "invalid sample size code (%d)\n", av_log(avctx, AV_LOG_ERROR + log_level_offset, "invalid sample size code (%d)\n",
bps_code); bps_code);
return -1; return -1;
} }
...@@ -73,20 +73,20 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, ...@@ -73,20 +73,20 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
/* reserved bit */ /* reserved bit */
if (get_bits1(gb)) { if (get_bits1(gb)) {
av_log(avctx, AV_LOG_ERROR, "broken stream, invalid padding\n"); av_log(avctx, AV_LOG_ERROR + log_level_offset, "broken stream, invalid padding\n");
return -1; return -1;
} }
/* sample or frame count */ /* sample or frame count */
fi->frame_or_sample_num = get_utf8(gb); fi->frame_or_sample_num = get_utf8(gb);
if (fi->frame_or_sample_num < 0) { if (fi->frame_or_sample_num < 0) {
av_log(avctx, AV_LOG_ERROR, "sample/frame number invalid; utf8 fscked\n"); av_log(avctx, AV_LOG_ERROR + log_level_offset, "sample/frame number invalid; utf8 fscked\n");
return -1; return -1;
} }
/* blocksize */ /* blocksize */
if (bs_code == 0) { if (bs_code == 0) {
av_log(avctx, AV_LOG_ERROR, "reserved blocksize code: 0\n"); av_log(avctx, AV_LOG_ERROR + log_level_offset, "reserved blocksize code: 0\n");
return -1; return -1;
} else if (bs_code == 6) { } else if (bs_code == 6) {
fi->blocksize = get_bits(gb, 8) + 1; fi->blocksize = get_bits(gb, 8) + 1;
...@@ -106,7 +106,7 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, ...@@ -106,7 +106,7 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
} else if (sr_code == 14) { } else if (sr_code == 14) {
fi->samplerate = get_bits(gb, 16) * 10; fi->samplerate = get_bits(gb, 16) * 10;
} else { } else {
av_log(avctx, AV_LOG_ERROR, "illegal sample rate code %d\n", av_log(avctx, AV_LOG_ERROR + log_level_offset, "illegal sample rate code %d\n",
sr_code); sr_code);
return -1; return -1;
} }
...@@ -115,7 +115,7 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, ...@@ -115,7 +115,7 @@ int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
skip_bits(gb, 8); skip_bits(gb, 8);
if (av_crc(av_crc_get_table(AV_CRC_8_ATM), 0, gb->buffer, if (av_crc(av_crc_get_table(AV_CRC_8_ATM), 0, gb->buffer,
get_bits_count(gb)/8)) { get_bits_count(gb)/8)) {
av_log(avctx, AV_LOG_ERROR, "header crc mismatch\n"); av_log(avctx, AV_LOG_ERROR + log_level_offset, "header crc mismatch\n");
return -1; return -1;
} }
......
...@@ -131,8 +131,9 @@ int ff_flac_get_max_frame_size(int blocksize, int ch, int bps); ...@@ -131,8 +131,9 @@ int ff_flac_get_max_frame_size(int blocksize, int ch, int bps);
* @param avctx AVCodecContext to use as av_log() context * @param avctx AVCodecContext to use as av_log() context
* @param gb GetBitContext from which to read frame header * @param gb GetBitContext from which to read frame header
* @param[out] fi frame information * @param[out] fi frame information
* @param log_level_offset log level offset. can be used to silence error messages.
* @return non-zero on error, 0 if ok * @return non-zero on error, 0 if ok
*/ */
int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb, int ff_flac_decode_frame_header(AVCodecContext *avctx, GetBitContext *gb,
FLACFrameInfo *fi); FLACFrameInfo *fi, int log_level_offset);
#endif /* AVCODEC_FLAC_H */ #endif /* AVCODEC_FLAC_H */
...@@ -476,7 +476,7 @@ static int decode_frame(FLACContext *s) ...@@ -476,7 +476,7 @@ static int decode_frame(FLACContext *s)
GetBitContext *gb = &s->gb; GetBitContext *gb = &s->gb;
FLACFrameInfo fi; FLACFrameInfo fi;
if (ff_flac_decode_frame_header(s->avctx, gb, &fi)) { if (ff_flac_decode_frame_header(s->avctx, gb, &fi, 0)) {
av_log(s->avctx, AV_LOG_ERROR, "invalid frame header\n"); av_log(s->avctx, AV_LOG_ERROR, "invalid frame header\n");
return -1; return -1;
} }
......
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