[PATCH] etc/ksh.kshrc - unify command substitution

2017-07-06 Thread Raf Czlonka
Hi all, I've noticed that etc/ksh.kshrc uses both types of command substitution `command` and $(command). The below diff unifies it and uses $(command) notation consistently. While there: - remove ':' (null utility) from the very first line of the file - I *do* understand what it does but it d

[PATCH] clarify history of htpasswd(1) in its manpage

2017-07-06 Thread Raf Czlonka
Hi all, Just gone through htpasswd(1)'s man page and noticed this in the AUTHORS section: Florian Obser implemented htpasswd from scratch after httpd was removed from OpenBSD base. To those not familiar with OpenBSD's release history, this may seem wrong (after all, httpd *is*

Re: lock(1): use crypt_checkpass(3) for one-off keys

2017-07-06 Thread Scott Cheloha
> On Jun 26, 2017, at 10:49 PM, Ted Unangst wrote: > > [...] > >> >> CC'd tedu@ because I'm not sure if I'm using crypt_newhash(3) >> correctly. >> >> Ted: In other places people use _PASSWORD_LEN for the length >> of the hash buffer. Clearly this works, but it feels off. >> _PASSWO

Re: Standard conformance of strtol(3)

2017-07-06 Thread Gerhard Roth
On 06.07.2017 17:53, Todd C. Miller wrote: On Thu, 06 Jul 2017 07:37:19 -0600, "Todd C. Miller" wrote: glibc strtol() behavior: AIX FreeBSD GNU/Linux Solaris macOS SunOS 4.1.3 has the same behavior as Solaris. That's as far back as I care to go. - todd FWIW: AT&

Re: add simple ifstated regression test script

2017-07-06 Thread Rob Pierce
On Sun, Jul 02, 2017 at 06:29:07PM +0200, Sebastian Benoit wrote: > Rob Pierce(r...@2keys.ca) on 2017.07.02 12:06:25 -0400: > > I am currently using this regression script for basic ifstated sanity > > testing. > > > > Still a work in progress. Requesting commit for safe keeping. > > > Hi, > >

Re: Standard conformance of strtol(3)

2017-07-06 Thread Marc Espie
On Thu, Jul 06, 2017 at 06:06:09PM +0200, Joerg Sonnenberger wrote: > On Thu, Jul 06, 2017 at 03:42:19PM +0200, Marc Espie wrote: > > 7.20.1.4 (3) If the value of base is zero, the expected form of the subject > > sequence is that of an integer constant *as described in 6.4.4.1*, > > optionally >

Re: Standard conformance of strtol(3)

2017-07-06 Thread Todd C. Miller
I've just committed a fix for this. - todd

Re: Standard conformance of strtol(3)

2017-07-06 Thread Joerg Sonnenberger
On Thu, Jul 06, 2017 at 03:42:19PM +0200, Marc Espie wrote: > 7.20.1.4 (3) If the value of base is zero, the expected form of the subject > sequence is that of an integer constant *as described in 6.4.4.1*, optionally > preceded by a plus or minus sign but not including an integer suffix [...] > >

armv7 bootstrap-only variables

2017-07-06 Thread Artturi Alm
Hi, is/has anyone been working on a diff that would collect&move these into a structure, so that those could easier get gotten rid of once bootstrap is done? Or have i missed something about this new bootstrap split-up to locore/0.S, i mean, is the gap alone good enough to leave these around, or i

Re: Standard conformance of strtol(3)

2017-07-06 Thread Todd C. Miller
On Thu, 06 Jul 2017 07:37:19 -0600, "Todd C. Miller" wrote: > Sorry, HP-UX actually has the same behavior as us. Here's what I > have so far: > > 4.4BSD strtol() behavior: > HP-UX > NetBSD > OpenBSD strtol(3) was added to BSD in 4.3-Reno along with the rest of the Torek libc. > gli

Re: ifstated unused variable

2017-07-06 Thread Lee Clagett
On Sun, 2 Jul 2017 16:31:29 +0200 Sebastian Benoit wrote: > Thanks, i commited all three. > > /Benno > > Rob Pierce(r...@2keys.ca) on 2017.07.02 00:32:27 -0400: > > Remove unused variable from header file. > > > > Index: ifstated.h > > ==

Re: armv7 small bootstrap improvement/simplification

2017-07-06 Thread Artturi Alm
On Wed, Jul 05, 2017 at 09:40:41PM +0300, Artturi Alm wrote: > > diff --git a/sys/arch/armv7/armv7/armv7_machdep.c > b/sys/arch/armv7/armv7/armv7_machdep.c > index aa1c549b29b..105fbf1 100644 > --- a/sys/arch/armv7/armv7/armv7_machdep.c > +++ b/sys/arch/armv7/armv7/armv7_machdep.c > @@ -356,6

Re: mg backup directory (bump)

2017-07-06 Thread Lucas Gabriel Vuotto
Bump. Emacs gets HOME from environment here. I think that getting it from pw entry is more correct, but I can make a patch to behave like emacs if needed. On 19/05/17 14:11, Lucas Gabriel Vuotto wrote: > Previous patch shall be ignored, as it was an ugly hack. Below is a patch > that is simpler

Re: [patch] mg: fix overflow on vteeol()

2017-07-06 Thread Florian Obser
Hi Ingo, thanks for your detailed analysis, makes sense to me OK florian@ Apologies to Hiltjo for slacking on this. I looked at it multiple times but couldn't make any progress on it. Florian On Thu, Jul 06, 2017 at 04:08:09PM +0200, Ingo Schwarze wrote: > Hi, > > Hiltjo Posthuma wrote on Sun,

Re: Standard conformance of strtol(3)

2017-07-06 Thread Bryan Steele
On Thu, Jul 06, 2017 at 07:37:19AM -0600, Todd C. Miller wrote: > Sorry, HP-UX actually has the same behavior as us. Here's what I > have so far: > > 4.4BSD strtol() behavior: > HP-UX > NetBSD > OpenBSD I had discovered with awolk@ and Dragonfly also shares this behaviour: http://gi

Re: [patch] mg: fix overflow on vteeol()

2017-07-06 Thread Ingo Schwarze
Hi, Hiltjo Posthuma wrote on Sun, Jun 18, 2017 at 03:04:31PM +0200: > mg crashes with certain (unicode) characters and moving the cursor to the > end of the line. Even though i failed to reproduce the crash, even with MALLOC_OPTIONS=S, i see how it may happen. Your analysis looks at the right f

Re: Standard conformance of strtol(3)

2017-07-06 Thread Marc Espie
On Wed, Jul 05, 2017 at 07:12:28PM -0400, Ted Unangst wrote: > Olivier Antoine wrote: > > Hi all, > > > > Recently a bug has been identified in Tor: > > > > https://trac.torproject.org/projects/tor/ticket/22789 > > > > As comments were made, questions were raised about the use of strtol(3), > >

Re: Standard conformance of strtol(3)

2017-07-06 Thread Todd C. Miller
Sorry, HP-UX actually has the same behavior as us. Here's what I have so far: 4.4BSD strtol() behavior: HP-UX NetBSD OpenBSD glibc strtol() behavior: AIX FreeBSD GNU/Linux Solaris macOS - todd

Re: Standard conformance of strtol(3)

2017-07-06 Thread Todd C. Miller
Solaris, AIX, and HP-UX all have the same behavior as glibc. We are the outlier. Since the standard is clear that the 0x/0X prefix is optional I believe our behavior is wrong. - todd

Re: patch: fix inteldrm for Intel P4000 graphics

2017-07-06 Thread Joe Gidi
>> Date: Tue, 4 Jul 2017 17:24:27 -0400 >> From: "Joe Gidi" >> >> I have a machine with a Xeon E3-1225v2 CPU, which includes integrated >> Intel P4000 graphics. This required a patch back in 2015 to avoid >> matching >> on the mythical "Intel Quanta Transcode" device, which kettenis@ >> committed

Re: dhcpd: don't reject DHCPINFORM from behind relay

2017-07-06 Thread Landry Breuil
On Wed, Jul 05, 2017 at 04:37:39PM +0200, Reyk Floeter wrote: > Hi, > > landry@ sees many log messages 'DHCPINFORM from xx but ciaddr yy is > not consistent with actual address' in a setup where dhcpd runs behind > dhcrelay. > > The code in dhcpd's dhcpinform() seems wrong - it assumes that ciadd