Oops when player thread keeps reading input unsuccessfully

Reading input fails after EOF and while the current command is
aborted.  Commands should detect that and fail.  If a command neglects
to do that in a loop, the loop can become infinite.  This is
especially bad after EOF, because then the client might not read
output anymore.  Output gets buffered until memory runs out.

Mitigate such bugs by counting how many calls have failed in a row,
oopsing on the 256th, and sleeping one minute from the 256th on.
This commit is contained in:
Markus Armbruster 2008-07-12 22:41:52 -04:00
parent f2294d67a4
commit 49c24d7b78
2 changed files with 14 additions and 4 deletions

View file

@ -78,6 +78,7 @@ struct player {
time_t curup; /* when last input was received */
int aborted; /* interrupt cookie received? */
int eof; /* EOF (cookie or real) received? */
int recvfail; /* #recvclient() failures */
int curid; /* for pr, cur. line's id, -1 none */
char *map; /* pointer to in-mem map */
char *bmap; /* pointer to in-mem bmap */