From: Mark Mitchell <mark@codesourcery.com>
Date: Sat, 14 Jan 2006 05:13:21 +0000 (+0000)
Subject: 	* event-loop.c (gdb_select): Detect file descriptors that have
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=6da866186a9826152254528a505fec13adfcfc6e;p=binutils-gdb.git

	* event-loop.c (gdb_select): Detect file descriptors that have
	been closed.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1a73ed9c884..b31d144ddf3 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2006-01-13  Mark Mitchell  <mark@codesourcery.com>
+
+	* event-loop.c (gdb_select): Detect file descriptors that have
+	been closed.
+
 2006-01-13  Mark Kettenis  <kettenis@gnu.org>
 
 	* hppabsd-tdep.c (hppabsd_init_abi): Set long_double_bit to 64,
diff --git a/gdb/event-loop.c b/gdb/event-loop.c
index e336b339424..4f859b115c6 100644
--- a/gdb/event-loop.c
+++ b/gdb/event-loop.c
@@ -749,22 +749,39 @@ gdb_select (int n, fd_set *readfds, fd_set *writefds, fd_set *exceptfds,
   int fd;
   int num_ready;
 
+  num_ready = 0;
   num_handles = 0;
   for (fd = 0; fd < n; ++fd)
     {
-      /* EXCEPTFDS is silently ignored.  GDB always sets GDB_EXCEPTION
-	 when calling add_file_handler, but there is no natural analog
-	 under Windows.  */
       /* There is no support yet for WRITEFDS.  At present, this isn't
 	 used by GDB -- but we do not want to silently ignore WRITEFDS
 	 if something starts using it.  */
       gdb_assert (!FD_ISSET (fd, writefds));
-      if (FD_ISSET (fd, readfds))
+      if (!FD_ISSET (fd, readfds) 
+	  && !FD_ISSET (fd, exceptfds))
+	continue;
+      h = (HANDLE) _get_osfhandle (fd);
+      if (h == INVALID_HANDLE_VALUE)
 	{
-	  gdb_assert (num_handles < MAXIMUM_WAIT_OBJECTS);
-	  handles[num_handles++] = (HANDLE) _get_osfhandle (fd);
+	  /* If the underlying handle is INVALID_HANDLE_VALUE, then
+	     this descriptor is no more.  */
+	  if (FD_ISSET (fd, exceptfds))
+	    ++num_ready;
+	  continue;
 	}
+      /* The only exceptional condition we recognize is a closed file
+	 descriptor.  Since we have already checked for that
+	 condition, clear the exceptional bit for this descriptor.  */
+      FD_CLR (fd, exceptfds);
+      if (FD_ISSET (fd, readfds))
+      {
+	gdb_assert (num_handles < MAXIMUM_WAIT_OBJECTS);
+	handles[num_handles++] = h;
+      }
     }
+  /* If we don't need to wait for any handles, we are done.  */
+  if (!num_handles)
+    return num_ready;
   event = WaitForMultipleObjects (num_handles,
 				  handles,
 				  FALSE,
@@ -779,10 +796,10 @@ gdb_select (int n, fd_set *readfds, fd_set *writefds, fd_set *exceptfds,
   if (event == WAIT_FAILED)
     return -1;
   if (event == WAIT_TIMEOUT)
-    return 0;
+    return num_ready;
   /* Run through the READFDS, clearing bits corresponding to descriptors
      for which input is unavailable.  */
-  num_ready = num_handles; 
+  num_ready += num_handles; 
   h = handles[event - WAIT_OBJECT_0];
   for (fd = 0; fd < n; ++fd)
     {
@@ -798,10 +815,6 @@ gdb_select (int n, fd_set *readfds, fd_set *writefds, fd_set *exceptfds,
 	  --num_ready;
 	}
     }
-  /* We never report any descriptors available for writing or with
-     exceptional conditions.  */ 
-  FD_ZERO (writefds);
-  FD_ZERO (exceptfds);
 
   return num_ready;
 #else