JehanneOS/newlib
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

* poll.cc: Add bounds checking for file descriptors. Return POLLNVAL

Browse Source 
if fd is invalid. Return POLLERR for each valid fd if cygwin_select
        returned with error.
        include/sys/poll.h: Change POLLERR comment according to above change.
This commit is contained in:
Corinna Vinschen
2000-08-11 12:51:47 +00:00
parent 7b972f5da5
commit e82d75cc2b
3 changed files with 30 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
winsup/cygwin/poll.cc
View File

@ -25,7 +25,8 @@ poll (struct pollfd *fds, unsigned int nfds, int timeout)
FD_ZERO (&except_fds);
for (unsigned int i = 0; i < nfds; ++i)
if (!dtable.not_open (fds[i].fd))
if (fds[i].fd < FD_SETSIZE
&& !dtable.not_open (fds[i].fd))
{
FD_SET (fds[i].fd, &open_fds);
if (fds[i].events & POLLIN)
@ -41,24 +42,26 @@ poll (struct pollfd *fds, unsigned int nfds, int timeout)
int ret = cygwin_select (max_fd + 1, &read_fds, &write_fds, &except_fds,
timeout < 0 ? NULL : &tv);
if (ret >= 0)
for (unsigned int i = 0; i < nfds; ++i)
{
if (!FD_ISSET (fds[i].fd, &open_fds))
fds[i].revents = POLLNVAL;
else if (dtable.not_open(fds[i].fd))
fds[i].revents = POLLHUP;
else
{
fds[i].revents = 0;
if (FD_ISSET (fds[i].fd, &read_fds))
fds[i].revents |= POLLIN;
if (FD_ISSET (fds[i].fd, &write_fds))
fds[i].revents |= POLLOUT;
if (FD_ISSET (fds[i].fd, &except_fds))
fds[i].revents |= POLLPRI;
}
}
for (unsigned int i = 0; i < nfds; ++i)
{
if (fds[i].fd >= FD_SETSIZE
|| !FD_ISSET (fds[i].fd, &open_fds))
fds[i].revents = POLLNVAL;
else if (dtable.not_open(fds[i].fd))
fds[i].revents = POLLHUP;
else if (ret < 0)
fds[i].revents = POLLERR;
else
{
fds[i].revents = 0;
if (FD_ISSET (fds[i].fd, &read_fds))
fds[i].revents |= POLLIN;
if (FD_ISSET (fds[i].fd, &write_fds))
fds[i].revents |= POLLOUT;
if (FD_ISSET (fds[i].fd, &except_fds))
fds[i].revents |= POLLPRI;
}
}
return ret;
}