Re: ld.so diff that needs testing on landisk

2017-01-04 Thread Philip Guenther
On Tue, Jan 3, 2017 at 2:13 PM, Mark Kettenis wrote: > The diff below (partly by guenther@) removes ld.so's dependency on the > __got_{start,end} symbols by looking at PT_GNU_RELRO instead. On some > platforms (hppa and perhaps a few others) this leads to even less > writable pages. However, we'

Re: Recursive NET_LOCK()

2017-01-04 Thread Alexander Bluhm
On Tue, Jan 03, 2017 at 08:21:58PM +0100, Martin Pieuchot wrote: > I am also looking for more feedbacks and/or inputs so I appreciate your > email on the matter. Together with a coworker Zaur Molotnikov we are using static code analysis to find places where a netlock is needed. We try to create p

Re: Unable to boot since latest installer HTTPS changes

2017-01-04 Thread Leo Unglaub
Hey, On 01/03/17 17:26, Leo Unglaub wrote: Hey tech@ i wanted to upgrade to the latest snapshot (as i always do, every week) but since you added the HTTPS support to the installer i am unable to boot my system or the install.fs from a flash drive. I get to the MBR and then into the PBR but afte

less(1): fix some memory leaks

2017-01-04 Thread Julien Ramseier
This does not fix all "leaks" that may be detected. In particular during program initialization copies of a few strings and such are made and those copies may persist. However, those are benign as the program does not continuously leak memory. Furthermore, fixing all of them would be substantial

Re: llvm: pass library paths for cross-build

2017-01-04 Thread Mark Kettenis
> Date: Wed, 4 Jan 2017 17:46:23 +0100 > From: Patrick Wildt > > Hi, > > with GCC as cross-compiler the PREFIX define (which is set to the > CROSSDIR) makes sure that the linker includes the correct dirs. > Otherwise the cross-build will only find the host libraries, which > does not really work

llvm: pass library paths for cross-build

2017-01-04 Thread Patrick Wildt
Hi, with GCC as cross-compiler the PREFIX define (which is set to the CROSSDIR) makes sure that the linker includes the correct dirs. Otherwise the cross-build will only find the host libraries, which does not really work well. Our clang doesn't do this so far. The changes in clang make sure tha

Re: libcrypto: another ARM assembly change

2017-01-04 Thread Mark Kettenis
> Date: Wed, 4 Jan 2017 16:56:08 +0100 > From: Patrick Wildt > > Hi, > > with our base clang targeting armv7 by default, this enables the NEON > assembly in libcrypto as well. Note that is only used if NEON is > actually available. > > Clang doesn't like the extra size definition ".32" on vmov

libcrypto: another ARM assembly change

2017-01-04 Thread Patrick Wildt
Hi, with our base clang targeting armv7 by default, this enables the NEON assembly in libcrypto as well. Note that is only used if NEON is actually available. Clang doesn't like the extra size definition ".32" on vmov which is not needed, so simply remove it. ok? Patrick diff --git a/lib/libc

Re: sort(1): fix segfault with -m flag

2017-01-04 Thread Todd C. Miller
On Wed, 04 Jan 2017 12:24:15 +0100, Julien Ramseier wrote: > sort segfaults when using the -m flag and no files. > It should default to stdin instead. Committed, thanks. - todd

Re: less(1): carefully handle null in strchr arguments

2017-01-04 Thread Theo Buehler
On Wed, Jan 04, 2017 at 03:55:20PM +0100, Julien Ramseier wrote: > The environment variable LESSBINFMT is not properly validated. If it is > set to "*", less will perform an out of boundary access. > > This happens because strchr can be called with '\0' as second argument. > Such a call won't retu

less(1): carefully handle null in strchr arguments

2017-01-04 Thread Julien Ramseier
The environment variable LESSBINFMT is not properly validated. If it is set to "*", less will perform an out of boundary access. This happens because strchr can be called with '\0' as second argument. Such a call won't return NULL but the address of the '\0' in the string. Therefore, the checkfmt

Re: libcrypto: fix assembly for clang

2017-01-04 Thread Brent Cook
On Wed, Jan 4, 2017 at 5:12 AM, Mark Kettenis wrote: > > Date: Wed, 4 Jan 2017 11:04:43 +0100 > > From: Patrick Wildt > > > > On Tue, Jan 03, 2017 at 02:16:25PM -0500, Ted Unangst wrote: > > > Patrick Wildt wrote: > > > > > > > > Maybe they want to support older binutils that do not support the

xenocara fontconfig: make slight hinting the default

2017-01-04 Thread Nils Reuße
Dear all, fontconfig made slight hinting the default in version 2.11.95 (see commit at [1]). xenocara currently ships, but does not install the new hinting conf files: $ cd /usr/xenocara/dist/fontconfig/conf.d/ $ for file in *conf; do if ! test -f /etc/fonts/conf.avail/$file; then echo "$f

sort(1): fix segfault with -m flag

2017-01-04 Thread Julien Ramseier
sort segfaults when using the -m flag and no files. It should default to stdin instead. Easily reproducible with a simple: sort -m Index: sort.c === RCS file: /cvs/src/usr.bin/sort/sort.c,v retrieving revision 1.86 diff -u -r1.86 sor

Re: libcrypto: fix assembly for clang

2017-01-04 Thread Mark Kettenis
> Date: Wed, 4 Jan 2017 11:04:43 +0100 > From: Patrick Wildt > > On Tue, Jan 03, 2017 at 02:16:25PM -0500, Ted Unangst wrote: > > Patrick Wildt wrote: > > > > > > Maybe they want to support older binutils that do not support the unified > > > syntax? > > > What's our policy there? > > > > Open

Re: libcrypto: fix assembly for clang

2017-01-04 Thread Patrick Wildt
On Tue, Jan 03, 2017 at 02:16:25PM -0500, Ted Unangst wrote: > Patrick Wildt wrote: > > > > Maybe they want to support older binutils that do not support the unified > > syntax? > > What's our policy there? > > OpenBSD is just about the oldest binutils around. I wouldn't worry about > anything o

Re: [patch] fix typo in pf.conf(5)

2017-01-04 Thread Theo Buehler
On Wed, Jan 04, 2017 at 09:52:46AM +, Dimitris Papastamos wrote: > Hi, > > Typo fix, below. committed, thanks! > > diff --git a/share/man/man5/pf.conf.5 b/share/man/man5/pf.conf.5 > index 3559803..955539b 100644 > --- a/share/man/man5/pf.conf.5 > +++ b/share/man/man5/pf.conf.5 > @@ -368,7 +

[patch] fix typo in pf.conf(5)

2017-01-04 Thread Dimitris Papastamos
Hi, Typo fix, below. diff --git a/share/man/man5/pf.conf.5 b/share/man/man5/pf.conf.5 index 3559803..955539b 100644 --- a/share/man/man5/pf.conf.5 +++ b/share/man/man5/pf.conf.5 @@ -368,7 +368,7 @@ Translates to the point-to-point interface's peer address(es). .Pp Host names may also have the

ftp(1): anonymous login and .netrc

2017-01-04 Thread Anton Lindqvist
I'm running a script as root which drops privileges while fetching files using ftp(1) with anonymous login enabled: # doas -u unpriv ftp -a URL ... and was surprised to see the following error: ftp: /root/.netrc: Permission denied I'm not sure if the .netrc file should be considered when anonym