From 3218a5808229b06bd7e2d41a912e7e7031525d56 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Ekstr=C3=B6m?= Date: Thu, 20 Sep 2018 20:11:45 +0300 Subject: ao_alsa: early exit get_space if paused or ALSA is not ready This has been way too long coming, and for me to notice that a whole lot of ao_alsa functions do an early return if the AO is paused. For the STATE_SETUP case, I had this reproduced once, and never since. Still, seems like we can start calling this function before the ALSA device has been fully initialized so we might as well early exit in that case. --- audio/out/ao_alsa.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/audio/out/ao_alsa.c b/audio/out/ao_alsa.c index 32f08bef59..6b93b854c0 100644 --- a/audio/out/ao_alsa.c +++ b/audio/out/ao_alsa.c @@ -940,6 +940,11 @@ static int get_space(struct ao *ao) { struct priv *p = ao->priv; + // in case of pausing or the device still being configured, + // just return our buffer size. + if (p->paused || snd_pcm_state(p->alsa) == SND_PCM_STATE_SETUP) + return p->buffersize; + snd_pcm_sframes_t space = snd_pcm_avail(p->alsa); if (space < 0) { MP_ERR(ao, "Error received from snd_pcm_avail (%ld, %s)!\n", -- cgit v1.2.3