upstream: When poll(2) returns -1, for some error conditions

pfd[].revents is not cleared.  There are subtle errors in various programs.
In this particular case, the program should error out. ok djm millert

OpenBSD-Commit-ID: 00f839b16861f7fb2adcf122e95e8a82fa6a375c
This commit is contained in:
deraadt@openbsd.org 2022-01-21 00:53:40 +00:00 committed by Damien Miller
parent e204b34337
commit d069b020a0

View File

@ -1,4 +1,4 @@
/* $OpenBSD: ssh-keyscan.c,v 1.144 2021/12/02 23:45:36 djm Exp $ */ /* $OpenBSD: ssh-keyscan.c,v 1.145 2022/01/21 00:53:40 deraadt Exp $ */
/* /*
* Copyright 1995, 1996 by David Mazieres <dm@lcs.mit.edu>. * Copyright 1995, 1996 by David Mazieres <dm@lcs.mit.edu>.
* *
@ -592,9 +592,11 @@ conloop(void)
else else
timespecclear(&seltime); timespecclear(&seltime);
while (ppoll(read_wait, maxfd, &seltime, NULL) == -1 && while (ppoll(read_wait, maxfd, &seltime, NULL) == -1) {
(errno == EAGAIN || errno == EINTR || errno == EWOULDBLOCK)) if (errno == EAGAIN || errno == EINTR || errno == EWOULDBLOCK)
; continue;
error("poll error");
}
for (i = 0; i < maxfd; i++) { for (i = 0; i < maxfd; i++) {
if (read_wait[i].revents & (POLLHUP|POLLERR|POLLNVAL)) if (read_wait[i].revents & (POLLHUP|POLLERR|POLLNVAL))