Age | Commit message (Collapse) | Author | |
---|---|---|---|
2018-02-22 | The IF_EVT_NBR_CHNG event needs to be fired when a neighbor transitions to | Claudio 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-11 | Use the new route filter ROUTE_PRIOFILTER in ospfd. Usually we only | Sebastian 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-09 | Style tweaks | Jeremie Courreges-Anglas | |
ok remi@ benno@ | |||
2018-02-09 | Simplify logic a bit by moving a block. nbr_stop_itimer() does not use | Claudio Jeker | |
the dr or bdr fields so clear them first. | |||
2018-02-08 | Bump the read sockbuf of the routing socket to 2MB, may help reduce some of | Claudio Jeker | |
the desync cases. OK benno@ | |||
2018-02-05 | simplify macros, and avoid some arg splitting; | Jason McIntyre | |
2018-02-05 | Introduce "depend on". This allows ospfd to set the metric dependent | remi | |
on the status of another interface. in collaboration with benno@ jca@ OK benno@ jca@ | |||
2017-11-07 | document how ospfd interacts with carp | remi | |
improvement from jmc@ ok phessler@ jca@ benno@ for previous version ok jmc@ | |||
2017-07-24 | Unify 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-21 | From a syslog perspective it does not make sense to log fatal and | Alexander Bluhm | |
warn with the same severity. Switch log_warn() to LOG_ERR and keep fatal() at LOG_CRIT. OK reyk@ florian@ | |||
2017-01-24 | sync 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-23 | copy log.c/h from bgpd. | Sebastian Benoit | |
ok claudio | |||
2017-01-23 | Remove incomplete "forced nexthop" support. | Jeremie Courreges-Anglas | |
Discussed with claudio@ | |||
2017-01-08 | Replace hand-rolled for(;;) traversal of ctl_conns TAILQ with | Kenneth R Westerback | |
TAILQ_FOREACH(). No intentional functional change. ok reyk@ | |||
2017-01-05 | Replace hand-rolled for(;;) emptying of 'symhead' TAILQ with more | Kenneth R Westerback | |
modern TAILQ_FOREACH_SAFE(). No intentional functional change. ok millert@ bluhm@ gilles@ | |||
2017-01-05 | Replace symset()'s hand-rolled for(;;) traversal of 'symhead' TAILQ | Kenneth 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-27 | struct rroute has been removed, rename variables. | Jeremie Courreges-Anglas | |
ok claudio@ | |||
2016-12-26 | Fix typo, happend -> happened | Jeremie Courreges-Anglas | |
2016-12-21 | For IFINFO msgs, don't run the FSM if the interface state hasn't changed. | Jeremie Courreges-Anglas | |
ok claudio@ | |||
2016-12-21 | When receiving an IFINFO message, also update the MTU of the interface | Jeremie Courreges-Anglas | |
Helps recovering from MTU mismatches. Prompted by a diff from Remi Locherer, ok claudio@ | |||
2016-11-19 | Print 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-04 | A few bits of stray leading/embedded/trailing whitespace spotted while | Kenneth R Westerback | |
stealing code. | |||
2016-09-03 | Simplify 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-02 | work 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-02 | work 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-21 | do 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-06 | add format attributes to the proper functions and then fix the | Sebastian Benoit | |
warning in rde.c ok stsp@ | |||
2016-02-18 | Fix format string of a warning. | Alexander Bluhm | |
from markus@; OK claudio@ | |||
2016-02-02 | Remove setproctitle() for the parent process. Because rc.d(8) uses process | Stuart 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-05 | Pledge ospfd SE ("stdio inet mcast") and RDE ("stdio") move some code | Claudio 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-03 | imsg_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-22 | Improve 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-26 | Kernel got fixed we no longer need to check if rtm_pid is 0 to filter out | Claudio Jeker | |
route messages that are not from us. | |||
2015-10-24 | Filter our RTM_GET messages which are not from us. | Claudio Jeker | |
2015-09-27 | As done for bgpd recently, rename if_mediatype to if_type in ospfd/ospf6d. | Stefan Sperling | |
And some ifmedia64 fixes. "move forward" deraadt@ | |||
2015-07-27 | use file system path (.Pa) semantic markup macros where appropriate. | Igor Sobrado | |
ok jmc@ | |||
2015-07-20 | Fix a segfault at startup when if_change() ist called before | Sebastian 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-19 | revert previous | Sebastian 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-18 | Properly 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-17 | Do 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-05 | use the sizeof the struct not the sizeof a pointer to the struct | Jonathan Gray | |
ok claudio@ | |||
2015-03-14 | When removing interfaces in the RDE we also need to remove all the RDE | Claudio 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-13 | Make sure that the debug messages include the neighbor ID so we hava a chance | Claudio Jeker | |
to know which neighbor caused the problems. While there make sure that all messages log roughly in the same way. | |||
2015-03-11 | tweak wording; "a value different to 1" -> "...other than 1", from Sevan Janiyan | Stuart Henderson | |
2015-02-11 | Use sizeof(u_short) in the first check since there are RT messages that | Claudio Jeker | |
are less then sizeof(*rtm) bytes long (e.g. interface announcements). Found the hard way by phessler@ | |||
2015-02-10 | Forgot the EAGAIN or EINTR check on read. | Claudio Jeker | |
2015-02-10 | Convert ospfd over to SOCK_CLOEXEC | SOCK_NONBLOCK and make the route | Claudio Jeker | |
socket non-blocking. Introduce the same trigger for partial rt msgs. | |||
2015-01-28 | When comparing the LSA with the Ack, also check the age field. This | Alexander Bluhm | |
ensures that a LSA withdrawal is not acked by a previous update. From Florian Riehm; OK claudio@ | |||
2015-01-22 | remove a stupid -Wstrict-prototype warning by making the hash function | Ted Unangst | |
static. noticed by florian riehm |