Huw Davies : wineoss: Move is_started to the unixlib.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 4b469fe157244821319489070d2401ffdb37eb94
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=4b469fe157244821319489070d2401ffdb37eb94

Author: Huw Davies <huw at codeweavers.com>
Date:   Fri Apr 15 21:40:42 2022 +0100

wineoss: Move is_started to the unixlib.

Signed-off-by: Huw Davies <huw at codeweavers.com>
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/wineoss.drv/mmdevdrv.c |  8 ++++----
 dlls/wineoss.drv/oss.c      | 11 +++++++++++
 dlls/wineoss.drv/unixlib.h  |  7 +++++++
 3 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/dlls/wineoss.drv/mmdevdrv.c b/dlls/wineoss.drv/mmdevdrv.c
index c921b4975c9..cdd9b09bff7 100644
--- a/dlls/wineoss.drv/mmdevdrv.c
+++ b/dlls/wineoss.drv/mmdevdrv.c
@@ -1596,6 +1596,7 @@ static HRESULT WINAPI AudioSessionControl_GetState(IAudioSessionControl2 *iface,
         AudioSessionState *state)
 {
     AudioSessionWrapper *This = impl_from_IAudioSessionControl2(iface);
+    struct is_started_params params;
     ACImpl *client;
 
     TRACE("(%p)->(%p)\n", This, state);
@@ -1612,14 +1613,13 @@ static HRESULT WINAPI AudioSessionControl_GetState(IAudioSessionControl2 *iface,
     }
 
     LIST_FOR_EACH_ENTRY(client, &This->session->clients, ACImpl, entry){
-        oss_lock(client->stream);
-        if(client->stream->playing){
+        params.stream = client->stream;
+        OSS_CALL(is_started, &params);
+        if(params.result == S_OK){
             *state = AudioSessionStateActive;
-            oss_unlock(client->stream);
             LeaveCriticalSection(&g_sessions_lock);
             return S_OK;
         }
-        oss_unlock(client->stream);
     }
 
     LeaveCriticalSection(&g_sessions_lock);
diff --git a/dlls/wineoss.drv/oss.c b/dlls/wineoss.drv/oss.c
index a39b0bc4f66..79f2e994d89 100644
--- a/dlls/wineoss.drv/oss.c
+++ b/dlls/wineoss.drv/oss.c
@@ -1336,6 +1336,16 @@ static NTSTATUS set_event_handle(void *args)
     return oss_unlock_result(stream, &params->result, S_OK);
 }
 
+static NTSTATUS is_started(void *args)
+{
+    struct is_started_params *params = args;
+    struct oss_stream *stream = params->stream;
+
+    oss_lock(stream);
+
+    return oss_unlock_result(stream, &params->result, stream->playing ? S_OK : S_FALSE);
+}
+
 unixlib_entry_t __wine_unix_call_funcs[] =
 {
     test_connect,
@@ -1359,4 +1369,5 @@ unixlib_entry_t __wine_unix_call_funcs[] =
     get_frequency,
     get_position,
     set_event_handle,
+    is_started,
 };
diff --git a/dlls/wineoss.drv/unixlib.h b/dlls/wineoss.drv/unixlib.h
index 982b11ec44f..ed93f0836b1 100644
--- a/dlls/wineoss.drv/unixlib.h
+++ b/dlls/wineoss.drv/unixlib.h
@@ -215,6 +215,12 @@ struct set_event_handle_params
     HRESULT result;
 };
 
+struct is_started_params
+{
+    struct oss_stream *stream;
+    HRESULT result;
+};
+
 enum oss_funcs
 {
     oss_test_connect,
@@ -238,6 +244,7 @@ enum oss_funcs
     oss_get_frequency,
     oss_get_position,
     oss_set_event_handle,
+    oss_is_started,
 };
 
 extern unixlib_handle_t oss_handle;