From: cgf <cgf>
Date: Thu, 8 Dec 2011 06:49:55 +0000 (+0000)
Subject: * select.cc (cygwin_select): Make sure that poll is called when appropriate.
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=17d65ae7439e330c953bc535b3ce61324a3896cf;p=pf3gnuchains%2Fpf3gnuchains4x.git

* select.cc (cygwin_select): Make sure that poll is called when appropriate.
---

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 329d163b3d..9ab00e78b3 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,5 +1,10 @@
 2011-12-08  Christopher Faylor  <me.cygwin2011@cgf.cx>
 
+	* select.cc (cygwin_select): Make sure that poll is called when
+	appropriate.
+
+2011-12-08  Christopher Faylor  <me.cygwin2011@cgf.cx>
+
 	* dll_init.cc (dll_dllcrt0): Don't try to initialize dll data if we're
 	dynamically loaded since fork() doesn't work in that scenario anyway.
 	(dll_dllcrt0_1): Don't accommodate dynamically loaded dlls.
diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc
index c60cd225fb..52714a0605 100644
--- a/winsup/cygwin/select.cc
+++ b/winsup/cygwin/select.cc
@@ -124,7 +124,6 @@ cygwin_select (int maxfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds,
 
   select_printf ("sel.always_ready %d", sel.always_ready);
 
-  int timeout = 0;
   /* Allocate some fd_set structures using the number of fds as a guide. */
   fd_set *r = allocfd_set (maxfds);
   fd_set *w = allocfd_set (maxfds);
@@ -133,34 +132,27 @@ cygwin_select (int maxfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds,
   int res = 1;
   /* Degenerate case.  No fds to wait for.  Just wait. */
   if (sel.start.next == NULL)
-    {
-      switch (cygWFMO (0, ms))
-	{
-	case WAIT_OBJECT_0:
-	  select_printf ("signal received");
-	  set_sig_errno (EINTR);
-	  return -1;
-	case WAIT_OBJECT_0 + 1:
-	  sel.destroy ();
-	  pthread::static_cancel_self ();
-	  /*NOTREACHED*/
-	default:
-	  break;
-	}
-      res = 0;
-    }
-  else if (sel.always_ready || ms == 0)
-    res = 0;
-  else if ((timeout = sel.wait (r, w, e, ms) < 0))
-    res = -1;	/* some kind of error */
-
-  if (res >= 0)
+    switch (cygWFMO (0, ms))
+      {
+      case WAIT_OBJECT_0:
+	select_printf ("signal received");
+	set_sig_errno (EINTR);
+	res = -1;
+      case WAIT_OBJECT_0 + 1:
+	sel.destroy ();
+	pthread::static_cancel_self ();
+	/*NOTREACHED*/
+      default:
+	res = 0;
+	break;
+      }
+  else if ((sel.always_ready || ms == 0)
+  	   || (res = sel.wait (r, w, e, ms)) == 0)
     {
       copyfd_set (readfds, r, maxfds);
       copyfd_set (writefds, w, maxfds);
       copyfd_set (exceptfds, e, maxfds);
-      if (res > 0)
-	res = sel.poll (readfds, writefds, exceptfds);
+      res = sel.poll (readfds, writefds, exceptfds);
     }
 
   syscall_printf ("%R = select (%d, %p, %p, %p, %p)", res, maxfds, readfds,