summaryrefslogtreecommitdiff
path: root/usr.sbin/ospfd
AgeCommit message (Collapse)Author
2018-02-22The IF_EVT_NBR_CHNG event needs to be fired when a neighbor transitions toClaudio Jeker
a state of 2-Way or higher. There is no need to trigger the event for new neighbors. With this situations with multiple DRs after a netsplit should be solved. OK sthen@ and remi@
2018-02-11Use the new route filter ROUTE_PRIOFILTER in ospfd. Usually we onlySebastian Benoit
need to see routes with a higher priority (lower value) than ospfds own routes. ok claudio, ok henning previous version, feedback from sthen
2018-02-09Style tweaksJeremie Courreges-Anglas
ok remi@ benno@
2018-02-09Simplify logic a bit by moving a block. nbr_stop_itimer() does not useClaudio Jeker
the dr or bdr fields so clear them first.
2018-02-08Bump the read sockbuf of the routing socket to 2MB, may help reduce some ofClaudio Jeker
the desync cases. OK benno@
2018-02-05simplify macros, and avoid some arg splitting;Jason McIntyre
2018-02-05Introduce "depend on". This allows ospfd to set the metric dependentremi
on the status of another interface. in collaboration with benno@ jca@ OK benno@ jca@
2017-11-07document how ospfd interacts with carpremi
improvement from jmc@ ok phessler@ jca@ benno@ for previous version ok jmc@
2017-07-24Unify ROUNDUP macros for parsing route messages.friehm
Use the macro from route(8) / ospf6d(8) since it works also with argument 0. OK claudio@
2017-03-21From a syslog perspective it does not make sense to log fatal andAlexander Bluhm
warn with the same severity. Switch log_warn() to LOG_ERR and keep fatal() at LOG_CRIT. OK reyk@ florian@
2017-01-24sync log.c from relayd et al to ospfd.Sebastian Benoit
there is still a little difference regarding handling of the verbosity value that will be handled later. ok claudio@
2017-01-23copy log.c/h from bgpd.Sebastian Benoit
ok claudio
2017-01-23Remove incomplete "forced nexthop" support.Jeremie Courreges-Anglas
Discussed with claudio@
2017-01-08Replace hand-rolled for(;;) traversal of ctl_conns TAILQ withKenneth R Westerback
TAILQ_FOREACH(). No intentional functional change. ok reyk@
2017-01-05Replace hand-rolled for(;;) emptying of 'symhead' TAILQ with moreKenneth R Westerback
modern TAILQ_FOREACH_SAFE(). No intentional functional change. ok millert@ bluhm@ gilles@
2017-01-05Replace symset()'s hand-rolled for(;;) traversal of 'symhead' TAILQKenneth R Westerback
with more modern TAILQ_FOREACH(). This what symget() was already doing. Add paranoia '{}' around body of symget()'s TAILQ_FOREACH(). No intentional functional change. ok bluhm@ otto@
2016-12-27struct rroute has been removed, rename variables.Jeremie Courreges-Anglas
ok claudio@
2016-12-26Fix typo, happend -> happenedJeremie Courreges-Anglas
2016-12-21For IFINFO msgs, don't run the FSM if the interface state hasn't changed.Jeremie Courreges-Anglas
ok claudio@
2016-12-21When receiving an IFINFO message, also update the MTU of the interfaceJeremie Courreges-Anglas
Helps recovering from MTU mismatches. Prompted by a diff from Remi Locherer, ok claudio@
2016-11-19Print metric and type for "redistribute" in ospfd -v.Stuart Henderson
From Remi Locherer, ok claudio
2016-09-28'unneded stuff' -> 'unneeded stuff' in comments.Kenneth R Westerback
2016-09-04A few bits of stray leading/embedded/trailing whitespace spotted whileKenneth R Westerback
stealing code.
2016-09-03Simplify shutdown process.Renato Westphal
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@
2016-09-02work on making log.c similar in all daemons:Sebastian Benoit
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@
2016-09-02work on making log.c similar in all daemons:Sebastian Benoit
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@
2016-06-21do not allow whitespace in macro names, i.e. "this is" = "a variable".Sebastian Benoit
change this in all config parsers in our tree that support macros. problem reported by sven falempin. feedback from henning@, stsp@, deraadt@ ok florian@ mikeb@
2016-06-06add format attributes to the proper functions and then fix theSebastian Benoit
warning in rde.c ok stsp@
2016-02-18Fix format string of a warning.Alexander Bluhm
from markus@; OK claudio@
2016-02-02Remove setproctitle() for the parent process. Because rc.d(8) uses processStuart Henderson
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@
2015-12-05Pledge ospfd SE ("stdio inet mcast") and RDE ("stdio") move some codeClaudio Jeker
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@
2015-12-03imsg_read will return EAGAIN in some cases so prep code for it.Claudio Jeker
This is in the way for ospfd pledge so in it goes. OK benno@
2015-11-22Improve ABR support especially for self-originated stub networks.Claudio Jeker
This seems to solve the last issues people reported when using ospfd in multiple areas. OK sthen@ prodding by deraadt@
2015-10-26Kernel got fixed we no longer need to check if rtm_pid is 0 to filter outClaudio Jeker
route messages that are not from us.
2015-10-24Filter our RTM_GET messages which are not from us.Claudio Jeker
2015-09-27As done for bgpd recently, rename if_mediatype to if_type in ospfd/ospf6d.Stefan Sperling
And some ifmedia64 fixes. "move forward" deraadt@
2015-07-27use file system path (.Pa) semantic markup macros where appropriate.Igor Sobrado
ok jmc@
2015-07-20Fix a segfault at startup when if_change() ist called beforeSebastian Benoit
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@
2015-07-19revert previousSebastian Benoit
Properly handle carp(4) interfaces in "backup" mode on start-up. Problem analyzed and fix provided by Johan Ymerson, thanks! ok claudio@, benno@ it breaks on systems without carp because iev_ospfe is not initiallized at the time kif_init() is called in main.
2015-07-18Properly handle carp(4) interfaces in "backup" mode on start-up.Martin Pieuchot
Problem analyzed and fix provided by Johan Ymerson, thanks! ok claudio@, benno@
2015-07-17Do similar handling of connected routes and filtering of llinfo and broadcast.Claudio Jeker
With this the ospfd fib and the kernel routing table are better in sync.
2015-05-05use the sizeof the struct not the sizeof a pointer to the structJonathan Gray
ok claudio@
2015-03-14When removing interfaces in the RDE we also need to remove all the RDEClaudio Jeker
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@
2015-03-13Make sure that the debug messages include the neighbor ID so we hava a chanceClaudio Jeker
to know which neighbor caused the problems. While there make sure that all messages log roughly in the same way.
2015-03-11tweak wording; "a value different to 1" -> "...other than 1", from Sevan JaniyanStuart Henderson
2015-02-11Use sizeof(u_short) in the first check since there are RT messages thatClaudio Jeker
are less then sizeof(*rtm) bytes long (e.g. interface announcements). Found the hard way by phessler@
2015-02-10Forgot the EAGAIN or EINTR check on read.Claudio Jeker
2015-02-10Convert ospfd over to SOCK_CLOEXEC | SOCK_NONBLOCK and make the routeClaudio Jeker
socket non-blocking. Introduce the same trigger for partial rt msgs.
2015-01-28When comparing the LSA with the Ack, also check the age field. ThisAlexander Bluhm
ensures that a LSA withdrawal is not acked by a previous update. From Florian Riehm; OK claudio@
2015-01-22remove a stupid -Wstrict-prototype warning by making the hash functionTed Unangst
static. noticed by florian riehm