From 5f375bf464d99c5d6ce4126f5e815da1839105c5 Mon Sep 17 00:00:00 2001 From: Andre Noll Date: Fri, 15 Jan 2010 20:21:13 +0100 Subject: [PATCH] sched: Reset the descriptor sets on timeouts. This is certainly not neccessary on linux, but it does not hurt and is clearly more portable. --- sched.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/sched.c b/sched.c index e58483e9..5c185715 100644 --- a/sched.c +++ b/sched.c @@ -120,6 +120,16 @@ again: ret = s->select_function(s->max_fileno + 1, &s->rfds, &s->wfds, &s->timeout); if (ret < 0) return ret; + if (ret == 0) { + /* + * APUE: Be careful not to check the descriptor sets on return + * unless the return value is greater than zero. The return + * state of the descriptor sets is implementation dependent if + * either a signal is caught or the timer expires. + */ + FD_ZERO(&s->rfds); + FD_ZERO(&s->wfds); + } gettimeofday(now, NULL); sched_post_select(s); if (list_empty(&pre_select_list) && list_empty(&post_select_list)) -- 2.30.2