diff options
author | Darren Tucker <dtucker@zip.com.au> | 2008-08-30 07:32:37 +1000 |
---|---|---|
committer | Darren Tucker <dtucker@zip.com.au> | 2008-08-30 07:32:37 +1000 |
commit | 661f63b7d2a89d91c29691ba8d01b825600e6789 (patch) | |
tree | 3c4f42cc328145aa7cf05fb0771d9bdb24653859 /openbsd-compat | |
parent | 212f0b08790c415648db250797b1b060b62a015d (diff) |
- (dtucker) [openbsd-compat/bsd-poll.c] correctly check for number of FDs
larger than FD_SETSIZE (OpenSSH only ever uses poll with one fd). Patch
from Nicholas Marriott.
Diffstat (limited to 'openbsd-compat')
-rw-r--r-- | openbsd-compat/bsd-poll.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/openbsd-compat/bsd-poll.c b/openbsd-compat/bsd-poll.c index 284db3a1f..f899d7a24 100644 --- a/openbsd-compat/bsd-poll.c +++ b/openbsd-compat/bsd-poll.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $Id: bsd-poll.c,v 1.3 2008/04/04 05:16:36 djm Exp $ */ | 1 | /* $Id: bsd-poll.c,v 1.4 2008/08/29 21:32:38 dtucker Exp $ */ |
2 | 2 | ||
3 | /* | 3 | /* |
4 | * Copyright (c) 2004, 2005, 2007 Darren Tucker (dtucker at zip com au). | 4 | * Copyright (c) 2004, 2005, 2007 Darren Tucker (dtucker at zip com au). |
@@ -46,11 +46,12 @@ poll(struct pollfd *fds, nfds_t nfds, int timeout) | |||
46 | struct timeval tv, *tvp = NULL; | 46 | struct timeval tv, *tvp = NULL; |
47 | 47 | ||
48 | for (i = 0; i < nfds; i++) { | 48 | for (i = 0; i < nfds; i++) { |
49 | fd = fds[i].fd; | ||
49 | if (fd >= FD_SETSIZE) { | 50 | if (fd >= FD_SETSIZE) { |
50 | errno = EINVAL; | 51 | errno = EINVAL; |
51 | return -1; | 52 | return -1; |
52 | } | 53 | } |
53 | maxfd = MAX(maxfd, fds[i].fd); | 54 | maxfd = MAX(maxfd, fd); |
54 | } | 55 | } |
55 | 56 | ||
56 | nmemb = howmany(maxfd + 1 , NFDBITS); | 57 | nmemb = howmany(maxfd + 1 , NFDBITS); |