- djm@cvs.openbsd.org 2012/04/11 13:16:19

     [channels.c channels.h clientloop.c serverloop.c]
     don't spin in accept() when out of fds (ENFILE/ENFILE) - back off for a
     while; ok deraadt@ markus@

1 files changed, 10 insertions, 4 deletions

diff --git a/serverloop.c b/serverloop.c
index 19b84ff27..50be16b7c 100644
--- a/serverloop.c
+++ b/serverloop.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: serverloop.c,v 1.160 2011/05/15 08:09:01 djm Exp $ */
+/* $OpenBSD: serverloop.c,v 1.161 2012/04/11 13:16:19 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -281,9 +281,18 @@ wait_until_can_do_something(fd_set **readsetp, fd_set **writesetp, int *maxfdp,
 {
         struct timeval tv, *tvp;
         int ret;
+        time_t minwait_secs;
         int client_alive_scheduled = 0;
         int program_alive_scheduled = 0;
 
+        /* Allocate and update select() masks for channel descriptors. */
+        channel_prepare_select(readsetp, writesetp, maxfdp, nallocp,
+            &minwait_secs, 0);
+
+        if (minwait_secs != 0)
+                max_time_milliseconds = MIN(max_time_milliseconds,
+                    (u_int)minwait_secs * 1000);
+
         /*
          * if using client_alive, set the max timeout accordingly,
          * and indicate that this particular timeout was for client
@@ -298,9 +307,6 @@ wait_until_can_do_something(fd_set **readsetp, fd_set **writesetp, int *maxfdp,
                 max_time_milliseconds = options.client_alive_interval * 1000;
         }
 
-        /* Allocate and update select() masks for channel descriptors. */
-        channel_prepare_select(readsetp, writesetp, maxfdp, nallocp, 0);
-
         if (compat20) {
 #if 0
                 /* wrong: bad condition XXX */