[PATCH 1/2] server: Retry socket connection on ECONNABORTED error.

[Paul Gofman]

From: Paul Gofman

---
 dlls/ws2_32/tests/sock.c | 29 ++++++++++++++++++++++++++++-
 server/sock.c            | 11 +++++++++++
 2 files changed, 39 insertions(+), 1 deletion(-)

diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c
index 58dcf9e7f4e..b9f1fec36bc 100644
--- a/dlls/ws2_32/tests/sock.c
+++ b/dlls/ws2_32/tests/sock.c
@@ -8320,7 +8320,6 @@ static void test_connect(void)

closesocket(connector);
     closesocket(acceptor);
-    closesocket(listener);

tcp_socketpair(&connector, &acceptor);

@@ -8380,6 +8379,34 @@ static void test_connect(void)

WSACloseEvent(overlapped.hEvent);
     closesocket(connector);
+
+    /* Test connect after previous connect attempt failure. */
+    connector = socket(AF_INET, SOCK_STREAM, 0);
+    ok(connector != INVALID_SOCKET, "failed to create socket, error %u\n", WSAGetLastError());
+
+    conaddress.sin_addr.s_addr = inet_addr("127.0.0.1");
+    conaddress.sin_port = htons(59243);
+    iret = connect(connector, (struct sockaddr *)&conaddress, sizeof(conaddress));
+    if (iret != -1)
+    {
+        skip("Connection succeeded, skipping failed connection test.\n");
+    }
+    else
+    {
+        ok(WSAGetLastError() == WSAECONNREFUSED, "got error %u\n", WSAGetLastError());
+        set_blocking( connector, FALSE );
+        iret = getsockname(listener, (struct sockaddr*)&address, &addrlen);
+        ok(!iret, "failed to get address, error %u\n", WSAGetLastError());
+
+        iret = connect(connector, (struct sockaddr *)&address, sizeof(address));
+        ok(iret == -1 && WSAGetLastError() == WSAEWOULDBLOCK, "unexpected iret %d, error %d.\n",
+                iret, WSAGetLastError());
+        acceptor = accept(listener, NULL, NULL);
+        ok(acceptor != INVALID_SOCKET, "could not accept socket error %d\n", WSAGetLastError());
+        closesocket(acceptor);
+    }
+    closesocket(connector);
+    closesocket(listener);
 }

static void test_AcceptEx(void)
diff --git a/server/sock.c b/server/sock.c
index a64cb22404e..610e0a94cc0 100644
--- a/server/sock.c
+++ b/server/sock.c
@@ -2602,6 +2602,17 @@ static void sock_ioctl( struct fd *fd, ioctl_code_t code, struct async *async )
             unix_addr.in.sin_addr.s_addr = htonl( INADDR_LOOPBACK );

ret = connect( unix_fd, &unix_addr.addr, unix_len );
+        if (ret < 0 && errno == ECONNABORTED)
+        {
+            /* On Linux with nonblocking socket if the previous connect() failed for any reason (including
+             * timeout), next connect will fail. If the error code was queried by getsockopt( SO_ERROR )
+             * the error code returned now is ECONNABORTED (otherwise that is the actual connect() failure
+             * error code). If we got here after previous connect attempt on the socket that means
+             * we already queried SO_ERROR in sock_error(), so retrying on ECONNABORTED only is
+             * sufficient. */
+            ret = connect( unix_fd, &unix_addr.addr, unix_len );
+        }
+
         if (ret < 0 && errno != EINPROGRESS)
         {
             set_error( sock_get_ntstatus( errno ) );