summaryrefslogtreecommitdiffstats
path: root/usr.sbin/dvmrpd (follow)
Commit message (Collapse)AuthorAgeFilesLines
* list example files in FILES with a short description: generally, "Examplejmc2020-05-161-4/+4
| | | | | | | 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
* briefly mention /etc/examples/ in the FILES section of all theschwarze2020-02-101-3/+5
| | | | | manual pages that document the corresponding configuration files; OK jmc@, and general direction discussed with many
* Consistently use _rcctl enable foo_ in examples, it's simpler and lesslandry2019-11-101-12/+9
| | | | | | | | | | | | | error prone than manually editing rc.conf.local, and also works to enable ipsec and accounting. tweak from schwarze@ to use the \(dq\(dq syntax for quotes in '.Dl foo_flags="" lines' instead of \&"\&". while at it, fix a reference to a bogus /dev/dhclient.conf file that recently snuck in. ok jmc@ deraadt@ schwarze@
* When system calls indicate an error they return -1, not some arbitraryderaadt2019-06-282-18/+18
| | | | | | 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.
* (unsigned) means (unsigned int) which on ptrdiff_t or size_t or otherderaadt2019-02-131-3/+3
| | | | | | larger types really is a range reduction... Almost any cast to (unsigned) is a bug. ok millert tb benno
* - odd condition/test in PF lexersashan2018-11-011-2/+3
| | | | | | | | | | | | | | (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@
* add missing braces implied by indentationjsg2018-09-211-2/+3
| | | | ok millert@ claudio@
* replace malloc()+strlcpy() with strndup() in cmdline_symset().miko2018-09-071-9/+4
| | | | "looks good" gilles@ halex@
* Do for most running out of memory err() what was done for most runningkrw2018-07-111-3/+3
| | | | | | | | 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@
* No need to mention which memory allocation entry point failed (malloc,krw2018-07-091-4/+4
| | | | | | | | | 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@
* Be consistent in warn() and log_warn() usage whenkrw2018-07-081-3/+3
| | | | | | | | running out of memory. Next step, be correct *and* consistent. ok dennis@ tb@ benno@ schwarze@
* remove the SECTIONS header, since a one line DESCRIPTION is a bit silly;jmc2018-06-181-4/+4
| | | | | | | 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
* Plug leak in error case of the common 'varset' implementations.krw2018-04-261-1/+3
| | | | ok benno@
* Unify ROUNDUP macros for parsing route messages.friehm2017-07-241-4/+4
| | | | | | Use the macro from route(8) / ospf6d(8) since it works also with argument 0. OK claudio@
* From a syslog perspective it does not make sense to log fatal andbluhm2017-03-211-6/+6
| | | | | | warn with the same severity. Switch log_warn() to LOG_ERR and keep fatal() at LOG_CRIT. OK reyk@ florian@
* Replace hand-rolled for(;;) traversal of ctl_conns TAILQ withkrw2017-01-081-7/+9
| | | | | | | | TAILQ_FOREACH(). No intentional functional change. ok reyk@
* Replace hand-rolled for(;;) emptying of 'symhead' TAILQ with morekrw2017-01-051-3/+2
| | | | | | | | modern TAILQ_FOREACH_SAFE(). No intentional functional change. ok millert@ bluhm@ gilles@
* Replace symset()'s hand-rolled for(;;) traversal of 'symhead' TAILQkrw2017-01-051-5/+7
| | | | | | | | | | | 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@
* work on making log.c similar in all daemons:benno2016-09-028-84/+121
| | | | | | | 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@
* Simplify shutdown process.renato2016-09-023-72/+43
| | | | | | | | | | | | | 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. "reads good" claudio@
* Close connections when msgbuf_write() returns 0.renato2016-09-023-8/+18
| | | | | | Pulled from ospfd. Original author: claudio@ ok rzalamena@ benno@ claudio@
* Don't fatal if the imsg pipe is closed, this is often triggered in therenato2016-09-023-16/+48
| | | | | | | | parent and hides the real cause of the termination. Pulled from ospfd. Original author: claudio@ ok rzalamena@ benno@ claudio@
* do not allow whitespace in macro names, i.e. "this is" = "a variable".benno2016-06-211-1/+9
| | | | | | | | 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@
* Remove setproctitle() for the parent process. Because rc.d(8) uses processsthen2016-02-021-4/+1
| | | | | | 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@
* Remove unneeded strings.h includes. From Serguey Parkhomovsky.mmcc2015-12-073-6/+3
|
* Replace bzero with memset, and change a strings.h include to string.h tommcc2015-12-076-14/+14
| | | | | get the declaration. From Serguey Parkhomovsky. No objects from deraadt@.
* strings.h -> string.h to prevent implicit declarations. From Sergueymmcc2015-12-072-4/+4
| | | | Parkhomovsky.
* EAGAIN handling for imsg_read. OK henning@ benno@claudio2015-12-054-11/+12
|
* As done for bgpd recently, rename if_mediatype to if_type in dvrmpd.stsp2015-09-273-10/+10
| | | | | Remove unused function get_ifms_type(). No ifmedia in here anymore. "move forward" deraadt@
* use file system path (.Pa) semantic markup macros where appropriate.sobrado2015-07-271-4/+4
| | | | ok jmc@
* fix a non safe use of LIST_FOREACH with LIST_REMOVEjsg2015-05-221-2/+2
| | | | ok claudio@ kettenis@ reyk@
* use the sizeof the struct not the sizeof a pointer to the structjsg2015-05-057-14/+14
| | | | ok claudio@
* Use sizeof(u_short) in the first check since there are RT messages thatclaudio2015-02-111-2/+2
| | | | | are less then sizeof(*rtm) bytes long (e.g. interface announcements). Found the hard way by phessler@
* More SOCK_NONBLOCK adjustments similar to the other daemons.claudio2015-02-103-7/+16
|
* Oups forgot to remove the enumsclaudio2015-02-091-6/+1
|
* Kill another session_socket_blockmode() ... 6 or so to go.claudio2015-02-093-38/+13
|
* Replace <sys/param.h> with <limits.h> and other less dirty headers wherederaadt2015-01-162-4/+2
| | | | | | | | | 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)
* Don't allow embedded nul characters in strings.jsg2014-11-201-1/+4
| | | | | | | Fixes a pfctl crash with an anchor name containing an embedded nul found with the afl fuzzer. pfctl parse.y patch from and ok deraadt@
* Nuke more obvious #include duplications.krw2014-11-184-8/+4
| | | | ok deraadt@ millert@ tedu@
* Add gcc printf format attributes to yyerror() in parse.y files.doug2014-11-141-2/+4
| | | | | | No yyerror() calls needed to be changed. ok bluhm@
* Convert the logic in yyerror(). Instead of creating a temporarybluhm2014-11-033-10/+9
| | | | | format string, create a temporary message. OK benno@ doug@ claudio@
* Remove unnecessary netinet/in_systm.h include.lteo2014-10-258-16/+8
| | | | ok millert@
* Fixed typo in error message.tobias2014-07-201-2/+2
| | | | okay deraadt@
* msgbuf_write() <= 0 for a few more daemons.krw2014-07-123-8/+8
| | | | ok claudio@
* Close the control fd when it has reported EOF.krw2014-07-111-2/+2
| | | | ok henning@
* The second level of the CTL_NET sysctl is a PF_*, not an AF_*guenther2014-06-231-2/+2
| | | | inconsistent usage in route(8) noted by Gregor Best (gbe (at) ring0.de)
* relax the cfg file secrecy check slightly to allow group readabilityhenning2014-01-221-3/+3
| | | | | default permissions and mtree NOT changed. prodded by benno, ok phessler benno jmatthew theo pelikan florian
* deal with msgbuf_write EAGAIN, ok gilles bennohenning2013-11-264-10/+10
|
* use u_char for buffers in yylex, for ctype callsbenno2013-11-251-7/+7
| | | | found by millert@, ok deraadt@
* no longer any need to quote macro lines with >9 args;jmc2013-08-141-3/+3
| | | | From: Jan Stary
Copyright © 1996 – 2023 Jason A. Donenfeld. All Rights Reverse Engineered.