[PATCH v2 3/5] ws2_32: Return TRUE from WSAGetOverlappedResult() if the NT status denotes success.
Zebediah Figura
z.figura12 at gmail.com
Thu May 6 17:54:07 CDT 2021
Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
dlls/ws2_32/socket.c | 2 +-
dlls/ws2_32/tests/sock.c | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c
index 273a6b159f4..3a8bfa0ce41 100644
--- 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 );
}
diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c
index a84f05becbf..eeba272076b 100644
--- 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 */
--
2.30.2
More information about the wine-devel
mailing list