winepulse.drv: Check that the server connection actually succeeded.

When there is no PA server running, some older PulseAudio versions have pa_mainloop_iterate fail instead of setting the connection state to failed. So, we should also check the connection state after exiting the loop to ensure it really succeeded before continuing. Signed-off-by: Andrew Eikum <aeikum@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winepulse.drv: Check that the server connection actually succeeded.
0766334d · Andrew Eikum · Alexandre Julliard · 10deca7c · 0766334d
Commit 0766334d authored Mar 04, 2016 by Andrew Eikum Committed by Alexandre Julliard Mar 07, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

mmdevdrv.c dlls/winepulse.drv/mmdevdrv.c +3 -0

No files found.
--- a/dlls/winepulse.drv/mmdevdrv.c
+++ b/dlls/winepulse.drv/mmdevdrv.c
@@ -573,6 +573,9 @@ static HRESULT pulse_test_connect(void)
            break;
    }

+    if (pa_context_get_state(pulse_ctx) != PA_CONTEXT_READY)
+        goto fail;
+
    TRACE("Test-connected to server %s with protocol version: %i.\n",
        pa_context_get_server(pulse_ctx),
        pa_context_get_server_protocol_version(pulse_ctx));