summaryrefslogtreecommitdiff
path: root/usr.sbin/rtadvd
AgeCommit message (Collapse)Author
2016-05-07Replace .Po/.Pc with .Pq, and drop extra .Li; suggested by jmc@Jeremie Courreges-Anglas
2016-05-07Document that both raflags and rtflags can hold route preference flags.Jeremie Courreges-Anglas
Input from an ok jmc@
2016-03-01fatal("malloc") -> fatal(NULL)Jeremie Courreges-Anglas
2016-03-01Pointless 'return;' in void functionsJeremie Courreges-Anglas
2016-03-01Kill unused #definesJeremie Courreges-Anglas
No functional change
2016-02-29Merge add/make_prefix() and kill last use of struct in6_prefixreq in userland.Jeremie Courreges-Anglas
ok florian@
2016-02-29init_prefix() is now trivial, merge it with its caller.Jeremie Courreges-Anglas
Suggested by and ok bluhm@
2016-02-26sys/signal.h -> signal.hJeremie Courreges-Anglas
2016-02-26Unused variable.Jeremie Courreges-Anglas
2016-02-26GC remnants of the Router Renumbering code; ok florian@Jeremie Courreges-Anglas
2016-02-26Remove #if'0ed code that uses a deprecated ioctl; ok florian@Jeremie Courreges-Anglas
2016-02-26Kill some useless members of struct rainfo; ok florian@Jeremie Courreges-Anglas
2016-02-09another typoJeremie Courreges-Anglas
2016-02-09Kill code to send unicast packets which never got enabled.Jeremie Courreges-Anglas
and get rid of pointless memory allocations. ok bluhm@
2016-02-08Kill trailing whitespace.Jeremie Courreges-Anglas
2016-02-08typosJeremie Courreges-Anglas
2016-02-08Print MAX addresses padded with a leading zero.Jeremie Courreges-Anglas
2016-02-07Mark a few others functions as __dead; ok bluhm@Jeremie Courreges-Anglas
2016-02-05select -> poll in log message; ok bluhm@ florian@Jeremie Courreges-Anglas
2016-02-05Bail out if an unknown option is passed.Jeremie Courreges-Anglas
There is no reason for rtadvd to be different from other daemons. Input from and ok bluhm@ florian@
2016-02-05rtadvd used a strncpy with a src interface name from the command lineReyk Floeter
without checking its length. Replace it with strlcpy, check for truncation and move the check before using the user-specified interface name. While here, replace another strncpy of a interface name with memset-zero + strlcpy and check for truncation in if_getmtu() (should not happen as we validated the length before, but it's better to be safe and generally a better practice). OK jsg@
2016-01-25Redundant assignement is redundant.Jeremie Courreges-Anglas
2015-12-11Remove NULL-checks before free().mmcc
2015-12-01Use setsockopt(ROUTE_MSGFILTER) instead of hand-rolled filtering.Jeremie Courreges-Anglas
ok florian@
2015-12-01This RTM_GET case can't trigger due to the filtering done above.Jeremie Courreges-Anglas
ok florian@
2015-11-30Kill Mobile IPv6 references. Also spotted by florian@Jeremie Courreges-Anglas
2015-11-30Clean up get_next_msg(): kill dead argument/codeJeremie Courreges-Anglas
The "ifindex" argument has always been zero, making the interface index checks within get_next_msg() useless. The caller already properly retrieves and handle the interface index of the routing messages. ok florian@
2015-11-30Remove the already disabled Router Renumbering support (RFC2894).Jeremie Courreges-Anglas
This code has only been reachable for two months after rtadvd has been imported into the tree. This unsafe protocol is supposed to be used along with IPsec and rrenumd(8), a KAME tool that we don't have. I couldn't find an example of use in the real world. ok sthen@ florian@ millter@
2015-11-30Fix dynamic prefix tracking.Jeremie Courreges-Anglas
This code should not be using RTAX_GATEWAY to retrieve the interface index. Also this index is meaningless within get_next_msg(), further cleanup is planned. Issue reported by Sly Midnight on bugs@, ok mpi@
2015-11-03Recognize carp(4) interfaces (yes, they are special...) in order to sendMartin Pieuchot
the src lladdr option. Problem reported and diff provided by Lauri Tirkkonen on bugs@ ok sthen@
2015-10-25In case of error, also log the file name we failed to open.Jeremie Courreges-Anglas
2015-10-25ansifyJeremie Courreges-Anglas
2015-10-25pledge "stdio inet route"; ok deraadt@Jeremie Courreges-Anglas
2015-10-25Stop writing down a pidfile.Jeremie Courreges-Anglas
pid files aren't useful on OpenBSD, the rc.d(8) framework doesn't use them. In rtadvd, not writing a pid file means that we can set tighter pledge(2) settings. Also the pid file handling is pretty basic and the atexit handler can't remove the pid file since rtadvd did a chroot - beurk. ok florian@, "perfect" deraadt@ (!)
2015-08-20stdlib.h is in scope; do not cast malloc/calloc/realloc*Theo de Raadt
ok millert krw
2015-08-18Fix inconsistent spelling of `inconsistent'Miod Vallat
2015-06-03Do not assume that asprintf() clears the pointer on failure, whichTodd C. Miller
is non-portable. Also add missing asprintf() return value checks. OK deraadt@ guenther@ doug@
2015-06-02write Mobile IP{,v6} in a consistent way, as given in RFCs 5944 and 6275.Igor Sobrado
ok jmc@
2015-04-21According to RFC 4861 and the actuall implementation rltime may notFlorian Obser
exceed 9000 seconds. Reported by Alexander Lobodzinski on bugs@, thanks! OK benno@
2015-01-19remove #ifndef #define blocks trying to handle lack of BUFSIZTheo de Raadt
ok millert
2015-01-16Replace <sys/param.h> with <limits.h> and other less dirty headers whereTheo de Raadt
possible. Annotate <sys/param.h> lines with their current reasons. Switch to PATH_MAX, NGROUPS_MAX, HOST_NAME_MAX+1, LOGIN_NAME_MAX, etc. Change MIN() and MAX() to local definitions of MINIMUM() and MAXIMUM() where sensible to avoid pulling in the pollution. These are the files confirmed through binary verification. ok guenther, millert, doug (helped with the verification protocol)
2015-01-01"The logging of errors SHOULD be restricted to conflicting informationFlorian Obser
that causes hosts to switch from one value to another with each received advertisement." (RFC 4861) Receiving of RAs on non-advertising interfaces is not a conflicting information; don't log it. "Mike." (the.lists AT mgm51 DOT com) reported that this leads to excessive logging with certain cable providers in 2013. Recently reported again by Aaron Riekenberg (aaron.riekenberg AT gmail DOT com). Thanks! OK bluhm@
2015-01-01KNF; no binary changeFlorian Obser
2014-12-03rtsol is goneFlorian Obser
OK deraadt@
2014-11-18Nuke more obvious #include duplications.Kenneth R Westerback
ok deraadt@ millert@ tedu@
2014-11-02make comment select() vs poll() agnosticTheo de Raadt
2014-11-02convert select() to poll().Theo de Raadt
tested by brad (who found a conversion bug..)
2014-10-22Replace abs(3) with llabs(3).Brad Smith
Pointed out by LLVM. rtadvd.c:979:7: warning: absolute value function 'abs' given an argument of type 'long long' but has parameter of type 'int' which may cause truncation of value [-Wabsolute-value] ok dcoppa@ millert@
2014-07-04malloc+memset -> callocPhilip Guenther
from Benjamin Baier (programmer (at) netzbasis.de)
2014-06-29If you want <sys/queue.h>, you need to include it. Don't assume thatTheo de Raadt
<sys/sysctl.h> will pull in the universe (I am working on breaking that assumption in a gentle fashion)