remove prototypes with no matching function and externs with no var
partly checked by millert@

getifaddrs() can return entries where ifa_addr is NULL. Check for this
before accessing anything in ifa_addr.
ok claudio@

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if

When system calls indicate an error they return -1, not some arbitrary
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.

show more ...

PF_ROUTE -> AF_ROUTE in the scattered sock()/setsockopt() calls
where the "wrong" #define was used.

ok dlg@

Remove RTM_LOSING from the file, it is no longer used by OpenBSD and
route6d is not realy using it anyway.
OK henning@

more RTM_LOCK removals

Retire support for unused RTM_LOCK messages, it's redundant w/ RTM_CHANGE.

ok tb@, sthen@

fix mem leak
from Thomas Barabosch (thomas.barabosch AT fkie fraunhofer de), thanks!
OK tb

Recycle IFF_NOTRAILERS into IFF_STATICARP and document ownerhsip
of IFF* flags.

inputs from jmc@, ok bluhm@, visa@

No need to handle more than one routing message here.

Add a -u switch to always log route insertions/deletions.

Route updates (and associated warnings) are always prefixed with "RTADD"
or "RTDEL". This is useful for people that previously used the -R

Add a -u switch to always log route insertions/deletions.

Route updates (and associated warnings) are always prefixed with "RTADD"
or "RTDEL". This is useful for people that previously used the -R
option that got removed.

Tested by Freddy Dissaux.

show more ...

Use tighter pledge(2) restrictions.

Made possible by the logging overhaul. Tested by Freddy Dissaux.

Rework logging.

Use log.c like in many other daemons, with small additions to maintain
a log queue (useful to dump a single log line with several logging
statements). Use log.c to dump the internal

Rework logging.

Use log.c like in many other daemons, with small additions to maintain
a log queue (useful to dump a single log line with several logging
statements). Use log.c to dump the internal state, and remove the -R
option (a similar feature will be implemented differently soon).

The end goal is to avoid touching the filesystem and use much tighter
pledge(2) restrictions.

Tested by Freddy Dissaux.

show more ...

Introduce RTF_MULTICAST and flag corresponding IPv6 routes as such
instead of abusing RTF_CLONING.

Fix a leak reporeted by Aaron Riekenberg on misc@, ok sthen@

route6d spring cleanup

Various tweaks and fixes:
- nuke util.h, not needed since pidfile(3) went away
- nuke the rrt_same member of struct riprt, "future use" since import
- mark rtdexit as __dead
-

route6d spring cleanup

Various tweaks and fixes:
- nuke util.h, not needed since pidfile(3) went away
- nuke the rrt_same member of struct riprt, "future use" since import
- mark rtdexit as __dead
- nuke progname handling
- fix pid handling: cache the pid *after* calling daemon(3)
- nuke setting rtm_pid. The kernel is responsible for setting this
in routing messages.
- nuke the useless myseq variable

ok florian@ benno@ millert@ deraadt@ renato@

show more ...

No need to declare main().

malloc+memset -> calloc

Remove NULL-check before free().

Sort includes.

Fix error message after conversion from select(2) to poll(2).

Partial revert of r1.79, "np" handling is a bit trickier than expected.

Rename the struct pollfd array, "set" -> "pfd"

"set" reminds of select(2) fd_set, and is never used elsewhere in the
tree as a name for a pollfd array. No functional change.

Those variables should be local to the functions where they are used.

Gets rid of a lot of warnings with -Wshadow. No functional change.

Kill unused variable and function.

No functional change.

hms() returns a pointer to static storage, annotate the return type with "const"

No functional change.