diff options
| author | Jeremy Allison <jra@samba.org> | 2011-02-28 11:23:53 -0800 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2011-02-28 21:10:08 +0100 |
| commit | 6e66208f6320949ba740e8d939fcfbb259ab0c67 (patch) | |
| tree | 22cb56f16306372cab8f45b5dea73430dafdc9be /lib/tevent/tevent_standard.c | |
| parent | 26321c6010dbd461e85111daf358e6c23a0b47b2 (diff) | |
| download | samba-6e66208f6320949ba740e8d939fcfbb259ab0c67.tar.gz samba-6e66208f6320949ba740e8d939fcfbb259ab0c67.tar.bz2 samba-6e66208f6320949ba740e8d939fcfbb259ab0c67.zip | |
Finish the range checking on all FD_SET calls for select().
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Feb 28 21:10:08 CET 2011 on sn-devel-104
Diffstat (limited to 'lib/tevent/tevent_standard.c')
| -rw-r--r-- | lib/tevent/tevent_standard.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/tevent/tevent_standard.c b/lib/tevent/tevent_standard.c index a68a03c2ac..35f7ded9b7 100644 --- a/lib/tevent/tevent_standard.c +++ b/lib/tevent/tevent_standard.c @@ -457,6 +457,10 @@ static int std_event_loop_select(struct std_event_context *std_ev, struct timeva /* setup any fd events */ for (fde = std_ev->ev->fd_events; fde; fde = fde->next) { + if (fde->fd < 0 || fde->fd >= FD_SETSIZE) { + std_ev->exit_code = EBADF; + return -1; + } if (fde->flags & TEVENT_FD_READ) { FD_SET(fde->fd, &r_fds); } |