When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

Pull in <sys/select.h> for fd_set

ok deraadt@

ANSIfy hunt(6)

Note: casted 2 args to struct sockaddr * on list.c to shut off compiler
warnings, and also changed an int len to socklen_t since recvfrom(2) receives
the last argument as the latter.

I'm running out of credits, but this was once again with great help and OK from
tb@

Headers cleanup and sorting

OK tb@

Userland reallocarray() audit.

Avoid potential integer overflow in the size argument of malloc() and
realloc() by using reallocarray() to avoid unchecked multiplication.

ok deraadt@

Some more simple malloc(n * m) -> calloc(n, m) conversions.

ok deraadt@

my licence now permits modification. plus some tiny style cleanup

Quiet alpha warnings, principally with correct includes

re-initialize timeout before each invocation of select()

isplit complicated probe_drivers() out to own file. reviewed by pjanzen@

Pull in <sys/select.h> for fd_set

ok deraadt@

ANSIfy hunt(6)

Note: casted 2 args to struct sockaddr * on list.c to shut off compiler
warnings, and also changed an int len to socklen_t since recvfrom(2) receives
the last argument as the latter.

I'm running out of credits, but this was once again with great help and OK from
tb@

Headers cleanup and sorting

OK tb@

Userland reallocarray() audit.

Avoid potential integer overflow in the size argument of malloc() and
realloc() by using reallocarray() to avoid unchecked multiplication.

ok deraadt@

Some more simple malloc(n * m) -> calloc(n, m) conversions.

ok deraadt@

my licence now permits modification. plus some tiny style cleanup

Quiet alpha warnings, principally with correct includes

re-initialize timeout before each invocation of select()

isplit complicated probe_drivers() out to own file. reviewed by pjanzen@