From 2beeabcd0723ef5d39d02aca9cc8ee92588eb8af Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Thu, 27 Jun 2013 12:36:44 -0400 Subject: [PATCH] Expect EWOULDBLOCK from a non-blocking connect() call only on Windows. On Unix-ish platforms, EWOULDBLOCK may be the same as EAGAIN, which is *not* a success return, at least not on Linux. We need to treat it as a failure to avoid giving a misleading error message. Per the Single Unix Spec, only EINPROGRESS and EINTR returns indicate that the connection attempt is in progress. On Windows, on the other hand, EWOULDBLOCK (WSAEWOULDBLOCK) is the expected case. We must accept EINPROGRESS as well because Cygwin will return that, and it doesn't seem worth distinguishing Cygwin from native Windows here. It's not very clear whether EINTR can occur on Windows, but let's leave that part of the logic alone in the absence of concrete trouble reports. Also, remove the test for errno == 0, effectively reverting commit da9501bddb42222dc33c031b1db6ce2133bcee7b, which AFAICS was just a thinko; or at best it might have been a workaround for a platform-specific bug, which we can hope is gone now thirteen years later. In any case, since libpq makes no effort to reset errno to zero before calling connect(), it seems unlikely that that test has ever reliably done anything useful. Andres Freund and Tom Lane --- src/interfaces/libpq/fe-connect.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c index 7db780f5c6..0f0dd651eb 100644 --- a/src/interfaces/libpq/fe-connect.c +++ b/src/interfaces/libpq/fe-connect.c @@ -1779,9 +1779,10 @@ keep_going: /* We will come back to here until there is addr_cur->ai_addrlen) < 0) { if (SOCK_ERRNO == EINPROGRESS || +#ifdef WIN32 SOCK_ERRNO == EWOULDBLOCK || - SOCK_ERRNO == EINTR || - SOCK_ERRNO == 0) +#endif + SOCK_ERRNO == EINTR) { /* * This is fine - we're in non-blocking mode, and -- 2.40.0