* fhandler_socket.cc (fhandler_socket::close): Respond to signals while

looping, waiting for socket to close.  Superstitiously clear last error when
WSAEWOULDBLOCK.

2 files changed, 20 insertions, 6 deletions

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 48a40cd73..525167caf 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,9 @@
+2002-03-05  Christopher Faylor  <cgf@redhat.com>
+
+	* fhandler_socket.cc (fhandler_socket::close): Respond to signals while
+	looping, waiting for socket to close.  Superstitiously clear last error
+	when WSAEWOULDBLOCK.
+
 2002-03-05  Robert Collins  <rbtcollins@hotmail.com>
 
 	* cygserver_transport_pipes.cc (transport_layer_pipes::transport_layer_pipes): 
diff --git a/winsup/cygwin/fhandler_socket.cc b/winsup/cygwin/fhandler_socket.cc
index cf8ec1c24..e343c9021 100644
--- a/winsup/cygwin/fhandler_socket.cc
+++ b/winsup/cygwin/fhandler_socket.cc
@@ -352,13 +352,21 @@ fhandler_socket::close ()
   setsockopt (get_socket (), SOL_SOCKET, SO_LINGER,
 	      (const char *)&linger, sizeof linger);
 
-  while ((res = closesocket (get_socket ()))
-	 && WSAGetLastError () == WSAEWOULDBLOCK)
-    continue;
-  if (res)
+  while ((res = closesocket (get_socket ())) != 0)
     {
-      set_winsock_errno ();
-      res = -1;
+      if (WSAGetLastError () != WSAEWOULDBLOCK)
+	{
+	  set_winsock_errno ();
+	  res = -1;
+	  break;
+	}
+      if (WaitForSingleObject (signal_arrived, 10) == WAIT_OBJECT_0)
+	{
+	  set_errno (EINTR);
+	  res = -1;
+	  break;
+	}
+      WSASetLastError (0);
     }
 
   close_secret_event ();