You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
gentoo-overlay/media-video/mpv/files/0.18.1/mpv-0.18.1-fix-audio-uninit...

40 lines
1.0 KiB

commit 79974e7ad94e7658c780e3b446a7822a533675e4
Author: wm4 <wm4@nowhere>
Date: Fri Jul 15 23:11:25 2016 +0200
audio: fix crashes due to broken uninit check
Since mixer->ao is always NULL now (it was really just forgotten to be
removed), the uninit call never actually cleared the af field, leaving
a dangling pointer that could be accessed by volume control.
diff --git a/audio/mixer.c b/audio/mixer.c
index a58a814..795edfa 100644
--- a/audio/mixer.c
+++ b/audio/mixer.c
@@ -23,7 +23,6 @@
#include <libavutil/common.h>
#include "config.h"
-#include "audio/out/ao.h"
#include "audio/filter/af.h"
#include "common/global.h"
#include "common/msg.h"
@@ -33,7 +32,6 @@
struct mixer {
struct mp_log *log;
struct MPOpts *opts;
- struct ao *ao;
struct af_stream *af;
// Other stuff
float balance;
@@ -145,8 +143,5 @@ void mixer_reinit_audio(struct mixer *mixer, struct af_stream *af)
*/
void mixer_uninit_audio(struct mixer *mixer)
{
- if (!mixer->ao)
- return;
-
mixer->af = NULL;
}