From 2afa7f58828c67b33214e3ba10d71fa59909db97 Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Tue, 11 Mar 2003 13:05:36 +0000 Subject: [PATCH] 2003-03-11 Corinna Vinschen * fhandler_socket.cc (fhandler_socket::dup): On NT systems avoid using WinSock2 socket duplication methods. Add comment. 2003-03-11 Pierre Humblet * fhandler_socket.cc (fhandler_socket::fixup_after_fork): Set io_handle to INVALID_SOCKET in case of failure. (fhandler_socket::dup): Return 0 if the io_handle is valid. --- winsup/cygwin/ChangeLog | 11 +++++++++++ winsup/cygwin/fhandler_socket.cc | 17 +++++++++++++---- 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 587c46c17..6751d1888 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,14 @@ +2003-03-11 Corinna Vinschen + + * fhandler_socket.cc (fhandler_socket::dup): On NT systems avoid + using WinSock2 socket duplication methods. Add comment. + +2003-03-11 Pierre Humblet + + * fhandler_socket.cc (fhandler_socket::fixup_after_fork): + Set io_handle to INVALID_SOCKET in case of failure. + (fhandler_socket::dup): Return 0 if the io_handle is valid. + 2003-03-10 Corinna Vinschen * sec_acl.cc (setacl): Don't handle DELETE flag specially. diff --git a/winsup/cygwin/fhandler_socket.cc b/winsup/cygwin/fhandler_socket.cc index 01965bf67..c673df2af 100644 --- a/winsup/cygwin/fhandler_socket.cc +++ b/winsup/cygwin/fhandler_socket.cc @@ -344,6 +344,7 @@ fhandler_socket::fixup_after_fork (HANDLE parent) prot_info_ptr, 0, 0)) == INVALID_SOCKET) { debug_printf ("WSASocket error"); + set_io_handle ((HANDLE)INVALID_SOCKET); set_winsock_errno (); } else if (!new_sock && !winsock2_active) @@ -385,11 +386,19 @@ fhandler_socket::dup (fhandler_base *child) fhs->set_sun_path (get_sun_path ()); fhs->set_socket_type (get_socket_type ()); - fhs->fixup_before_fork_exec (GetCurrentProcessId ()); - if (winsock2_active) + /* Using WinSock2 methods for dup'ing sockets seem to collide + with user context switches under... some... conditions. So we + drop this for NT systems at all and return to the good ol' + DuplicateHandle way of life. This worked fine all the time on + NT anyway and it's even a bit faster. */ + if (!wincap.has_security ()) { - fhs->fixup_after_fork (hMainProc); - return 0; + fhs->fixup_before_fork_exec (GetCurrentProcessId ()); + if (winsock2_active) + { + fhs->fixup_after_fork (hMainProc); + return get_io_handle () == (HANDLE) INVALID_SOCKET; + } } return fhandler_base::dup (child); }