oysteikt/mpd
oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

encoder/flac: fix crash with 32 bit playback

Browse Source 
Copy to encoder->audio_format *after* adjusting the sample format to
S24_P32.

Fixes http://bugs.musicpd.org/view.php?id=4433
This commit is contained in:
Max Kellermann 2015-10-16 18:05:34 +02:00
parent 205fba74cf
commit 4e3d182189
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
NEWS
View File

@ -1,6 +1,8 @@
ver 0.19.11 (not yet released) ver 0.19.11 (not yet released)
* tags * tags
- ape: fix buffer overflow - ape: fix buffer overflow
* encoder
- flac: fix crash with 32 bit playback
ver 0.19.10 (2015/06/21) ver 0.19.10 (2015/06/21)
* input * input

4
src/encoder/plugins/FlacEncoderPlugin.cxx
View File

@ -157,8 +157,6 @@ flac_encoder_open(Encoder *_encoder, AudioFormat &audio_format, Error &error)
struct flac_encoder *encoder = (struct flac_encoder *)_encoder; struct flac_encoder *encoder = (struct flac_encoder *)_encoder;
unsigned bits_per_sample; unsigned bits_per_sample;
encoder->audio_format = audio_format;
/* FIXME: flac should support 32bit as well */ /* FIXME: flac should support 32bit as well */
switch (audio_format.format) { switch (audio_format.format) {
case SampleFormat::S8: case SampleFormat::S8:
@ -178,6 +176,8 @@ flac_encoder_open(Encoder *_encoder, AudioFormat &audio_format, Error &error)
audio_format.format = SampleFormat::S24_P32; audio_format.format = SampleFormat::S24_P32;
} }
encoder->audio_format = audio_format;
/* allocate the encoder */ /* allocate the encoder */
encoder->fse = FLAC__stream_encoder_new(); encoder->fse = FLAC__stream_encoder_new();
if (encoder->fse == nullptr) { if (encoder->fse == nullptr) {