"Todd C. Miller" <[email protected]> writes:

> On Mon, 04 Jul 2016 12:43:12 +0200, Jeremie Courreges-Anglas wrote:
>
>> Showing this to a wider audience,
>> 
>> This was initially prompted by a diff to replace calloc with
>> reallocarray.  As noted by guenther, there is a problem with the
>> comparisons between width, rpcsock and pingsock; this, plus the fd_set
>> allocation inconvenience would be easily fixed by moving from select to
>> poll.
>> 
>> I do not use YP so tests are more than welcome.
>
> You need to check POLLHUP as well as POLLIN.

The two sockets we check for are SOCK_DGRAM sockets, I assumed that we
can't get POLLHUP on such sockets.  Wrong assumption?

> It would also be
> good to handle EINTR.

ack

-- 
jca | PGP: 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to