Paul Gofman : server: Ignore some ICMP-originated socket errors for connectionless sockets.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 4d56a33ab0b8f6f808a3a507e76c29b905dd6d83
URL:    https://gitlab.winehq.org/wine/wine/-/commit/4d56a33ab0b8f6f808a3a507e76c29b...

Author: Paul Gofman 
Date:   Thu Mar 14 20:05:31 2024 -0600

server: Ignore some ICMP-originated socket errors for connectionless sockets.

---

server/sock.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/server/sock.c b/server/sock.c
index 3ad1ce39194..8a95caded8b 100644
--- a/server/sock.c
+++ b/server/sock.c
@@ -813,7 +813,7 @@ static void post_sock_messages( struct sock *sock )
     }
 }

-static inline int sock_error( struct sock *sock )
+static inline int sock_error( struct sock *sock, int *poll_event )
 {
     int error = 0;
     socklen_t len = sizeof(error);
@@ -839,8 +839,14 @@ static inline int sock_error( struct sock *sock )
             error = sock->errors[AFD_POLL_BIT_ACCEPT];
         break;

-    case SOCK_CONNECTED:
     case SOCK_CONNECTIONLESS:
+        if (error == ENETUNREACH || error == EHOSTUNREACH || error == ECONNRESET)
+        {
+            if (poll_event) *poll_event &= ~POLLERR;
+            return 0;
+        }
+        /* fallthrough */
+    case SOCK_CONNECTED:
         if (error == ECONNRESET || error == EPIPE)
         {
             sock->reset = 1;
@@ -1346,7 +1352,7 @@ static void sock_poll_event( struct fd *fd, int event )
         fprintf(stderr, "socket %p select event: %x\n", sock, event);

if (event & (POLLERR | POLLHUP))
-        error = sock_error( sock );
+        error = sock_error( sock, &event );

switch (sock->state)
     {
@@ -3115,7 +3121,7 @@ static void sock_ioctl( struct fd *fd, ioctl_code_t code, struct async *async )
             return;
         }

-        error = sock_error( sock );
+        error = sock_error( sock, NULL );
         if (!error)
         {
             for (i = 0; i < ARRAY_SIZE( sock->errors ); ++i)