[PATCH 0/1] MR9936: msado15: Fix WrapDSOandSession usage in connection_Open.

[Piotr Caban (＠piotr)]

Fixes regression introduced by 3bc4b5a9f.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9936

[PATCH 1/1] msado15: Fix WrapDSOandSession usage in connection_Open.

[Piotr Caban]

From: Piotr Caban <[e-mail]>

Fixes regression introduced by 3bc4b5a9f.
---
 dlls/msado15/connection.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/msado15/connection.c b/dlls/msado15/connection.c
index 50345c6f725..bba1a8fd8b9 100644
--- a/dlls/msado15/connection.c
+++ b/dlls/msado15/connection.c
@@ -424,10 +424,11 @@ static HRESULT WINAPI connection_Open( _Connection *iface, BSTR connect_str, BST
 
     hr = ADOConnectionConstruction15_WrapDSOandSession(
             &connection->ADOConnectionConstruction15_iface, (IUnknown *)dso, session );
-    if (hr == S_OK)
+    if (SUCCEEDED(hr))
     {
         connection->dso_initialized = TRUE;
         dso_initialized = FALSE;
+        hr = S_OK;
     }
 done:
     if (session) IUnknown_Release( session );
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/9936