diff options
author | wm4 <wm4@mplayer2.org> | 2012-01-14 09:06:16 +0100 |
---|---|---|
committer | Uoti Urpala <uau@mplayer2.org> | 2012-03-09 20:48:54 +0200 |
commit | 8a6b0b813a1431ca50986c90d97b72fa6640dbea (patch) | |
tree | 517f7d5ccb31266de04ed8dd4ee10ffee75e9855 | |
parent | 1f6ba56d6ab699f5a6137a176f8183c82046be43 (diff) | |
download | mpv-8a6b0b813a1431ca50986c90d97b72fa6640dbea.tar.bz2 mpv-8a6b0b813a1431ca50986c90d97b72fa6640dbea.tar.xz |
af: fix crash when trying to use volume controls with AC3 pass-through
Changing the volume when softvol is enabled or if the audio output driver
doesn't support volume controls causes insertion of the "volume" filter.
This fails with AC3. Since the filter wasn't removed after that, and the
filter chain was in a bogus state, random crashes occured past this
point.
Fix it by reinitializing the filter chain completely on failure. Volume
controls simply won't work. (This can't be fixed, because AC3 is a
compressed format, and would require additional decoding/encoding passes
in order to support arbitrary volume changes.)
This also affects balance controls.
-rw-r--r-- | libaf/af.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/libaf/af.c b/libaf/af.c index 82a7d7398b..e4015727ac 100644 --- a/libaf/af.c +++ b/libaf/af.c @@ -278,6 +278,11 @@ static void af_print_filter_chain(af_stream_t* s) mp_msg(MSGT_AFILTER, MSGL_V, "\n"); } +// Warning: +// A failed af_reinit() leaves the audio chain behind in a useless, broken +// state (for example, format filters that were tentatively inserted stay +// inserted). +// In that case, you should always rebuild the filter chain, or abort. int af_reinit(af_stream_t* s, af_instance_t* af) { do{ @@ -587,7 +592,9 @@ af_instance_t* af_add(af_stream_t* s, char* name){ // Reinitalize the filter list if(AF_OK != af_reinit(s, s->first) || AF_OK != fixup_output_format(s)){ - free(new); + while (s->first) + af_remove(s, s->first); + af_init(s); return NULL; } return new; |