aboutsummaryrefslogtreecommitdiffstats
path: root/src/audioOutputs
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2006-07-17 01:28:38 +0000
committerEric Wong <normalperson@yhbt.net>2006-07-17 01:28:38 +0000
commit5f508702229febe6c042de2d4fe6d23b855cd410 (patch)
tree17f081fcfc0ba3c5cbde288aff5a44a83652b937 /src/audioOutputs
parent368034e1995e8cb27b96f98df0fe07542f446023 (diff)
downloadmpd-5f508702229febe6c042de2d4fe6d23b855cd410.tar.gz
mpd-5f508702229febe6c042de2d4fe6d23b855cd410.tar.xz
mpd-5f508702229febe6c042de2d4fe6d23b855cd410.zip
alsa: fix memory leaks from snd_*_open*()
ALSA uses a global config structure that's overwritten (and not free'd) every time one of those functions is called, so we have to manually call snd_config_update_free_global() to release it. Hint taken from MEMORY-LEAK in the ALSA source code git-svn-id: https://svn.musicpd.org/mpd/trunk@4381 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to '')
-rw-r--r--src/audioOutputs/audioOutput_alsa.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/audioOutputs/audioOutput_alsa.c b/src/audioOutputs/audioOutput_alsa.c
index 351647ace..5150dd502 100644
--- a/src/audioOutputs/audioOutput_alsa.c
+++ b/src/audioOutputs/audioOutput_alsa.c
@@ -106,6 +106,7 @@ static int alsa_testDefault(void)
int ret = snd_pcm_open(&handle, "default", SND_PCM_STREAM_PLAYBACK,
SND_PCM_NONBLOCK);
+ snd_config_update_free_global();
if(ret) {
WARNING("Error opening default alsa device: %s\n",
@@ -152,6 +153,7 @@ static int alsa_openDevice(AudioOutput * audioOutput)
err = snd_pcm_open(&ad->pcmHandle, ad->device,
SND_PCM_STREAM_PLAYBACK, SND_PCM_NONBLOCK);
+ snd_config_update_free_global();
if(err < 0) {
ad->pcmHandle = NULL;
goto error;