system/wine
system/wine
1
0
Fork 0
mirror of git://source.winehq.org/git/wine.git synced 2024-09-18 15:18:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

ws2_32: Check for valid socket in WSAGetOverlappedResult().

Browse source 
Signed-off-by: Paul Gofman <pgofman@codeweavers.com>
This commit is contained in:
Paul Gofman 2022-06-07 14:06:54 -05:00 committed by Alexandre Julliard
parent 94dc5714ac
commit 5a8ccc15a2
2 changed files with 19 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
dlls/ws2_32/socket.c
View file

@ -3626,6 +3626,12 @@ BOOL WINAPI WSAGetOverlappedResult( SOCKET s, LPWSAOVERLAPPED lpOverlapped,
return FALSE;
}
if (!socket_list_find( s ))
{
SetLastError( WSAENOTSOCK );
return FALSE;
}
status = lpOverlapped->Internal;
if (status == STATUS_PENDING)
{

13
dlls/ws2_32/tests/sock.c
View file

@ -11848,6 +11848,7 @@ static void test_WSAGetOverlappedResult(void)
NTSTATUS status;
unsigned int i;
SOCKET s;
HANDLE h;
BOOL ret;
static const NTSTATUS ranges[][2] =
@ -11862,8 +11863,20 @@ static void test_WSAGetOverlappedResult(void)
{0xd0070000, 0xd0080000},
};
WSASetLastError(0xdeadbeef);
ret = WSAGetOverlappedResult(0xdeadbeef, &overlapped, &size, FALSE, &flags);
ok(!ret, "got %d.\n", ret);
ok(WSAGetLastError() == WSAENOTSOCK, "got %u.\n", WSAGetLastError());
s = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
ret = DuplicateHandle(GetCurrentProcess(), (HANDLE)s, GetCurrentProcess(), &h, 0, FALSE, DUPLICATE_SAME_ACCESS);
ok(ret, "got %d.\n", ret);
ret = WSAGetOverlappedResult((SOCKET)h, &overlapped, &size, FALSE, &flags);
ok(!ret, "got %d.\n", ret);
ok(WSAGetLastError() == WSAENOTSOCK, "got %u.\n", WSAGetLastError());
CloseHandle(h);
for (i = 0; i < ARRAY_SIZE(ranges); ++i)
{
for (status = ranges[i][0]; status < ranges[i][1]; ++status)