diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 70185c17d..39bf01444 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,10 @@ +2001-10-26 Christopher Faylor + + * select.cc (MAKEready): Check for read_ready in loop since select_read + could set it. + (peek_socket): Check ready/write/except specifically since they could + have been set even prior to peek_socket call. + 2001-10-24 Christopher Faylor * shared_info.h (MOUNT_VERSION): Change to a smaller, still arbitrary diff --git a/winsup/cygwin/select.cc b/winsup/cygwin/select.cc index 8f1bab9ea..e07c23bb0 100644 --- a/winsup/cygwin/select.cc +++ b/winsup/cygwin/select.cc @@ -94,7 +94,8 @@ fhandler_##what::ready_for_read (int fd, DWORD howlong, int ignra) \ select_record me (this); \ me.fd = fd; \ (void) select_read (&me); \ - while (!peek_##what (&me, ignra) && howlong == INFINITE) \ + while (select_read (&me) && !me.read_ready && \ + !peek_##what (&me, ignra) && howlong == INFINITE) \ if (fd >= 0 && cygheap->fdtab.not_open (fd)) \ break; \ else if (WaitForSingleObject (signal_arrived, 10) == WAIT_OBJECT_0) \ @@ -1176,7 +1177,6 @@ peek_socket (select_record *me, int) WINSOCK_FD_ZERO (&ws_readfds); WINSOCK_FD_ZERO (&ws_writefds); WINSOCK_FD_ZERO (&ws_exceptfds); - int gotone = 0; HANDLE h; set_handle_or_return_if_not_open (h, me); @@ -1210,12 +1210,12 @@ peek_socket (select_record *me, int) } if (WINSOCK_FD_ISSET (h, &ws_readfds) || (me->read_selected && me->read_ready)) - gotone = me->read_ready = TRUE; + me->read_ready = TRUE; if (WINSOCK_FD_ISSET (h, &ws_writefds) || (me->write_selected && me->write_ready)) - gotone = me->write_ready = TRUE; + me->write_ready = TRUE; if (WINSOCK_FD_ISSET (h, &ws_exceptfds) || (me->except_selected && me->except_ready)) - gotone = me->except_ready = TRUE; - return gotone; + me->except_ready = TRUE; + return me->read_ready || me->write_ready || me->except_ready; } static int