From: Jeremy Allison Date: Fri, 3 Jun 2011 19:55:19 +0000 (-0700) Subject: Add the same fix to the S3 event backend as the previous commit added to the tevent... X-Git-Url: http://git.samba.org/?a=commitdiff_plain;h=3c9b3b2befc524f21c59f46ea9be1602b4b1bfe8;p=metze%2Fsamba%2Fwip.git Add the same fix to the S3 event backend as the previous commit added to the tevent poll backend. Metze please check ! Autobuild-User: Jeremy Allison Autobuild-Date: Sat Jun 4 00:27:37 CEST 2011 on sn-devel-104 --- diff --git a/source3/lib/events.c b/source3/lib/events.c index 9ff14880ecdb..fbe3db942b4d 100644 --- a/source3/lib/events.c +++ b/source3/lib/events.c @@ -258,7 +258,20 @@ bool run_events_poll(struct tevent_context *ev, int pollrtn, return false; } - if (pfd->revents & (POLLIN|POLLHUP|POLLERR)) { + if (pfd->revents & (POLLHUP|POLLERR)) { + /* If we only wait for EVENT_FD_WRITE, we + should not tell the event handler about it, + and remove the writable flag, as we only + report errors when waiting for read events + to match the select behavior. */ + if (!(fde->flags & EVENT_FD_READ)) { + EVENT_FD_NOT_WRITEABLE(fde); + continue; + } + flags |= EVENT_FD_READ; + } + + if (pfd->revents & POLLIN) { flags |= EVENT_FD_READ; } if (pfd->revents & POLLOUT) {