Replace the per-iop input_timeout by per-function timeouts

[empserver] / src / lib / player / recvclient.c

diff --git a/src/lib/player/recvclient.c b/src/lib/player/recvclient.c
index 22a85496b3fdcb83b65c3d87475368934de7d18e..33c3cbbe53ef9bef39ae424215f77f15db74f739 100644 (file)
--- a/src/lib/player/recvclient.c
+++ b/src/lib/player/recvclient.c
@@ -28,7 +28,7 @@
  *
  *  Known contributors to this file:
  *     Dave Pare, 1986
- *     Markus Armbruster, 2006-2009
+ *     Markus Armbruster, 2006-2012
  *     Ron Koenderink, 2009
  */
 
@@ -36,22 +36,24 @@
 
 #include "empio.h"
 #include "journal.h"
+#include "optlist.h"
 #include "player.h"
 #include "prototypes.h"
 
 /*
  * Receive a line of input from the current player.
- * If the player's eof flag is set, return -1 without receiving input.
- * If the player's aborted flag is set, return -2 without receiving
+ * If the player's aborted flag is set, return -1 without receiving
  * input.
  * Else receive one line and store it in CMD[SIZE].
  * This may block for input, yielding the processor.  Flush buffered
  * output when blocking, to make sure player sees the prompt.
  * If the player's connection has the I/O error or EOF indicator set,
- * or the line is "ctld", set the player's eof and aborted flag and
+ * or the line is "aborted", set the player's aborted flag and return
+ * -1.
+ * If we block and time out, set the EOF indicator on the player's
+ * connection, set the player's aborted flag, and return -1.
+ * If the line is "ctld", set the player's eof and aborted flag and
  * return -1.
- * If the line is "aborted", set the player's aborted flag and return
- * -2.
  * Else return the length of the line.
  * Design bug: there is no way to indicate truncation of a long line.
  */
@@ -59,6 +61,7 @@ int
 recvclient(char *cmd, int size)
 {
     int count, res;
+    struct timeval timeout;
 
     count = -1;
     while (!player->aborted) {
@@ -67,7 +70,7 @@ recvclient(char *cmd, int size)
        if (count >= 0) {
            /* got it */
            if (strcmp(cmd, "ctld") == 0)
-               player->aborted = player->eof = 1;
+               player->aborted = player->got_ctld = 1;
            if (strcmp(cmd, "aborted") == 0)
                player->aborted = 1;
            journal_input(cmd);
@@ -83,22 +86,25 @@ recvclient(char *cmd, int size)
            ;
 
        /*
-        * If io_output_all() blocked and got unblocked by command
+        * If io_output() blocked and got unblocked by command
         * abortion, we must return without blocking in io_input().
         */
        if (player->aborted)
            break;
 
-       res = io_input(player->iop, 1);
+       timeout.tv_sec = minutes(max_idle);
+       timeout.tv_usec = 0;
+       res = io_input(player->iop, &timeout);
        if (res > 0)
            ;
        else if (res < 0)
-           player->aborted = player->eof = 1;
+           player->aborted = 1;
        else if (io_eof(player->iop))
-           player->aborted = player->eof = 1;
+           player->aborted = 1;
        else if (!player->aborted) {
            pr_flash(player, "idle connection terminated\n");
-           player->aborted = player->eof = 1;
+           io_set_eof(player->iop);
+           player->aborted = 1;
        }
     }
 
@@ -112,7 +118,7 @@ recvclient(char *cmd, int size)
            CANT_HAPPEN(player->recvfail == 256);
            empth_sleep(time(NULL) + 60);
        }
-       return player->eof ? -1 : -2;
+       return -1;
     }
 
     player->recvfail = 0;