server: Check socket flags after all sockets got processed.

In rare cases we get an event during the second iteration of the loop, changing the first element. This would lead to an assertion later. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55054

server: Check socket flags after all sockets got processed.
13928e08 · Fabian Maurer · Alexandre Julliard · c241468f · 13928e08
Commit 13928e08 authored Nov 30, 2023 by Fabian Maurer Committed by Alexandre Julliard Dec 01, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

sock.c server/sock.c +3 -0

No files found.
--- a/server/sock.c
+++ b/server/sock.c
@@ -3509,7 +3509,10 @@ static void poll_socket( struct sock *poll_sock, struct async *async, int exclus
            req->sockets[i].flags |= AFD_POLL_CONNECT_ERR;
            req->sockets[i].status = sock_get_ntstatus( sock->errors[AFD_POLL_BIT_CONNECT_ERR] );
        }
+    }

+    for (i = 0; i < count; ++i)
+    {
        if (req->sockets[i].flags)
            signaled = TRUE;
    }