Age | Commit message (Collapse) | Author |
|
ok miod@ millert@
|
|
|
|
rename nconf in the ospf engine to noeconf. This fixes the last common
symbols in ospfd.
|
|
further. This is in line with what other networking daemons do.
ok mestre@
|
|
OK florian@, millert@, kn@
|
|
|
|
area. Most of the other interface settings allow this.
ok denis@
|
|
No functional change.
Suggested and OK claudio@
|
|
tested by Kapetanakis Giannis
ok claudio@
|
|
OK remi@
|
|
Shorten it to better match the style of other log messages.
ok jca@
|
|
Additional code would be needed to support actually changing the
router-id at runtime; for now be consistent and reject the new
configuration to avoid users a "wtf" moment.
From Mitchell Krome, input from benno@ and deraadt@, ok benno@
|
|
overwrite this when using "redistribute X set type 2 depend on ifX" and ifX
is down.
Problem reported for ospfd by Igor Podlesny.
ok benno@
|
|
OK millert, deraadt, job
|
|
OK claudio@
|
|
inserts into the kernel routing table.
OK claudio@
|
|
Unveil "/" readonly and allow to cleanup the control socket on exit.
Just unveiling ospfd.conf is not possible since it can contain an arbitrary
number of includes.
ok benno@
|
|
socket. The check on the control socket happens early so that the rc
scripts detect a failure.
OK claudio@ florian@
|
|
ok friehm@ jca@
|
|
but only when -v is used.
ok jca@ claudio@
|
|
need to see routes with a higher priority (lower value) than ospfds
own routes.
ok claudio, ok henning previous version, feedback from sthen
|
|
ok remi@ benno@
|
|
on the status of another interface.
in collaboration with benno@ jca@
OK benno@ jca@
|
|
there is still a little difference regarding handling of the verbosity
value that will be handled later.
ok claudio@
|
|
|
|
stealing code.
|
|
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.
ok benno@ claudio@
|
|
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@, feedback from henning@, deraadt@, reyk@
|
|
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@
|
|
around to make it possible. Parent can't be pledged at the moment because
of carp ioctl (carp demote). Putting it in so that people can test.
OK benno@
|
|
This is in the way for ospfd pledge so in it goes. OK benno@
|
|
And some ifmedia64 fixes.
"move forward" deraadt@
|
|
imsg_init() exposed by the second part of this diff which makes
carp(4) interfaces be recognized to be in "backup" mode on start-up.
Problem analyzed and fix provided by Johan Ymerson, thanks!
ok claudio@, mpi@
|
|
neighbors that are part of that interface or we open us up to use after
free situations like the one found by sthen@.
Diff makes sense sthen@
|
|
socket non-blocking. Introduce the same trigger for partial rt msgs.
|
|
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)
|
|
ok claudio@
|
|
"Let msgbuf_write return -1 with errno EAGAIN. The users then must
check if this was the case and readd the event or poll again. The
current handling in the imsg code is wrong for sure."
ok gilles, benno
|
|
|
|
"ospfd -v" (previously only "-vd" worked). Feedback on earlier implementation
from henning@ blambert@ claudio@, ok claudio@
|
|
to IF_STA_LOOPBACK in if_act_start() this way they will repsect
IFF_UP on startup. Also remove a now no longer needed workaround
when reloading interfaces.
Initial diff provided by Patrick Coleman. OK dlg@
|
|
leak is a runtime one.
OK bluhm@
|
|
(verified by both sthen@ and me).
ok sthen@; "just commit it" claudio@
|
|
is forced into IF_STA_NEW so that if_init() can be called to setup timers etc.
When a loopback interface is added to the config, this means there's no way
to get to the correct state.
Fix by avoiding changing an existing IF_STA_LOOPBACK and forcing loopback
interfaces to be passive (otherwise skipping the if_init on an active
interface causes problems).
ok claudio@
|
|
this tells the daemon to resync the kernels list of interfaces and routes
with the daemons list. this is very useful if the routing socket overflows
and you want to sync things up again.
lots and lots of help from claudio@
ok claudio@
|
|
with reloads when running ospfd on multiple aliases on the same interface.
Is also needed to handle interface address changes in a much better way.
OK dlg@, sthen@
|
|
It always annoyed me that in case of a problem I had to restart the ospf in
forground debug mode and by doing so losing all routes at least twice.
OK henning, sthen, michele
|
|
"looks ok" claudio@
|
|
ok claudio@
|
|
|