Age | Commit message (Collapse) | Author |
|
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
|
|
few remaining ".Tn UNIX" macros with ".Ux" ones.
pointed out by ratchov@, thanks!
ok jmc@
|
|
ok jmc@
|
|
Change this so that the real nexthop is announced if the following
conditions are met:
- the nexthop is reachable via an OSPF enabled interface
- the interface is a broadcast or NBMA interface
It does not make sense to announce the nexthop of a point-to-point link
since the traffic has to go through this box anyway. This is closer to
what other systems implement.
|
|
|
|
when fail-over happens, since removing the better route will not result
in a blackhole until the update from the new master is processed.
Tested, OK and input sthen@, phessler@
|
|
|
|
in an error path.
|
|
loop in the shutdown case. OK henning@
|
|
|
|
- remove some unnecessary \& escapes
ok claudio, jmc
|
|
function, which is additionally exported for use by others.
It will be needed by smtpd's SSL module when the SMTP client code
is changed to replace libevent's evbuffers with our msgbuf_* API.
ok gilles@ henning@ guenther@ eric@
|
|
bytes is not enough for larger networks causing send errors because of
too big packets. OK henning
|
|
freeing the msgbuf.
While here also remove an unnecessary while loop.
ok eric pyr
|
|
|
|
bytes that were filled, not the whole buffer.
ok pyr@ gilles@
|
|
|
|
|
|
Make the imsg protocol network-safe.
it might be network safe, but half the imsg based daemons on my firewalls
dont run anymore.
|
|
Currently the receiver fetches an imsg via imsg_get() and if he expects
an fd, he then calls imsg_get_fd() to fetch the next fd queued on the
imsgbuf from which the imsg came.
This changes hides the fd queueing mechanism to the API user. When closing
an imsg with an fd, the message is flagged so that the receiving end knows
it must dequeue the fd in imsg_get() and return it with the imsg structure.
This way there is no (less) possible screw up from imsg_get_fd() not being
called directly after imsg_get() by the user. The retreived imsg is
self-contained.
ok pyr@, "I like that" henning@
|
|
ok pyr@
|
|
add a flag field, use u_int32_t for pid_t and extend type to 32 bits
for padding.
ok pyr@
|
|
|
|
"looks ok" claudio@
|
|
ok eric@, claudio@
|
|
ok claudio@, henning@
|
|
|
|
ready daemons have NULL diffs now for the imsg files. This field should
eventually die a honorable death as it was a design screwup.
|
|
|
|
ok claudio@
|
|
|
|
this plays better with dynamic buffers which are now the norm.
ok by a slightly annoyed claudio@, ok eric@
|
|
ok claudio@
|
|
though they point to the loopback. Mainly used for redistribute default since
on default free routers we need to have a fake route now.
After discussion with Tonnerre Lombard, idea OK henning@
|
|
|
|
set at the same time so using a switch to do read or write is a bad idea.
Problem noticed by Eygene Ryabinkin on FreeBSD for some reasons it is not
triggered on OpenBSD.
Fix aggreed by a most other libevent hackers
|
|
Got broken in -r 1.62. Pointed out by bluhm@
OK bluhm@, claudio@
|
|
|
|
this is required to run multiple instances of ospfd.
ok claudio@
|
|
allocations fails.
looks right deraadt, krw
ok henning
|
|
Make this a more general concern about using 0 as key id. After discussion with
Tamas TEVESZ
|
|
and route labels.
ok claudio@
|
|
Now a default route have to be present in the fib to be correctly advertised.
Spotted and tested by Steven Surdok on ripd.
ok claudio@
|
|
so a warning later on could be hit in some cases. Found by dlg@ OK dlg@
|
|
buf/imsg specific code. buf_close() will no force a truncation of the buffer
to the wpos but actually add code in imsg.c to detect and report such silly
behaviour. Makes the buf API a bit more sane.
|
|
OK claudio@ stsp@
|
|
fails -- which should never happen -- the function does not return and is
accession the NULL set control pointer later on.
Found by Matthew Haub. OK deraadt@
|
|
jmc@ has provided a complete list of manual pages to be fixed,
and suggested using uppercase (i.e., Point-to-Point) when discussing
the protocol, and lowercase (point-to-point) otherwise.
ok jmc@
|
|
buf_left() returns the bytes left in a buffer. This makes the check to
keep MD5_DIGEST_LENGTH bytes unallocated in some messages easier.
From the buf cleanup diff that was OK norby@ and henning@
|
|
some direct access to struct buf internals in the rest of ospfe.
Based on a larger buffer cleanup diff that was OK norby@ henning@
|