Commits
- Commit:
6b5c4000434cc5dc67e202d8aede2f543b5b2bf3- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
Standardize logging with bgpd
OK claudio@
- Commit:
5b68d4baa971368454ca9967d0353d29749d64e1- From:
- claudio <claudio@openbsd.org>
- Date:
Spaces vs tabs
from florian@
- Commit:
7afbe98723dac710f6f4d73a6b85a7f4804ef5ee- From:
- claudio <claudio@openbsd.org>
- Date:
Rewrite pfe_route() to actually work on 64bit archs since IPv6 had to be
special. One can not define a struct for the route message since there is
different padding between 32 and 64 bit systems for struct sockaddr_in6.
Instead do what all other daemons do and use struct sockaddr_storage,
iovec and writev.
Problem reported by Joerg Streckfuss (streckfuss at dfn-cert.de)
OK tb@
- Commit:
09e53feeaa2a7c28a81c8bac20bab18c68d83883- From:
- benno <benno@openbsd.org>
- Date:
use __func__ in log messages. fix some whitespace while here.
From Hiltjo Posthuma hiltjo -AT codemadness -DOT- org, thanks!
ok florian, claudio
- Commit:
58fc320c602b00aed04b7887496e83d9892bed86- From:
- reyk <reyk@openbsd.org>
- Date:
Split "struct relayd" into two structs: "struct relayd" and "struct
relayd_config". This way we can send all the relevant global
configuration to the children, not just the flags and the opts.
With input from and
OK claudio@ benno@
- Commit:
141b9058ce43c28db842829304e8909a29d231bb- From:
- reyk <reyk@openbsd.org>
- Date:
In most cases we don't need all arguments of proc_compose*_imsg(),
so add a shortcut proc_compose*() that skips all of them. Only use the
full argument list if needed. The functions with full argument lists can
eventually be replaced with a nicer transaction-based approach later.
OK benno@
- Commit:
d93c36af2b8111f07ff7645770d4714a2e5aef96- From:
- reyk <reyk@openbsd.org>
- Date:
Clean up the relayd headers with help of include-what-you-use and some
manual review. Based on common practice, relayd.h now includes the
necessary headers for itself.
OK benno@
- Commit:
2f6ec04dcc8d2ec0050455263b94308f36252825- From:
- deraadt <deraadt@openbsd.org>
- Date:
Adapt to <limits.h> universe.
ok millert
- Commit:
270ed42a50acc8797df2df92305e394b39ded472- From:
- guenther <guenther@openbsd.org>
- Date:
Stop pulling in <arpa/inet.h> or <arpa/nameser.h> when unnecessary.
*Do* pull it in when in_{port,addr}_h is needed and <netinet/in.h> isn't.
ok reyk@
- Commit:
dc2f787fcea89b92a6b72e8b198262e0b0b3fa50- From:
- reyk <reyk@openbsd.org>
- Date:
sync copyright to reality according to my last changes
- Commit:
591e6ab59d9901a8a5a1eec8f4593ffc39656829- From:
- reyk <reyk@openbsd.org>
- Date:
Fix reload support in relayd(8) by reimplementing large parts of the
daemon infrastructure. The previous design made it fairly hard to
reload the complex data structures, especially relays and protocols.
One of the reasons was that the privsep'd relayd processes had two
ways of getting their configuration: 1) from memory after forking from
the parent process and 2) and (partially) via imsgs after reload. The
new implementation first forks the privsep'd children before the
parents loads the configuration and sends it via imsgs to them; so it
is only like 2) before. It is based on an approach that I first
implemented for iked(8) and I also fixed many bugs in the code.
Thanks to many testers including dlg@ sthen@ phessler@
ok pyr@ dlg@ sthen@
- Commit:
dbef7d4f82e95c334a131206f7d2ac02d3ef21bd- From:
- reyk <reyk@openbsd.org>
- Date:
Reorganize the relayd code to use the proc.c privsep API/commodity
functions that are based on work for iked and smtpd. This simplifies
the setup of privsep processes and moves some redundant and repeated
code to a single place - which is always good from a quality and
security point of view. The relayd version of proc.c is different to
the current version in iked because it uses 1:N communications between
processes, eg. a single parent process is talking to many forked relay
children while iked only needs 1:1 communications.
ok sthen@ pyr@
- Commit:
666bd316da38eef77379216e92680abfa93b938a- From:
- reyk <reyk@openbsd.org>
- Date:
Update all logging and debug functions to use the __func__ macro
instead of static function names. __func__ is C99 and perfectly fine
to use. It also avoids printing errors; for example if a statement
log_debug("foo:"..) was moved or copied from function foo() to bar()
and the log message was not updated...
- Commit:
246e5b16c2222a5a673b14cbe221b8ae0eb7c686- From:
- phessler <phessler@openbsd.org>
- Date:
Allow a user to specify the route priority
OK reyk@ claudio@ sthen@
- Commit:
39bc6125909b70df97543dfd1457cd1a8f26d73d- From:
- reyk <reyk@openbsd.org>
- Date:
add new 'router' functionality to dynamically add or remove routes
based on health check results, using the existing table syntax. this
allows to maintain multiple (uplink) gateways to implement link
balancing or WAN link failover if no routing protocol or other
keepalive method is available. works fine with or without
net.inet.ip.multipath enabled.
ok pyr@, jmc@ for manpages
