Make struct imsgev static like in the other routing daemons. With this
dvmrpd should compile with -fno-common

Kill dvrmpd_process and log_procnames. Unneeded abstraction for this
simple daemon.

work on making log.c similar in all daemons:

move daemon-local functions into new logmsg.c, and reduce
the (mostly whitespace) differences so that log.c's can be diffed easily.

ok claudio@

Simplify shutdown process.

On shutdown, there's no need to use kill(2) to kill the child
processes. Just closing the IPC sockets will make the children receive
an EOF, break out from the event loop

Simplify shutdown process.

On shutdown, there's no need to use kill(2) to kill the child
processes. Just closing the IPC sockets will make the children receive
an EOF, break out from the event loop and then exit.

Tha advantages of this "pipe teardown" are:
* simpler code;
* no need to pledge "proc" in the parent process;
* removal of a (hard to trigger) PID reuse race condition.

"reads good" claudio@

show more ...

Close connections when msgbuf_write() returns 0.

Pulled from ospfd. Original author: claudio@

ok rzalamena@ benno@ claudio@

Don't fatal if the imsg pipe is closed, this is often triggered in the
parent and hides the real cause of the termination.

Pulled from ospfd. Original author: claudio@

ok rzalamena@ benno@ claudio@

Remove setproctitle() for the parent process. Because rc.d(8) uses process
titles (including flags) to distinguish between daemons, this makes it
possible to manage multiple copies of a daemon using

Remove setproctitle() for the parent process. Because rc.d(8) uses process
titles (including flags) to distinguish between daemons, this makes it
possible to manage multiple copies of a daemon using the normal infrastructure
by symlinking rc.d scripts to a new name. ok jung@ ajacoutot@, smtpd ok gilles@

show more ...

EAGAIN handling for imsg_read. OK henning@ benno@

More SOCK_NONBLOCK adjustments similar to the other daemons.

Kill another session_socket_blockmode() ... 6 or so to go.

Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible. Annotate <sys/param.h> lines with their current reasons. Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_N

Replace <sys/param.h> with <limits.h> and other less dirty headers where
possible. Annotate <sys/param.h> lines with their current reasons. Switch
to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_NAME_MAX, etc. Change
MIN() and MAX() to local definitions of MINIMUM() and MAXIMUM() where
sensible to avoid pulling in the pollution. These are the files confirmed
through binary verification.
ok guenther, millert, doug (helped with the verification protocol)

show more ...

msgbuf_write() <= 0 for a few more daemons.

ok claudio@

deal with msgbuf_write EAGAIN, ok gilles benno

Decouple log_verbose() from log_init() so the verbose flag stays set with
"-v" (previously only "-vd" worked). Similar to recent ospfd commit.
ok claudio@

...and yet another typo copied from ospfd: the ev_sighup event handler
should handle SIGHUP and not SIGTERM. SIGTERM is already handled by
ev_sigterm.

Use SIGCHLD for the sigchild handler not SIGINT twice.
(This was inherited from ospfd; claudio fixed it there some time ago)
Spotted in ripd by haesbaert <at> haesbaert.org

Yet another implementation of IMSG_CTL_LOG_VERBOSE. Similar to the one in
ripd, ospfd and friends. OK michele, henning, sthen

make dvmrpd imsg-in-a-lib ready as well.
``put it in'' claudio@, ok eric@

no endpwent

Remove redundant imsg_event_add calls; ok claudio@

More libevent callback fixes. While there make n also ssize_t as in all
other daemons.

Error out with usage line if additional arguments are given after the
option parsing. Found out the hard way by jdixon on ifstated.

ok sobrado@, jdixon@, millert@

Sync parse.y with recent enhancements in hoststated.
Fix behavior when running -n.
henning@

in all these programs using the same pfctl-derived parse.y, re-unify the
yylex implementation and the code which interacts with yylex. this also
brings the future potential for include support to al

in all these programs using the same pfctl-derived parse.y, re-unify the
yylex implementation and the code which interacts with yylex. this also
brings the future potential for include support to all of the parsers.
in the future please do not silly modifications to one of these files
without checking if you are de-unifying the code.
checked by developers in all these areas.

show more ...

Ignore SIGPIPE in the parent and RDE as well.

ok claudio@