diff options
| author | Pete Beardmore <pete.beardmore@msn.com> | 2014-03-09 18:40:56 +0000 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2014-03-14 20:27:21 +0100 |
| commit | d0cd98a63aa40ba16946148590d3cf0648a1dd19 (patch) | |
| tree | f41be82e5140d52fd7288f72a1c8eeaac3e4e11f /test/test_byte_reverse.cxx | |
| parent | 88eae9dabbb57ddc697dd64e33c145775d4bb663 (diff) | |
| download | mpd-d0cd98a63aa40ba16946148590d3cf0648a1dd19.tar.gz mpd-d0cd98a63aa40ba16946148590d3cf0648a1dd19.tar.xz mpd-d0cd98a63aa40ba16946148590d3cf0648a1dd19.zip | |
MultipleOutputs: ensure input_audio_format is zero-initialised
-fixes SIGABRT when mpd is started from a previously paused state
-regression introduced by commit: f5a923b9d16e4c63942a033d1bdb2ab150aae342:
'OutputAll: convert to class, move instance to class Partition'
-input_audio_format was previously declared using the static modifier
ensuring it was zero-initialised by default
-the current default-initialised input_audio_format contains garbage at
runtime which allows the AudioFormat.IsDefined() 'fail fast' test in
MultipleOutputs::Update to pass and the SIGABRT follows in
AudioOutput::Open when passed the invalid input_audio_format struct
-switching AudioFormat.IsDefined() for AudioFormat.IsValid() is an
alternative workaround
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions