ws2_32: Return TRUE from WSAGetOverlappedResult() if the NT status denotes success.

Signed-off-by: Zebediah Figura <z.figura12@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

ws2_32: Return TRUE from WSAGetOverlappedResult() if the NT status denotes success.
0abc3245 · Zebediah Figura · Alexandre Julliard · 5a8dc6cd · 0abc3245 · 0abc3245
Commit 0abc3245 authored May 06, 2021 by Zebediah Figura Committed by Alexandre Julliard May 07, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

socket.c dlls/ws2_32/socket.c +1 -1

sock.c dlls/ws2_32/tests/sock.c +1 -2

No files found.
--- a/dlls/ws2_32/socket.c
+++ b/dlls/ws2_32/socket.c
@@ -5738,7 +5738,7 @@ BOOL WINAPI WSAGetOverlappedResult( SOCKET s, LPWSAOVERLAPPED lpOverlapped,
        *lpdwFlags = lpOverlapped->u.s.Offset;
    SetLastError( NtStatusToWSAError(status) );
-    return !status;
+    return NT_SUCCESS( status );
 }

--- a/dlls/ws2_32/tests/sock.c
+++ b/dlls/ws2_32/tests/sock.c
@@ -8690,8 +8690,7 @@ static void test_WSAGetOverlappedResult(void)
            overlapped.Internal = status;
            WSASetLastError(0xdeadbeef);
            ret = WSAGetOverlappedResult(s, &overlapped, &size, FALSE, &flags);
-            todo_wine_if (expect_ret && status)
+            ok(ret == expect_ret, "status %#x: expected %d, got %d\n", status, expect_ret, ret);
-                ok(ret == expect_ret, "status %#x: expected %d, got %d\n", status, expect_ret, ret);
            if (ret)
            {
                ok(WSAGetLastError() == expect /* >= win10 1809 */