CVS: cvs.openbsd.org: src

2017-05-28 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/05/28 22:40:35

Modified files:
libexec/getty  : main.c 

Log message:
do not need dev[] wasting bss space



CVS: cvs.openbsd.org: src

2017-05-28 Thread Henning Brauer
CVSROOT:/cvs
Module name:src
Changes by: henn...@cvs.openbsd.org 2017/05/28 18:05:28

Modified files:
share/misc : airport 

Log message:
mlarkin and airport.7 say "must have been there", not "flown there".
Add XFW, Hamburg-Finkenwerder (the Airbus factory)
ok mlarkin fcambus



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 17:56:13

Modified files:
usr.sbin/vmd   : vm.c 

Log message:
SVM: add some exit types

Also, fix a comment that wasn't applicable anymore, and change a format
from decimal to hex



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 17:50:20

Modified files:
sys/arch/amd64/include: vmmvar.h 

Log message:
rename some fields



CVS: cvs.openbsd.org: src

2017-05-28 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2017/05/28 17:23:40

Modified files:
usr.bin/tmux   : input.c 

Log message:
Support OSC 10 and 11 to set foreground and background colours, from
"bertnp" in GitHub issue 942.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 16:27:10

Modified files:
games/robots   : robots.6 

Log message:
remove some escapes that are unnecessary/harmful. ok jmc schwarze



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 15:59:56

Modified files:
usr.bin/signify: Makefile 
distrib/special/signify: Makefile 
Removed files:
usr.bin/signify: smult_curve25519_ref.c 

Log message:
when copying curvy files from ssh, an extra one snuck in.
signify doesn't do any kex stuff and doesn't need scalarmult.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 15:57:19

Modified files:
sys/net: pipex_local.h 

Log message:
fix broken include on previous pipex commit

ok deraadt



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 15:23:47

Modified files:
regress/sys/netinet/ipsec: Makefile 

Log message:
Now that pf looks behind IPv4 authentication headers, disable the
transport mode tests that fail because of floating states.  At least
IPv4 and IPv6 are in sync now.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Antoine Jacoutot
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/28 15:21:00

Modified files:
share/misc : airport 

Log message:
Add BND, I've been there many times.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Vadim Zhukov
CVSROOT:/cvs
Module name:src
Changes by: z...@cvs.openbsd.org2017/05/28 15:18:21

Modified files:
share/misc : airport 

Log message:
Add ZIA, even given that I didn't visit it after it actually became ZIA
instead of Ramenskoe Airfield.

approved by deraadt@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 15:13:39

Modified files:
games/robots   : rnd_pos.c 

Log message:
remove unused macro



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 15:09:24

Modified files:
games/robots   : robots.6 

Log message:
there are no bugs; i fixed them



CVS: cvs.openbsd.org: src

2017-05-28 Thread Christian Weisgerber
CVSROOT:/cvs
Module name:src
Changes by: na...@cvs.openbsd.org   2017/05/28 15:05:54

Modified files:
usr.sbin/slaacd: slaacd.c 

Log message:
only mark privacy addresses as such instead of all configured addresses
ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 15:01:13

Modified files:
games/robots   : main.c 

Log message:
if your dying scream would escape the death field, realign to stay within



CVS: cvs.openbsd.org: src

2017-05-28 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/05/28 14:48:29

Modified files:
sys/net: pipex.c 

Log message:
Process packets immediately without queuing since pipex is believed MP safe
already, for PPPoE case as first step.

ok mpi



CVS: cvs.openbsd.org: src

2017-05-28 Thread Florian Obser
CVSROOT:/cvs
Module name:src
Changes by: flor...@cvs.openbsd.org 2017/05/28 14:40:13

Modified files:
usr.sbin/slaacd: engine.c slaacd.c 

Log message:
removed if 0'ed code; it gets in the way of grepping for things



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 14:34:33

Modified files:
games/robots   : score.c 

Log message:
high scores printed too far to the right, move it left a little



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 14:20:00

Modified files:
sys/arch/amd64/amd64: vmm.c 

Log message:
typo in comment



CVS: cvs.openbsd.org: src

2017-05-28 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2017/05/28 14:15:02

Modified files:
usr.sbin/bgpd  : rde.c 

Log message:
Print when we send or recv an EOR marker.
Req by and OK benno@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2017/05/28 14:14:15

Modified files:
usr.sbin/bgpd  : bgpd.h logmsg.c 

Log message:
Introduce log_peer_info() and make log_peer_warn() log at LOG_ERR instead
of LOG_CRIT (which should only be used for fatal).
OK benno@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2017/05/28 14:10:59

Modified files:
usr.sbin/bgpd  : rde.c session.c 

Log message:
Close imsg pipes later in the process. The shutdown code still tires to
send imsgs and so the SE and RDE crashed because of this late in shutdown.
OK benno@ phessler@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Florian Obser
CVSROOT:/cvs
Module name:src
Changes by: flor...@cvs.openbsd.org 2017/05/28 13:57:38

Modified files:
usr.sbin/slaacd: engine.c engine.h slaacd.c 

Log message:
set autoconfprivacy flag; prodding naddy



CVS: cvs.openbsd.org: src

2017-05-28 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2017/05/28 13:46:55

Modified files:
usr.bin/tmux   : cmd-new-session.c 

Log message:
Change so that sessions created detached (-d or no client) are always
80x24 and the status line is not applied until they attach. Also make -x
and -y work for control clients whether the session is detached or not.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 13:44:52

Modified files:
share/man/man7 : Makefile 

Log message:
install new manpage, noted by tb@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 13:42:26

Modified files:
share/misc : Makefile 
Removed files:
share/misc : operator 

Log message:
please read operator(7) for this information.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 13:26:33

Added files:
share/man/man7 : airport.7 

Log message:
document /usr/share/misc/airport contents and rules, after lengthy
hackroom discussion about train stations.
ok mlarkin, feedback deraadt



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 13:15:18

Modified files:
share/misc : airport 

Log message:
typo



CVS: cvs.openbsd.org: src

2017-05-28 Thread Christian Weisgerber
CVSROOT:/cvs
Module name:src
Changes by: na...@cvs.openbsd.org   2017/05/28 13:13:13

Modified files:
usr.sbin/slaacd: engine.c 

Log message:
pasto, typos, spelling; ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2017/05/28 13:00:52

Modified files:
usr.bin/tmux   : cmd-refresh-client.c 

Log message:
Also recalculate session sizes when refreh-client -C is used. GitHub
issue 947.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 12:57:40

Modified files:
share/misc : airport 

Log message:
The High Council of Deciders has determined that railway stations, even
those with IATA codes, are not airports and therefore do not qualify for
inclusion in this file.



CVS: cvs.openbsd.org: src

2017-05-28 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/05/28 12:55:25

Modified files:
sys/net: pipex.c 

Log message:
Check also whether the interfaces is matched when pipex check PPPoE
packets.  This fixes the problem when pipex connects with pppoe(4)
through pair(4).



CVS: cvs.openbsd.org: src

2017-05-28 Thread Antoine Jacoutot
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/28 12:51:27

Modified files:
etc/rc.d   : rc.subr 

Log message:
Move check later to mitigate a possible race.



CVS: cvs.openbsd.org: src

2017-05-28 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/05/28 12:43:51

Modified files:
sys/net: if_pppx.c pipex_local.h 

Log message:
Use interface index and if_{put,get} instead of ifnet pointer.



CVS: cvs.openbsd.org: src

2017-05-28 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/05/28 12:42:00

Modified files:
sys/net: pipex.c 

Log message:
Use interface index instead of ifnet pointer.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Mike Larkin
CVSROOT:/cvs
Module name:src
Changes by: mlar...@cvs.openbsd.org 2017/05/28 12:35:58

Modified files:
share/misc : airport 

Log message:
add HWD - Hayward Executive Airport. After extensive discussion, the
decision was made that a missed approach at the DH counts as "being at
the airport"

ok phessler, deraadt



CVS: cvs.openbsd.org: src

2017-05-28 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/05/28 12:31:35

Modified files:
share/misc : inter.phone 

Log message:
convert to UTF-8



CVS: cvs.openbsd.org: src

2017-05-28 Thread Visa Hankala
CVSROOT:/cvs
Module name:src
Changes by: v...@cvs.openbsd.org2017/05/28 11:12:48

Modified files:
sys/arch/mips64/mips64: ipifuncs.c 

Log message:
Do not allow NULL callback at rendezvous and clear callback
pointer at the end to catch errors faster.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Joris Vink
CVSROOT:/cvs
Module name:src
Changes by: jo...@cvs.openbsd.org   2017/05/28 11:11:34

Modified files:
usr.bin/cvs: cvs.h file.c import.c 

Log message:
Let opencvs show ignored files on import, not showing them is just confusing.

ok stsp@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Joris Vink
CVSROOT:/cvs
Module name:src
Changes by: jo...@cvs.openbsd.org   2017/05/28 11:01:10

Modified files:
usr.bin/cvs: commit.c 

Log message:
Don't allow opencvs to commit towards tags that are not branches.

Gets rid of the old logic that wasn't working and replaced it with
a simplified version.

ok stsp@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Joris Vink
CVSROOT:/cvs
Module name:src
Changes by: jo...@cvs.openbsd.org   2017/05/28 10:58:54

Modified files:
usr.bin/cvs: status.c 

Log message:
Teach opencvs status to display the sticky tag according to what it really
represents if it is a branch or a revision.

ok stsp@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Joris Vink
CVSROOT:/cvs
Module name:src
Changes by: jo...@cvs.openbsd.org   2017/05/28 10:57:01

Modified files:
usr.bin/cvs: logmsg.c 

Log message:
Bring the opencvs log message template inline with other cvs implementations.

ok stsp@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 10:55:54

Modified files:
sys/net: pf.c 

Log message:
Limit the nested header chain for IPv6 extensions headers and for
authentication headers in the IPv4 case.  This prevents spending
excessive cpu time on crafted packets.
OK henning@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 10:52:27

Modified files:
usr.sbin/bgpd  : bgpd.conf.5 

Log message:
tweak previous;



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 10:43:45

Modified files:
sys/net: pf.c pfvar.h 

Log message:
Fix bad white spaces, wrap long lines, kill some empty lines.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Florian Obser
CVSROOT:/cvs
Module name:src
Changes by: flor...@cvs.openbsd.org 2017/05/28 10:36:53

Modified files:
usr.sbin/slaacd: engine.c 

Log message:
get a new privacy address before the old one expires



CVS: cvs.openbsd.org: src

2017-05-28 Thread Florian Obser
CVSROOT:/cvs
Module name:src
Changes by: flor...@cvs.openbsd.org 2017/05/28 09:58:02

Modified files:
usr.sbin/slaacd: engine.c 

Log message:
No need to constantly re-add the default route. It will not expire
like the prefixes.  We might want to check if someone deleted the
route by hand though.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jonathan Matthew
CVSROOT:/cvs
Module name:src
Changes by: jmatt...@cvs.openbsd.org2017/05/28 09:48:49

Modified files:
usr.sbin/ldapd : syntax.c 

Log message:
Fix checks for seconds and timezones in generalized times.
Fixing the CHECK_RANGE macro in r1.4 revealed that the seconds check
accidentally relied on the macro being broken.  While looking into this I
noticed that the timezone check was also wrong, treating the timezone as
optional for generalized times.

investigation and diff mostly by Seiya Kawashima.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Visa Hankala
CVSROOT:/cvs
Module name:src
Changes by: v...@cvs.openbsd.org2017/05/28 09:46:06

Modified files:
sys/arch/mips64/mips64: ipifuncs.c 

Log message:
Use fast path if remote call is not needed.

Note that rendezvous calls are no longer necessarily
serialized systemwide.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Anton Lindqvist
CVSROOT:/cvs
Module name:src
Changes by: an...@cvs.openbsd.org   2017/05/28 09:36:45

Modified files:
sys/dev/acpi   : dsdt.c 

Log message:
Remove a redundant assignment introduced in revision 1.219 but favor the
assignment outside of the conditional.

ok stsp@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Henning Brauer
CVSROOT:/cvs
Module name:src
Changes by: henn...@cvs.openbsd.org 2017/05/28 09:16:52

Modified files:
usr.sbin/bgpd  : bgpd.conf.5 

Log message:
we're not hardcoded to table 0 any more



CVS: cvs.openbsd.org: src

2017-05-28 Thread Henning Brauer
CVSROOT:/cvs
Module name:src
Changes by: henn...@cvs.openbsd.org 2017/05/28 09:16:33

Modified files:
usr.sbin/bgpd  : bgpd.c bgpd.h config.c kroute.c parse.y 

Log message:
so far, bgpd was hardcoded to use rtable 0 for nexthop verification.
instead, use the rtable bgpd was started in (route -T  exec / rc.d
daemon_rtable) for nexthop verification and as default Adj-RIB-In and
Loc-RIB. This allows multiple bgpds in different rdomains on the same
machine - bgp router virtualization if you like buzzwords.
initial version written under contract more than a year ago, it took us
a while to wrap our brains around the bgpd <-> rdomain interactions -
1) RIBs, 2) nexthop verification and 3) tcp sockets.
ok & input phessler claudio benno



CVS: cvs.openbsd.org: src

2017-05-28 Thread Visa Hankala
CVSROOT:/cvs
Module name:src
Changes by: v...@cvs.openbsd.org2017/05/28 09:16:08

Modified files:
sys/arch/mips64/mips64: ipifuncs.c 

Log message:
mips64_multicast_ipi() excludes current CPU.
The caller does not have to do that.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Michal Mazurek
CVSROOT:/cvs
Module name:src
Changes by: akf...@cvs.openbsd.org  2017/05/28 09:15:21

Modified files:
sbin/pfctl : parse.y 

Log message:
Don't check np->port for NULL - it's an array, it's never NULL.

OK bluhm@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 09:03:53

Modified files:
sys/net: if_trunk.c 

Log message:
Add missing NET_UNLOCK() in error path.

Spotted by sashan@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 08:54:00

Modified files:
sys/net: pf.c 

Log message:
Pf was handling IPv4 and IPv6 differently regarding AH extension
headers.  pf_walk_header6() steps over it and detects the real
protocol.  So to implement a minimal header walking function
pf_walk_header() for IPv4.  It does the header checks and jumps
over AH.  Then pf does not understand AH as a protocol, it is just
an extension that authenticates the packet.  Move some header and
option checks to pf_walk_header() for consistency with IPv6.  This
also improves the header check for IPv4 packets in ICMP payload.
OK henning@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Kenneth R Westerback
CVSROOT:/cvs
Module name:src
Changes by: k...@cvs.openbsd.org2017/05/28 08:37:48

Modified files:
sbin/dhclient  : dispatch.c 

Log message:
Elegant and reliable link status checking courtesy of mpi@.

Less of unreliable ioctl(SIOCGIFMEDIA), more getifaddrs().

ok mpi@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 08:26:38

Modified files:
sys/net: if_trunk.c 

Log message:
trunk_port_destroy() needs the NET_LOCK().

It brings the interface down and restore the original lladdr.

Found by Hrvoje Popovski



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 08:24:19

Modified files:
sys/ddb: db_ctf.c 
sys/arch/amd64/amd64: db_trace.c 
sys/arch/i386/i386: db_trace.c 

Log message:
If a function is not found in the CTF data, do not assume it takes no
argument.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 07:59:05

Modified files:
sys/netinet: ip_ipip.c 

Log message:
Call bpf_mtap_af() a bit earlier in ipip_input().  This prepares
upcoming diffs, no functional change.
OK mpi@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jonathan Gray
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 07:20:37

Modified files:
sys/arch/i386/conf: Makefile.i386 

Log message:
Build i386 kernels with -ffreestanding, matching amd64 and various
other platforms.

ok visa@ kettenis@



CVS: cvs.openbsd.org: src

2017-05-28 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/05/28 06:51:34

Modified files:
sys/net: if.c if_ethersubr.c if_pppx.c netisr.h pipex.c 
 pipex_local.h 

Log message:
Remove all splnet/splx from pipex(4) and pppx(4) and replace some of
them by NET_LOCK/NET_UNLOCK.  Also make the timeout for pipex_timer
run with a thread context and replace pipex softintr by NETISR_PIPEX.

ok mpi



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 06:47:24

Modified files:
sys/net: if.c 
sys/netinet: ip_carp.c 

Log message:
Leaving IP multicast group requires the NET_LOCK().

Grab the lock before calling carpdetach().

ok bluhm@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jonathan Gray
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 06:22:54

Modified files:
sys/netinet: ip_input.c 

Log message:
clang warns on unused labels.  Place a recently introduced label under
ifdef IPSEC to fix the clang build when IPSEC is not defined.

ok deraadt@ bluhm@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Claudio Jeker
CVSROOT:/cvs
Module name:src
Changes by: clau...@cvs.openbsd.org 2017/05/28 06:21:36

Modified files:
usr.sbin/bgpd  : bgpd.h control.c rde.c rde.h rde_rib.c 
 session.c session.h 

Log message:
Implement an XON/XOFF protocol between the RDE and the SE to throttle
per control session and peer the generation of imsg in the RDE. This
reduces the memory pressure in the SE substantially and also a bit in
the RDE. Makes the RDE more responsive for bgpctl commands.
Tested by me with 100 peers * 2000 prefixes and by phessler@ on an AMS-IX
border router with 200+ session. Convergance time got quite a bit better.
OK phessler@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 05:41:52

Modified files:
sys/ddb: db_ctf.c db_elf.c db_elf.h 

Log message:
Merge two functions to lookup ELF sections by name.

ok claudio@, jasper@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 05:17:33

Modified files:
usr.sbin/smtpd : smtpd.conf.5 

Log message:
reinstate the description of "mask-source" to "listen on socket": my changes
two revisions previous inadvertently removed it;

ok gilles



CVS: cvs.openbsd.org: src

2017-05-28 Thread Stefan Sperling
CVSROOT:/cvs
Module name:src
Changes by: s...@cvs.openbsd.org2017/05/28 05:03:48

Modified files:
sys/dev/pci: if_iwm.c 

Log message:
Fix some spurious fatal firmware errors in iwm(4).

If we are not in SCAN state anymore by the time hardware signals completion
of a scan command, exit the scan completion handler immediately instead of
calling ieee80211_end_scan().

Tested by tb@ and myself.
ok mpi@ tb@ zhuk@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:39:15

Modified files:
usr.sbin/relayd: agentx.c ca.c check_icmp.c check_script.c 
 check_tcp.c hce.c pfe.c pfe_filter.c 
 pfe_route.c proc.c relay.c relay_http.c 
 relay_udp.c relayd.c snmp.c ssl.c 

Log message:
use __func__ in log messages. fix some whitespace while here.
>From Hiltjo Posthuma hiltjo -AT codemadness -DOT- org, thanks!
ok florian, claudio



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:37:26

Modified files:
usr.sbin/httpd : config.c httpd.c httpd.h proc.c 

Log message:
use __func__ in log messages.
>From Hiltjo Posthuma hiltjo -AT codemadness -DOT- org, thanks!
ok florian, claudio



CVS: cvs.openbsd.org: src

2017-05-28 Thread Antoine Jacoutot
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/28 04:12:42

Modified files:
etc/rc.d   : rc.subr 

Log message:
Indent and rename var; no functional change.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:06:13

Modified files:
usr.sbin/traceroute: traceroute.c traceroute.h 

Log message:
style(9) some variable declarations
ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Michal Mazurek
CVSROOT:/cvs
Module name:src
Changes by: akf...@cvs.openbsd.org  2017/05/28 04:06:12

Modified files:
usr.sbin/tcpdump: pfctl_osfp.c 

Log message:
Reduce differences between the two pfctl_osfp.c files.

Apply three commits from pfctl/pfctl_osfp.c

OK bluhm@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:04:27

Modified files:
usr.sbin/traceroute: traceroute.c traceroute.h worker.c 

Log message:
check_tos() gets a parameter so i can remove another global var.
ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:01:52

Modified files:
usr.sbin/traceroute: traceroute.c traceroute.h worker.c 

Log message:
introduce struct tr_conf to keep all of the configuration.
Functions needing access to any of those vars get it passed as a parameter.
result: even less global vars.
ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Sebastian Benoit
CVSROOT:/cvs
Module name:src
Changes by: be...@cvs.openbsd.org   2017/05/28 04:00:00

Modified files:
usr.sbin/traceroute: traceroute.c 

Log message:
move as many globals as possible into the main function - thats the
only place where they are used.
Only exception: v6flags - make it an argument to usage()
ok florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Stefan Sperling
CVSROOT:/cvs
Module name:src
Changes by: s...@cvs.openbsd.org2017/05/28 03:59:58

Modified files:
sys/dev/pci: if_iwm.c if_iwmvar.h 

Log message:
Remove unused flag IWM_FLAG_STOPPED.
ok tedu@ of course



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 03:45:25

Modified files:
lib/librthread : rthread_mutex.c 

Log message:
Use membar_enter_after_atomic() and membar_exit_before_atomic().



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 03:38:32

Modified files:
sys/net: if_trunk.c 

Log message:
Remove useless splnet()/splx() dances.

Data structures modified in the ioctl path are protected by the NET_LOCK().

ok sashan@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Florian Obser
CVSROOT:/cvs
Module name:src
Changes by: flor...@cvs.openbsd.org 2017/05/28 03:35:56

Modified files:
usr.sbin/slaacd: engine.c engine.h frontend.c slaacd.c slaacd.h 
usr.sbin/slaacctl: slaacctl.c 

Log message:
propose and configure default gateway



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 03:35:14

Modified files:
sys/net: if_enc.c 

Log message:
Protect the global array of interfaces with the NET_LOCK().

ok sashan@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Antoine Jacoutot
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/28 03:31:45

Modified files:
etc/rc.d   : rc.subr 

Log message:
When a daemon reaches its timeout when starting, display "timeout" instead
of "ok" so the user is warned and has a chance to fix it (most of the time
due to bogus flags). Daemons reaching the timeout without being able to
start are still marked as "failed" (which should also give a clue to the
user that some investigation is needed).

prodded by beck@ a while ago
discussed with and ok sthen@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Alexander Bluhm
CVSROOT:/cvs
Module name:src
Changes by: bl...@cvs.openbsd.org   2017/05/28 03:25:51

Modified files:
sys/netinet: ip_input.c ip_var.h ipsec_input.c 
sys/netinet6   : ip6_input.c ip6_var.h 

Log message:
Rename ip_local() to ip_deliver() and give it the same parameters
as the pr_input functions.  Add an assert that IPv4 delivery ends
in IP proto done to assure that IPv4 protocol functions work like
IPv6.
OK mpi@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Robert Peichaer
CVSROOT:/cvs
Module name:src
Changes by: r...@cvs.openbsd.org2017/05/28 03:24:56

Modified files:
distrib/miniroot: install.sub 

Log message:
Remove _mode variable and use AI_MODE directly instead. Use the
exit code of ftp consistently to determine success of fetching the
response file.

discussed with and OK tb@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 03:00:24

Modified files:
usr.sbin/smtpd : smtpd.conf.5 

Log message:
mark up "masquerade";



CVS: cvs.openbsd.org: src

2017-05-28 Thread Jason McIntyre
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/28 02:59:05

Modified files:
usr.sbin/smtpd : smtpd.conf.5 

Log message:
split the two "listen on" directives into two separate items;
the markup that we were using wouldn;t have worked with groff anyway
but, more worringly, it didn;t work with mandoc either;



CVS: cvs.openbsd.org: src

2017-05-28 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/05/28 02:51:06

Modified files:
libexec/getty  : main.c 

Log message:
Do a better job of not printing sequences we cannot reverse in DELETE or
KILL.  Therefore we can do a better job cleaning up.
testing by benno



CVS: cvs.openbsd.org: src

2017-05-28 Thread Martin Pieuchot
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2017/05/28 02:47:19

Modified files:
sys/arch/amd64/amd64: lock_machdep.c 
sys/arch/i386/i386: lock_machdep.c 

Log message:
Use intr_disable()/intr_restore() to reduce differences with sparc64
mp_lock.

ok kettenis@, visa@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Antoine Jacoutot
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/28 02:38:10

Modified files:
etc/rc.d   : rc.subr 

Log message:
Drop useless lines continuation; no functional change.



CVS: cvs.openbsd.org: src

2017-05-28 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/05/28 02:07:33

Modified files:
etc: netstart 

Log message:
etc/netstart: use colon separator instead of dot with chown

OK jung@, deraadt@, jmc@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/05/28 02:03:36

Modified files:
bin/chmod  : Makefile chmod.c chown.8 

Log message:
chown: Remove SUPPORT_DOT ifdef - it's on by default for 22 years

The old syntax was deprecated 25 years ago when the utility was
first standardised in IEEE Std 1003.2-1992 ("POSIX.2"). There was
no POSIX version of chown with the dot separator.

Let's stop pretending that it will ever go away.

OK jung@, deraadt@, jmc@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Michal Mazurek
CVSROOT:/cvs
Module name:src
Changes by: akf...@cvs.openbsd.org  2017/05/28 01:40:12

Modified files:
usr.sbin/tcpdump: pfctl_osfp.c 

Log message:
Catch up with pfctl/pfctl_osfp.c, no binary change.

OK deraadt@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Anton Lindqvist
CVSROOT:/cvs
Module name:src
Changes by: an...@cvs.openbsd.org   2017/05/28 01:27:01

Modified files:
bin/ksh: vi.c 
regress/bin/ksh/vi: vi.sh 

Log message:
Fix cursor position while removing characters from the command line.

While here, remove a condition that becomes redundant.

ok schwarze@ tb@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Michal Mazurek
CVSROOT:/cvs
Module name:src
Changes by: akf...@cvs.openbsd.org  2017/05/28 01:17:53

Modified files:
sbin/pfctl : pfctl_osfp.c 

Log message:
print_ioctl() is unused if not debugging. Found with clang, after
marking the function "static". Use OSFP_DEBUG, in a similar
fashion to OPT_DEBUG (pfctl_optimize.c).

OK bluhm@