Re: Makefile.cross typo

2019-05-21 Thread Patrick Wildt
On Wed, May 22, 2019 at 02:38:04AM +0200, Krystian Lewandowski wrote: > I tried to rebuild cross tools recently and it seems there is a typo > in Makefile.cross. Thanks, committed!

Re: use getpwuid_r in doas

2019-05-21 Thread Lawrence Teo
On Tue, May 21, 2019 at 01:07:10PM -0400, Ted Unangst wrote: > I have a coming change which will need to access both the calling user and > target users' passwd entries. In order to accomplish this, we need to switch > to the reentrant flavor of getpwuid. No behaviorial change, but I think this >

Makefile.cross typo

2019-05-21 Thread Krystian Lewandowski
I tried to rebuild cross tools recently and it seems there is a typo in Makefile.cross. *** Parse error in src: Malformed conditional ((${TARGET} == "macppc") (Makefile.cross:20) -- Krystian Index: Makefile.cross === RCS file:

Re: [PATCH] Add swich to enable/disable re-link the kernel at startup

2019-05-21 Thread Paul de Weerd
Hi Stefan, On Tue, May 21, 2019 at 09:10:51PM +0200, Stefan Sperling wrote: | > Reminds me that I should find some bigger CF to stick in there; 2GB | > really doesn't cut it anymore these days. | | My alix survives reorder_kernel just fine with 2GB of swap space, | though 1GB would probably be

Re: [PATCH] Add swich to enable/disable re-link the kernel at startup

2019-05-21 Thread Stefan Sperling
On Mon, May 20, 2019 at 01:25:17PM +0200, Paul de Weerd wrote: > Hi Stuart, > > On Mon, May 20, 2019 at 12:14:56PM +0100, Stuart Henderson wrote: > | On 2019/05/20 10:46, Paul de Weerd wrote: > | > However, library_aslr delays the boot sequence (because libraries must > | > be completed before

pci_reserve_resources(9) fix

2019-05-21 Thread Mark Kettenis
Logic error. If pa->pa_pmemex is NULL we want to try allocating from pa->pa_memex. But if both of them are NULL we need to assume that the provided base address is correct. ok? Index: dev/pci/pci.c === RCS file:

rtcdrain: fix function reference in comment

2019-05-21 Thread Jasper Lievisse Adriaanse
Hi, This appears to have been wrong from the import, the comment explaining why rtcdrain() is needed is in rtcstart() in the same file. OK? Index: amd64/isa/clock.c === RCS file: /cvs/src/sys/arch/amd64/isa/clock.c,v retrieving

use getpwuid_r in doas

2019-05-21 Thread Ted Unangst
I have a coming change which will need to access both the calling user and target users' passwd entries. In order to accomplish this, we need to switch to the reentrant flavor of getpwuid. No behaviorial change, but I think this is clearer and less error prone as well, versus reusing a pointer to

bgpd: better tcp md5sig handling

2019-05-21 Thread Claudio Jeker
Hi, The last refactor of the pfkey handling uncovered an issue deep down in the pfkey handling. Some long time ago henning@ added a sleep(1) into pfkey_md5sig_establish(). This is now hunting us back because bgpd ends up with no tcp md5sig flow for 1 sec while a connection is established. Because

Re: [PATCH] Add swich to enable/disable re-link the kernel at startup

2019-05-21 Thread Ricardo Fraile
Hi Paul, El 2019-05-20 10:46, Paul de Weerd escribió: | However, library_aslr delays the boot sequence (because libraries must | be completed before the system can boot up) while kernel_aslr does not | - it only introduces a bit of extra load on your machine. What | problem are you trying to

Re: [patch] ex/vi(1): wait with pledge until *after* ~/.nexrc is read

2019-05-21 Thread Martijn van Duren
On 5/21/19 9:10 AM, Jesper Wallin wrote: > On Tue, May 21, 2019 at 07:34:05AM +0200, Martijn van Duren wrote: >> Hello Jesper, >> >> The behaviour should be identical, the only difference would be that >> pledge catches programming errors. So I see no particular reason to use >> -S over "set

Re: mdoc.7: fix Dd example

2019-05-21 Thread Ingo Schwarze
Hi Klemens, Jason McIntyre wrote on Sun, May 19, 2019 at 08:58:35PM +0100: > On Sun, May 19, 2019 at 09:52:39PM +0200, Klemens Nanni wrote: >> On Sun, May 19, 2019 at 07:31:19PM +0100, Jason McIntyre wrote: >>> no. just use $Mdocdate$ and it gets expanded on commit. just like >>> when you add

Re: [patch] ex/vi(1): wait with pledge until *after* ~/.nexrc is read

2019-05-21 Thread Bryan Steele
On Tue, May 21, 2019 at 07:34:05AM +0200, Martijn van Duren wrote: > Hello Jesper, > On 5/20/19 10:58 PM, Jesper Wallin wrote: > > Hi all, > > > > When ex/vi is started with -S (secure), a stricter pledge is used to > > prevent exec from being used. It's tedious to specify -S all the time > >

Re: grep ".one\|.two" doesn't work on OpenBSD. Is it expected?

2019-05-21 Thread Ingo Schwarze
Hi, Todd C. Miller wrote on Mon, May 20, 2019 at 01:22:21PM -0600: > On Mon, 20 May 2019 20:01:12 +0200, Juan Francisco Cantero Hurtado wrote: >> The grep command works with GNU, NetBSD, FreeBSD and BusyBox. It fails >> on OpenBSD and Solaris 11. I'm suggesting upstream to change the command >>

Re: [patch] ex/vi(1): wait with pledge until *after* ~/.nexrc is read

2019-05-21 Thread Jesper Wallin
On Tue, May 21, 2019 at 07:34:05AM +0200, Martijn van Duren wrote: > Hello Jesper, > > The behaviour should be identical, the only difference would be that > pledge catches programming errors. So I see no particular reason to use > -S over "set secure" for normal users; even without pledge.