ws2_32: Disallow NULL in socket lookups.

NULL means an unused slot, not a valid socket. Signed-off-by: Akihiro Sagawa <sagawa.aki@gmail.com> Signed-off-by: Zebediah Figura <zfigura@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

ws2_32: Disallow NULL in socket lookups.
ee2e5c02 · Akihiro Sagawa · Alexandre Julliard · ffe20add · ee2e5c02
Commit ee2e5c02 authored Aug 24, 2021 by Akihiro Sagawa Committed by Alexandre Julliard Aug 24, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

socket.c dlls/ws2_32/socket.c +4 -0

No files found.
--- a/dlls/ws2_32/socket.c
+++ b/dlls/ws2_32/socket.c
@@ -379,6 +379,8 @@ static BOOL socket_list_find( SOCKET socket )
 {
    unsigned int i;

+    if (!socket) return FALSE;
+
    EnterCriticalSection( &cs_socket_list );
    for (i = 0; i < socket_list_size; ++i)
    {
@@ -397,6 +399,8 @@ static BOOL socket_list_remove( SOCKET socket )
 {
    unsigned int i;

+    if (!socket) return FALSE;
+
    EnterCriticalSection(&cs_socket_list);
    for (i = 0; i < socket_list_size; ++i)
    {