Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#43 Decoding non 8/16/24 bps to raw format should fail cleanly

open
nobody
None
5
2012-12-12
2010-07-16
Christopher Key
No

When decoding a FLAC file with bits per sample that is not a multiple of 8 and outputting in raw format, the decoder gets as far as the write callback before realising that it doesn't know how to write the data. It then fails with an unhelpful message (or trips an assert() in debug mode). The attached patch performs the check earlier, and gives a more helpful error message.

#v+
# metaflac --show-bps data_20bps.flac
20
#v-

without patch:

#v+
# flac -dc --force-raw-format --sign signed --endian little data_20bps.flac

flac 1.2.1, Copyright (C) 2000,2001,2002,2003,2004,2005,2006,2007 Josh Coalson
flac comes with ABSOLUTELY NO WARRANTY. This is free software, and you are
welcome to redistribute it under certain conditions. Type `flac' for details.

data_20bps.flac: ERROR while decoding data
state = FLAC__STREAM_DECODER_READ_FRAME
#v-

with patch:

#v+
# flac -dc --force-raw-format --sign signed --endian little data_20bps.flac

flac 1.2.1, Copyright (C) 2000,2001,2002,2003,2004,2005,2006,2007 Josh Coalson
flac comes with ABSOLUTELY NO WARRANTY. This is free software, and you are
welcome to redistribute it under certain conditions. Type `flac' for details.

data_20bps.flac: ERROR: bits per sample is 20, must be 8/16/24 for raw format output
#v-

Discussion