CVS: cvs.openbsd.org: src

2017-01-17 Thread Peter Hessler
CVSROOT:/cvs
Module name:src
Changes by: phess...@cvs.openbsd.org2017/01/17 21:28:45

Modified files:
usr.sbin/bgpd  : parse.y 

Log message:
while a u_int is large enough for 32bit-asns, it is not big enough for
some of the magic values we use to indicate '*' or neighbor-as.

fixes "allow from any large-community neighbor-as:*:*"



CVS: cvs.openbsd.org: src

2017-01-17 Thread Ricardo Mestre
CVSROOT:/cvs
Module name:src
Changes by: mes...@cvs.openbsd.org  2017/01/17 16:28:04

Modified files:
libexec/spamd  : spamd.c 

Log message:
- spamd(8)'s -l accepts an IP address as argument to bind(2) and it calls
inet_pton(3) to check if it's valid and since that function doesn't provide
a proper errno (POSIX doesn't mandate to do so) then if a string is given we
may get this message:
spamd: inet_pton: Undefined error: 0

- Instead replace that code to use getaddrinfo(3) from which is possible to get
a proper error message, and at the same time being able to parse IPs and
hostnames (if either the IP or host is not local then the next bind(2) will
fail)

- By default without arguments, spamd(8) will still bind(2) to 127.0.0.1 as
it did before

With feedback from deraadt@ and OK beck@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2017/01/17 15:10:56

Modified files:
sbin/iked  : control.c 
usr.sbin/bgpd  : config.c 
usr.sbin/httpd : proc.c 
usr.sbin/ldapd : log.c 
usr.sbin/ldomctl: config.c 
usr.sbin/ripd  : control.c 
usr.sbin/snmpd : proc.c 
usr.sbin/switchd: control.c proc.c 

Log message:
Nuke some whitespace that keeps poking me in the eye as I try to
steal code.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2017/01/17 14:51:01

Modified files:
usr.sbin/vmd   : config.c control.c i8253.c i8259.c 
 loadfile_elf.c mc146818.c pci.c proc.c virtio.c 
 vmm.c 

Log message:
Nuke some whitespace that keeps poking me in the eye as I try to
steal code.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/01/17 14:08:34

Modified files:
share/man/man5 : pf.conf.5 

Log message:
"hosts" is optional; from matthew martin
ok henning



CVS: cvs.openbsd.org: src

2017-01-17 Thread Robert Nagy
CVSROOT:/cvs
Module name:src
Changes by: rob...@cvs.openbsd.org  2017/01/17 13:58:56

Modified files:
distrib/syspatch: bsd.syspatch.mk 

Log message:
rework some of the variables for better directory name handling and
add support for linking against libraries inside the fake root so
that syspatches can include static binaries linked with the fixed libraries



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 13:35:49

Modified files:
sys/conf   : GENERIC 

Log message:
Enable switch(4).

ok jca@, reyk@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Stefan Sperling
CVSROOT:/cvs
Module name:src
Changes by: s...@cvs.openbsd.org2017/01/17 13:07:39

Modified files:
sys/dev/ic : athn.c 

Log message:
Add missing malloc(9) return value check in athn_node_alloc().
ok millert@ tom@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Jeremie Courreges-Anglas
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/01/17 12:05:47

Modified files:
sbin/route : route.c 

Log message:
In monitor mode, also print the interface mtu of RTM_IFINFO messages.

ok deraadt@ millert@ mpi@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Stefan Kempf
CVSROOT:/cvs
Module name:src
Changes by: ste...@cvs.openbsd.org  2017/01/17 10:19:21

Modified files:
sys/uvm: uvm_addr.c uvm_addr.h uvm_map.c 

Log message:
Remove uaddr_hint allocator

The hint allocator would have to check that the
allocation does not overlap with brk, stack or text
areas. This would make the address selectors too
entagled. Just use the rnd allocator for hinted allocations
in case pivots are used. This also reduces the amount of code somewhat.

ok kettenis visa deraadt



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 09:54:40

Modified files:
sys/net: switchofp.c 

Log message:
Make switch(4) compile with debug again.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 09:47:55

Modified files:
sys/net: switchofp.c 

Log message:
Initialize swas_actions for set-field with the expected pointer to the
swpld_set_fields vector. This makes write-action work with set-field
actions.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Jeremie Courreges-Anglas
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/01/17 09:30:54

Modified files:
usr.sbin/ripd  : interface.c neighbor.c ripe.h 

Log message:
Keep track of dead peers instead of freeing them right away.

This mimics what ospfd does, and avoids a (mostly harmless)
use-after-free.  Delay suggested by claudio@, ok florian@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/01/17 08:32:39

Modified files:
usr.bin/mandoc : html.c html.h man_html.c mdoc_html.c 

Log message:
Completely delete the buf field of struct html and all the buf*()
interfaces.  Such a static buffer was a bad idea in the first place,
causing unfixable truncation that was only prevented by triggering
an assertion failure.  Instead, let the small number of remaining
users allocate and free their own, temporary dynamic buffers,
or for the case of .Xr and .In, pass the original data to be
assembled in print_otag().



CVS: cvs.openbsd.org: src

2017-01-17 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/01/17 06:19:37

Modified files:
regress/lib/libtls/tls: tlstest.c 

Log message:
Correctly tls_config_set_ca_file() return value (no effective change).

Spotted by inoguchi@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 05:34:21

Modified files:
sys/net: switchofp.c 

Log message:
Fix a panic when set-field with VLAN is set, but no VLANs were
classified in the packet.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 05:30:35

Modified files:
sys/net: if_switch.c switchofp.c 

Log message:
Clean-up switch(4) device by removing excess prototypes, verbose debugs
and unused functions.

ok reyk@



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 02:36:28

Modified files:
sys/net: switchofp.c 

Log message:
Add more action specific validations, unbreak instructions validation
with multiple actions and add more error reports with what went wrong.



CVS: cvs.openbsd.org: src

2017-01-17 Thread Rafael Zalamena
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/01/17 02:21:50

Modified files:
usr.sbin/switchd: ofp13.c 

Log message:
Fix OXM parser for set-field messages. Found it the hardway while
testing set-field configuration.