diff options
| author | Volker Lendecke <vl@samba.org> | 2010-10-06 18:47:08 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2010-10-06 23:40:24 +0000 |
| commit | 1c9b85667070ee3292862d007f03b33b10713edd (patch) | |
| tree | 955abe0a9ba462f920bf4cc3083d5a3b06dc04e6 /lib/tevent/tevent_standard.c | |
| parent | 0cec0f0912bc0cc15715fd68483f28e5b68e4439 (diff) | |
| download | samba-1c9b85667070ee3292862d007f03b33b10713edd.tar.gz | |
tevent: Only call handlers that ask for events
The s3 libsmbclient puts two different handlers on a single fd for readability
and writability. With select this works fine, however without this patch we
unconditionally call the write handler even if the socket is only readable. We
should have called the read handler instead.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Oct 6 23:40:24 UTC 2010 on sn-devel-104
Diffstat (limited to 'lib/tevent/tevent_standard.c')
| -rw-r--r-- | lib/tevent/tevent_standard.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/tevent/tevent_standard.c b/lib/tevent/tevent_standard.c index 1c8ed0e6ac3..a68a03c2acb 100644 --- a/lib/tevent/tevent_standard.c +++ b/lib/tevent/tevent_standard.c @@ -505,7 +505,7 @@ static int std_event_loop_select(struct std_event_context *std_ev, struct timeva if (FD_ISSET(fde->fd, &r_fds)) flags |= TEVENT_FD_READ; if (FD_ISSET(fde->fd, &w_fds)) flags |= TEVENT_FD_WRITE; - if (flags) { + if (flags & fde->flags) { fde->handler(std_ev->ev, fde, flags, fde->private_data); break; } |