audio/out/push: when using audio wait fallback, recheck condition

If calling ao->driver->wait() fails, we need to fallback to timeout-
based waiting. But it could be that at this point, the mutex was already
released (and then re-acquired). So we need to recheck the condition in
order to avoid missed wakeups.

This probably wasn't an actually occurring problem, but still could
cause a small race-condition window if the dynamic fallback is actually
used.

1 files changed, 2 insertions, 1 deletions

diff --git a/audio/out/push.c b/audio/out/push.c
index 187d5ef03a..5872b01913 100644
--- a/audio/out/push.c
+++ b/audio/out/push.c
@@ -352,7 +352,8 @@ static void *playthread(void *arg)
                     if (ao->driver->get_delay)
                         timeout = ao->driver->get_delay(ao);
                     timeout *= 0.25; // wake up if 25% played
-                    mpthread_cond_timedwait_rel(&p->wakeup, &p->lock, timeout);
+                    if (!p->need_wakeup)
+                        mpthread_cond_timedwait_rel(&p->wakeup, &p->lock, timeout);
                 }
             }
             MP_STATS(ao, "end audio wait");