From 0faaee016583a46006b46c5c2c82440dad1aefb8 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 30 Sep 2010 23:15:47 -0400 Subject: [PATCH] Fix a spurious-call bug on epoll.c We were trying to check whether any events had really been notified on an fd before calling evmap_io_active on it, but instead we were checking for an event pointer, which was always true. In practice, this patch shouldn't change much, since epoll_wait shouldn't return an event unless there is actually an event going on. Spotted by an anonymous bug reporter on Sourceforge. Closes bug 3078425. --- epoll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/epoll.c b/epoll.c index 4f4a512c84..b574bf4d59 100644 --- a/epoll.c +++ b/epoll.c @@ -349,7 +349,7 @@ epoll_dispatch(struct event_base *base, struct timeval *tv) ev |= EV_WRITE; } - if (!events) + if (!ev) continue; evmap_io_active(base, events[i].data.fd, ev | EV_ET);