Age | Commit message (Collapse) | Author |
|
OK tb@
|
|
Also use ibuf_data() and ibuf_size().
OK tb@
|
|
ok miod@ millert@
|
|
various *d, *conf, *ctl files (where relevant) and simple;
also makes "man -k routing" more useful;
help from claudio and florian
ok claudio florian millert
|
|
Remove the IN_BADCLASS() check which filters out the experimental IPv4
address space. Now there are no more experiments in IPv4 and so there
is less reason for these network daemons to deny such an IP.
Everything still disallows multicast IPs (224/4) and loopback (127/8)
a few also disallow 0/8 but this is not consistent.
In any case using 240/4 in production is a really bad idea but it is
not up to this software to prevent you from being a fool.
OK deraadt@ tb@
|
|
macro-build a replacement for sccsid, and was done without any concern
for namespace damage. Unfortunately this practice started infecting
other code as others were unaware they didn't need the file.
ok millert guenther
|
|
functions that take "char *" arguments. Where such chars are
assigned to int or passed to ctype functions, explicitly cast them
to unsigned char.
For OpenBSD's clang, -Wpointer-sign has been disabled by default,
but when the parse.y code was built elsewhere, the compiler would
complain.
With help from millert@
ok benno@ deraadt@
|
|
|
|
This code assumes some alignment of the buffer which may not be
the case with bss memory.
|
|
|
|
define READ_BUF_SIZE like the other daemons.
|
|
|
|
Instead assign the process name directly to log_procname which was
the last user of ripd_process.
|
|
This log.c should be further synced with the other log.c files.
|
|
it is needed (the if_del call in ripe.c). With this the ripd_process
check in ripe_demote_iface() can be removed.
|
|
|
|
Internalize control_state and ctl_conns.
|
|
configuration file.", but occasionally something else fit better; at the
same time, try to make the format for FILES more consistent;
original diff from clematis
|
|
manual pages that document the corresponding configuration files;
OK jmc@, and general direction discussed with many
|
|
RFC 2082, chapter 3.1 sys about the offset:
A 16 bit offset from the RIP-2 header to the MD5 digest (if no
other trailer fields are ever defined, this value equals the
RIP-2 Data Length).
Problem reported by Hiltjo Posthuma.
OK claudio@ deraadt@ benno@
|
|
for passive interface a bit further up so that the function can return
earlier.
Memory leak observed and reported by Jason Tubnor.
OK benno@
|
|
frees kr if it detects a duplicate).
From Hiltjo Posthuma
OK benno@
|
|
because of split-horizon simple. This fixes empty response packages sent
out by ripd.
Adapt send_request() to follow the same layout.
Input and OK claudio@
|
|
OK claudio@
|
|
OK claudio@
|
|
to misread.
as per suggestion by and OK deraadt@
|
|
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
|
|
OK claudio@ deraadt@
|
|
ok krw@, jsg@
|
|
larger types really is a range reduction...
Almost any cast to (unsigned) is a bug.
ok millert tb benno
|
|
insers into the kernel routing table.
OK deraadt@
|
|
an interface config example with and one without parameters.
comment and OK claudio@
|
|
OK claudio@
|
|
Use unveil to remove file system access for the parent proc.
Also remove control_cleanup as discussed on tech@. It allows to fully remove
fs access (ripd does not support reload). It's not an issue for ripd and
ripctrl if an unused socket file is lying around.
reads OK for claudio@ and mestre@
|
|
(and other lexers too)
This commit rectifies earlier change:
in the lex... even inside quotes, a \ followed by space or tab should
expand to space or tab, and a \ followed by newline should be ignored
(as a line continuation). compatible with the needs of hoststated
(which has the most strict quoted string requirements), and ifstated
(where one commonly does line continuations in strings).
OK deraadt@, OK millert@
|
|
"looks good" gilles@ halex@
|
|
out of memory log_warn(). i.e. ("%s", __func__) instead of manual
function names and redundant verbiage about which wrapper detected the
out of memory condition.
ok henning@
|
|
calloc or strdup), we just need to log that we ran out of memory in a
particular function.
Recommended by florian@ and deraadt@
ok benno@ henning@ tb@
|
|
running out of memory.
Next step, be correct *and* consistent.
ok dennis@ tb@ benno@ schwarze@
|
|
use a more general text for the sections, and avoid the catchup issue
that was trying to document how many there were;
ok benno rob
|
|
ok benno@
|
|
the desync cases. OK benno@
|
|
Use the macro from route(8) / ospf6d(8) since it works also with argument 0.
OK claudio@
|
|
warn with the same severity. Switch log_warn() to LOG_ERR and keep
fatal() at LOG_CRIT.
OK reyk@ florian@
|
|
steal code.
|
|
This mimics what ospfd does, and avoids a (mostly harmless)
use-after-free. Delay suggested by claudio@, ok florian@
|
|
TAILQ_FOREACH().
No intentional functional change.
ok reyk@
|
|
modern TAILQ_FOREACH_SAFE().
No intentional functional change.
ok millert@ bluhm@ gilles@
|
|
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@
|
|
Diff from Piotr Durlej, similar diff from chris@, ok mpi@
|