https://bugzilla.mindrot.org/show_bug.cgi?id=3416
--- Comment #5 from Darren Tucker <[email protected]> --- (In reply to Damien Miller from comment #4) > Looks good to me. Although unrelated, I think the POLLIN case should > check (revents & (POLLIN|POLLHUP)) as this can be returned too (not > sure whether without POLLIN too). I don't follow: it's not checking revents that will be returned from poll(), it's checking events that were passed to poll(). According to the man page POLLHUP is meaningless and ignored in events and only meaningful in revents (which we're not checking). > Also maybe both POLLIN and POLLOUT cases should check POLLERR in > revents. I saw this in bug #3405 Good point, I'll do that as a separate patch. -- You are receiving this mail because: You are watching someone on the CC list of the bug. You are watching the assignee of the bug. _______________________________________________ openssh-bugs mailing list [email protected] https://lists.mindrot.org/mailman/listinfo/openssh-bugs
