diff options
author | foo86 <foo86@users.noreply.github.com> | 2014-03-10 00:32:00 +0400 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2014-03-11 00:21:32 +0100 |
commit | dba25e13c08a02a3734ce7891dc1652826ce15b1 (patch) | |
tree | 5f1080b36fdf16dd42e49773544b00e53e17767f | |
parent | 623b97e45a90408b91931f213f99de97092b8e76 (diff) | |
download | mpv-dba25e13c08a02a3734ce7891dc1652826ce15b1.tar.bz2 mpv-dba25e13c08a02a3734ce7891dc1652826ce15b1.tar.xz |
ao_alsa: check ALSA PCM state before pause and resume
It is possible to have ao->reset() called between ao->pause() and
ao->resume() when seeking during the pause. If the underlying PCM
supports pausing, resuming an already reset PCM will produce an error.
Avoid that by explicitly checking PCM state before calling
snd_pcm_pause().
Signed-off-by: wm4 <wm4@nowhere>
-rw-r--r-- | audio/out/ao_alsa.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/audio/out/ao_alsa.c b/audio/out/ao_alsa.c index e8b3405bb0..08ddac75ea 100644 --- a/audio/out/ao_alsa.c +++ b/audio/out/ao_alsa.c @@ -561,9 +561,11 @@ static void audio_pause(struct ao *ao) int err; if (p->can_pause) { - p->delay_before_pause = get_delay(ao); - err = snd_pcm_pause(p->alsa, 1); - CHECK_ALSA_ERROR("pcm pause error"); + if (snd_pcm_state(p->alsa) == SND_PCM_STATE_RUNNING) { + p->delay_before_pause = get_delay(ao); + err = snd_pcm_pause(p->alsa, 1); + CHECK_ALSA_ERROR("pcm pause error"); + } } else { MP_VERBOSE(ao, "pause not supported by hardware\n"); if (snd_pcm_delay(p->alsa, &p->prepause_frames) < 0 @@ -591,8 +593,10 @@ static void audio_resume(struct ao *ao) } if (p->can_pause) { - err = snd_pcm_pause(p->alsa, 0); - CHECK_ALSA_ERROR("pcm resume error"); + if (snd_pcm_state(p->alsa) == SND_PCM_STATE_PAUSED) { + err = snd_pcm_pause(p->alsa, 0); + CHECK_ALSA_ERROR("pcm resume error"); + } } else { MP_VERBOSE(ao, "resume not supported by hardware\n"); err = snd_pcm_prepare(p->alsa); |