CVS: cvs.openbsd.org: src

2017-06-12 Thread YASUOKA Masahiko
CVSROOT:/cvs
Module name:src
Changes by: yasu...@cvs.openbsd.org 2017/06/12 23:40:22

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

Log message:
Initialize the length parameter for radius_get_vs_raw_attr() since
it's read/write.  diff from IIJ.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 19:44:29

Modified files:
sys/arch/macppc/conf: Makefile.macppc files.macppc 
sys/arch/macppc/macppc: locore.S 
Added files:
sys/arch/macppc/macppc: locore0.S 

Log message:
Split early startup code out of locore.S into locore0.S.  Adjust link
run so that this locore0.o is always at the start of the executable.
But randomize the link order of all other .o files in the kernel, so
that their exec/rodata/data/bss segments land all over the place.

As a result, the internal layout of every newly build bsd kernel is
different from past kernels.  Internal relative offsets are not known
to an outside attacker.  The only known offsets are in the startup code
(which will be gone when it is smashed).

The bootstrap code unmapping or smashing isn't done yet, and there is
some potential for more bits from locore.S to move to locore0.S



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 19:42:52

Added files:
sys/arch/macppc/conf: ld.script 

Log message:
add a dummy ld.script.  macppc does not use one of these yet, but
infrastructure that is coming requires an architecture to carry such a
file.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 19:42:12

Modified files:
sys/arch/macppc/macppc: machdep.c 

Log message:
Prepare install_extint() for the vector being relative-negative (ie
lower in the address space).  Previously it could only be higher
than... uhm locore.o
Discussed with drahn and tom



CVS: cvs.openbsd.org: www

2017-06-12 Thread Jonathan Gray
CVSROOT:/cvs
Module name:www
Changes by: j...@cvs.openbsd.org2017/06/12 18:33:00

Modified files:
.  : errata60.html 

Log message:
correct a typo



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 18:16:09

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

Log message:
First step of randomized link for sparc64:  gap.S and randomize files.
locore isn't split up yet, nor is the startup code smashed.
ok dlg, pieces seen before by kettenis



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ricardo Mestre
CVSROOT:/cvs
Module name:src
Changes by: mes...@cvs.openbsd.org  2017/06/12 17:37:44

Modified files:
sbin/pflogd: privsep.c 

Log message:
pledge(2) bpf has been in use for some time now on tcpdump(8), this will enable
it also for pflogd(8)'s priv proc.

OK deraadt@



CVS: cvs.openbsd.org: src

2017-06-12 Thread David Gwynne
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/06/12 17:20:10

Modified files:
sys/net: hfsc.c 

Log message:
have the timeout call if qstart via the serialiser isntead of directly

hfsc schedules a timeout to keep traffic moving if somethings has been
delayed an no other tx activity has occurred. that timeout was calling
(*ifp->if_qstart)(ifq) rather than ifq_start. the latter prevents
concurrent calls to if_qstart.

without this change bjorn ketelaars on misc@ was experiencing weird
pauses in traffic and lockups because the tx ring was corrupted
because re_start was run concurrently, once from the stack and once
from hfsc.

thanks to bjorn ketelaars for debugging, and mikeb@ for most of the
legwork in diagnosing the problem.
ok mikeb@



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 16:50:02

Modified files:
regress/usr.bin/mandoc/tbl/data: Makefile 
regress/usr.bin/mandoc/tbl/opt: box.in box.out_ascii 
Added files:
regress/usr.bin/mandoc/tbl/data: block_wrap.in 
 block_wrap.out_ascii 

Log message:
test the new allbox and wrapping features



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 16:48:52

Modified files:
usr.bin/mandoc : tbl_term.c 

Log message:
Two minor fixes for the "allbox" modifier:
1. It does not reduce explicit "||" in the layout to "|".
2. It does not cause three horizontal lines at the end of a table,
even if the table ends with an explicit "_" data line.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 16:05:41

Modified files:
usr.bin/mandoc : out.c 

Log message:
If a tbl(7) layout contains a 'w' (minimum width) modifier for a
given column, that column contains no literal or numeric cell of
larger width, and all text block cells in that column can be line
wrapped to fit into that minimum width, groff does not increase
that column width beyond the specified minimum: so do the same.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 15:55:16

Modified files:
sys/kern   : kern_pledge.c 

Log message:
Pledge is fairly done, so the kernel printf's can be converted to log()
calls.  They'll be a little less visible, but still in the system logs.
ok bluhm



CVS: cvs.openbsd.org: src

2017-06-12 Thread Michal Mazurek
CVSROOT:/cvs
Module name:src
Changes by: akf...@cvs.openbsd.org  2017/06/12 14:56:20

Modified files:
games/fortune/datfiles: fortunes2 

Log message:
/usr/games/lib doesn't exist any more, use the new path.

OK tb@ deraadt@



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 14:44:57

Modified files:
usr.bin/mandoc : tbl_term.c 

Log message:
implement the tbl(7) "allbox" option;
used for example by curs_getch(3) and GLwDrawingArea(3)



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 14:14:03

Modified files:
usr.bin/mandoc : out.c out.h tbl_html.c tbl_term.c 

Log message:
fix column width calculation for text block cells



CVS: cvs.openbsd.org: www

2017-06-12 Thread T . J . Townsend
CVSROOT:/cvs
Module name:www
Changes by: t...@cvs.openbsd.org2017/06/12 13:59:19

Modified files:
.  : errata60.html errata61.html 

Log message:
release wsmux and sti errata.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/06/12 12:56:13

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

Log message:
Use unlinkat() when removing the contents of a directory so we don't
need to construct paths.  OK deraadt@



CVS: cvs.openbsd.org: src

2017-06-12 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2017/06/12 12:55:42

Modified files:
usr.bin/mandoc : mandoc_aux.c mandoc_aux.h tbl_term.c term.c 
 term.h 
regress/usr.bin/mandoc/char/unicode: input.out_ascii 

Log message:
Implement automatic line breaking
inside individual table cells that contain text blocks.
This cures overlong lines in various Xenocara manuals.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Marc Espie
CVSROOT:/cvs
Module name:src
Changes by: es...@cvs.openbsd.org   2017/06/12 12:51:53

Modified files:
sbin/ldattach  : Makefile 

Log message:
redundant, gets BINDIR from ../Makefile.inc



CVS: cvs.openbsd.org: src

2017-06-12 Thread Marc Espie
CVSROOT:/cvs
Module name:src
Changes by: es...@cvs.openbsd.org   2017/06/12 12:49:34

Modified files:
distrib/special/route: Makefile 
etc: Makefile 
games/wargames : Makefile 
gnu/lib/libobjc: Makefile.bsd-wrapper 
gnu/lib/libstdc++: Makefile.bsd-wrapper 
gnu/usr.bin/binutils: Makefile.bsd-wrapper 
gnu/usr.bin/binutils-2.17: Makefile.bsd-wrapper 
gnu/usr.bin/cvs: Makefile.bsd-wrapper 
gnu/usr.bin/gcc: Makefile.bsd-wrapper 
gnu/usr.bin/texinfo: Makefile.bsd-wrapper 
sbin/route : Makefile 
share/dict : Makefile 
share/misc : Makefile 
share/snmp : Makefile 
share/tabset   : Makefile 
share/tmac : Makefile 
share/tmac/mdoc: Makefile 
sys/arch/alpha/stand/boot: Makefile 
sys/arch/alpha/stand/bootxx: Makefile 
sys/arch/alpha/stand/netboot: Makefile 
usr.sbin/bind  : Makefile.bsd-wrapper 
usr.sbin/nsd   : Makefile.bsd-wrapper 

Log message:
destroy lint remnants.
okay millert@ deraadt@



CVS: cvs.openbsd.org: src

2017-06-12 Thread T . J . Townsend
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/06/12 12:41:46

Modified files:
sys/dev/wscons : Tag: OPENBSD_6_0 wsmux.c 

Log message:
MFC: specify free() size from the old allocation, not new allocation.

ok tb



CVS: cvs.openbsd.org: src

2017-06-12 Thread T . J . Townsend
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/06/12 12:40:43

Modified files:
sys/dev/wscons : Tag: OPENBSD_6_1 wsmux.c 

Log message:
MFC: specify free() size from the old allocation, not new allocation.

ok tb



CVS: cvs.openbsd.org: src

2017-06-12 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/06/12 12:38:57

Modified files:
usr.bin/vi/common: recover.c 

Log message:
Use openat() and unlinkat() instead of chdir()ing to the recovery dir.
Since we use flock() and not fcntl() locking we can open the recovery
file read-only.  OK martijn@



CVS: cvs.openbsd.org: src

2017-06-12 Thread T . J . Townsend
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/06/12 12:36:58

Modified files:
sys/dev/ic : Tag: OPENBSD_6_0 sti.c 

Log message:
MFC: fix an integer overflow in two range checks of the sti display driver.

ok tb



CVS: cvs.openbsd.org: src

2017-06-12 Thread Todd C . Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/06/12 12:37:12

Modified files:
lib/libc/stdio : printf.3 

Log message:
Add dprintf() and vdprintf() RETURN VALUES.  OK deraadt@



CVS: cvs.openbsd.org: src

2017-06-12 Thread T . J . Townsend
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2017/06/12 12:34:05

Modified files:
sys/dev/ic : Tag: OPENBSD_6_1 sti.c 

Log message:
MFC: fix an integer overflow in two range checks of the sti display driver.

ok tb



CVS: cvs.openbsd.org: src

2017-06-12 Thread Marcus Glocker
CVSROOT:/cvs
Module name:src
Changes by: mgloc...@cvs.openbsd.org2017/06/12 12:26:33

Modified files:
sys/arch/armv7/stand/efiboot: exec.c 

Log message:
Backout last change to remove esym .data patching because it breaks
boot for e.g. the BBB and the C.H.I.P.; 'panic: initarm: out of memory'.
Better solution to be found as discussed with patrick@.

ok jsg@



CVS: cvs.openbsd.org: src

2017-06-12 Thread Otto Moerbeek
CVSROOT:/cvs
Module name:src
Changes by: o...@cvs.openbsd.org2017/06/12 10:58:32

Modified files:
regress/usr.bin/sed: Makefile 
Added files:
regress/usr.bin/sed: commandD1.expected commandD1.sed 
 commandc1.expected commandc1.sed 
 commandl1.expected commandl1.sed 
 commandl2.expected commandl2.sed 

Log message:
Add some tests from ksh...@zoho.com. Currently failing, but will be fixed soon.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/06/12 10:39:51

Modified files:
sys/dev: softraid_crypto.c softraidvar.h 

Log message:
Limit the maximum size of softraid crypto volumes that can be created to
the size that is currently supported (a data area of 16TB, or
2^32 * 32 + 528 blocks including the softraid metadata/boot area), rather
than successfully creating the volume and then failing during resource
allocation (without properly reporting why it failed).

Found the hard way by sharon s. 



CVS: cvs.openbsd.org: src

2017-06-12 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/06/12 09:43:25

Modified files:
sys/dev: softraid.c 

Log message:
Upon failure, only return zero from the softraid ioctl handler when there
is at least one bio status message. It is a much nicer user experience when
we properly report failures, however in the case that we fail to do this,
getting an errno back from the ioctl is somewhat better than simply
printing "unknown error" from bioctl(8).



CVS: cvs.openbsd.org: src

2017-06-12 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/06/12 09:15:08

Modified files:
sys/dev: softraid_crypto.c 

Log message:
Fix another long standing softraid crypto bug where if all 32 keys are in
use, when freeing crypto sessions we run straight off the end of the array
and start blatting memory - clearly no one has a softraid crypto volume
that exceeds 15.5TB in size...



CVS: cvs.openbsd.org: src

2017-06-12 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/06/12 09:09:07

Modified files:
sys/dev: softraid_crypto.c 

Log message:
Factor out and reuse the code that frees the crypto sessions, instead of
duplicating it.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2017/06/12 08:46:00

Modified files:
sys/dev: softraid_crypto.c 

Log message:
Fix a long standing off-by-one bug in the softraid crypto number of keys
calculation - we allow one key per 0.5TB, which should allow up to 16TB
disks, however the disk size was treated like a block offset and shifted.
This meant that the maximum size was actually 16TB minus one block.

While here also calculate the number of keys as an absolute value, rather
than as the upper inclusive bound - adjust the logic in the associated for
and if statements to match.

ok krw@ tb@



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 08:23:26

Modified files:
usr.bin/patch  : patch.c 

Log message:
rejname[] is also -r option buffer, and should be PATH_MAX
ok millert



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 07:45:39

Modified files:
sys/dev/wscons : wsmux.c 

Log message:
specify free() size from the old allocation, not new allocation.
fix from C Turt



CVS: cvs.openbsd.org: src

2017-06-12 Thread Theo de Raadt
CVSROOT:/cvs
Module name:src
Changes by: dera...@cvs.openbsd.org 2017/06/12 07:41:24

Modified files:
usr.sbin/vmd   : vmd.h 

Log message:
device path should be PATH_MAX.  Any NAME_MAX without +1 is suspect
to begin with anyways.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Mike Belopuhov
CVSROOT:/cvs
Module name:src
Changes by: mi...@cvs.openbsd.org   2017/06/12 06:35:07

Modified files:
sys/dev/pv : if_xnf.c 

Log message:
ifq_start does OACTIVE and RUNNING checks for the driver

Reminded by dlg@.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2017/06/12 04:57:35

Modified files:
usr.bin/tmux   : screen-write.c 

Log message:
Need to flush out the linefeed after wrapper. GitHub issue 970.



CVS: cvs.openbsd.org: src

2017-06-12 Thread Nicholas Marriott
CVSROOT:/cvs
Module name:src
Changes by: n...@cvs.openbsd.org2017/06/12 01:04:24

Modified files:
usr.bin/tmux   : input-keys.c key-string.c tmux.h tty-keys.c 

Log message:
Add explicit keys for the bracketed paste sequences, both to avoid mix
ups with other keys and to make logs clearer.