Actually, the C standard only guarantees that atexit(3) returns non-zero
on error, so tweak previous to test "atexit(...) != 0" for portability.
"OK ok ok sorry backwards" deraadt@

atexit() returns -1 on failure

Remove three NULL-checks before free(). ok millert@

Do not assume that asprintf() clears the pointer on failure, which
is non-portable. Also add missing asprintf() return value checks.
OK deraadt@ guenther@ doug@

sys/types.h rather than sys/param.h, where applicable. avoid overreach.

Fix FILE * leak in error path if fprintf fails. Found by and OK gilles@

First pass at removing clauses 3 and 4 from NetBSD licenses.

Not sure what's more surprising: how long it took for NetBSD to
catch up to the rest of the BSDs (including UCB), or the amount of
code that NetBSD has claimed for itself without attributing to the
actual authors.

OK deraadt@

scrape $Id$ tags.
okay deraadt@, millert@, krw@

knf; otto ok

pid_t cleanup

atexit(3) can fail. handle error accordingly. deraadt ok
XXX libraries should not use atexit(3) from within, as program can terminate
with _exit.

pids should be pid_t, not int

save the pid as well, and only do the atexit in the same pid

add pidfile(3) - write a daemon pid file. ok deraadt@, millert@.

atexit() returns -1 on failure

Remove three NULL-checks before free(). ok millert@

Do not assume that asprintf() clears the pointer on failure, which
is non-portable. Also add missing asprintf() return value checks.
OK deraadt@ guenther@ doug@

sys/types.h rather than sys/param.h, where applicable. avoid overreach.

Fix FILE * leak in error path if fprintf fails. Found by and OK gilles@

First pass at removing clauses 3 and 4 from NetBSD licenses.

Not sure what's more surprising: how long it took for NetBSD to
catch up to the rest of the BSDs (including UCB), or the amount of
code that NetBSD has claimed for itself without attributing to the
actual authors.

OK deraadt@

scrape $Id$ tags.
okay deraadt@, millert@, krw@

knf; otto ok

pid_t cleanup

atexit(3) can fail. handle error accordingly. deraadt ok
XXX libraries should not use atexit(3) from within, as program can terminate
with _exit.

pids should be pid_t, not int

save the pid as well, and only do the atexit in the same pid

add pidfile(3) - write a daemon pid file. ok deraadt@, millert@.

Remove three NULL-checks before free(). ok millert@

Do not assume that asprintf() clears the pointer on failure, which
is non-portable. Also add missing asprintf() return value checks.
OK deraadt@ guenther@ doug@

sys/types.h rather than sys/param.h, where applicable. avoid overreach.

Fix FILE * leak in error path if fprintf fails. Found by and OK gilles@

First pass at removing clauses 3 and 4 from NetBSD licenses.

Not sure what's more surprising: how long it took for NetBSD to
catch up to the rest of the BSDs (including UCB), or the amount of
code that NetBSD has claimed for itself without attributing to the
actual authors.

OK deraadt@

scrape $Id$ tags.
okay deraadt@, millert@, krw@

knf; otto ok

pid_t cleanup

atexit(3) can fail. handle error accordingly. deraadt ok
XXX libraries should not use atexit(3) from within, as program can terminate
with _exit.

pids should be pid_t, not int

save the pid as well, and only do the atexit in the same pid

add pidfile(3) - write a daemon pid file. ok deraadt@, millert@.