Re: systemd compat for doas

> On 2. Jul 2017, at 8:59 PM, Ted Unangst wrote: > > If the username starts with a digit, but isn't a number, treat it like root. I question the simplicity of this patch due to the fact that it leaves no head room for further security-related regressions. Maybe more progressive over-engineerin

Re: dont use the malloc ks_limit in softdeps

hrm. ignore this. On Mon, Jul 03, 2017 at 04:30:11PM +1000, David Gwynne wrote: > the only thing using ks_limit is the ffs softdep code. > > this cleans that up. > > ok? > > Index: ffs/ffs_softdep.c > === > RCS file: /cvs/src/sys/u

dont use the malloc ks_limit in softdeps

the only thing using ks_limit is the ffs softdep code. this cleans that up. ok? Index: ffs/ffs_softdep.c === RCS file: /cvs/src/sys/ufs/ffs/ffs_softdep.c,v retrieving revision 1.135 diff -u -p -r1.135 ffs_softdep.c --- ffs/ffs_softd

ifstated readability diff

Remove obvious clear_config() comments and misleading state_change() comments. Also relocate do_action() calls for the init block from change_state() to occur with the corresponding do_action() calls for the body block within the calling function for improved readability. No functional change. R

rebound listen address

I received a similar diff from Angelos a while back, though I turned it down on the grounds that rebound wasn't ready to be a recursive resolver. But I think we're in better shape now. This adds a -l address option to specify the listening address, turning rebound into a lightweight partial replac

armv7 a4x dsb fix

Hi, just the bug fix, so this diff leaves the unused relics around and so, but better than current/nothing, and hopefully small enough to get looked at. -Artturi diff --git a/sys/arch/arm/armv7/armv7_a4x_io.S b/sys/arch/arm/armv7/armv7_a4x_io.S index 3fa09faa4c2..5e9db421853 100644 --- a/sys/a

Re: arm/armv7 KNF - sad shit.

On Mon, Jul 03, 2017 at 01:30:09AM +0300, Artturi Alm wrote: > Hi, > > would anyone be willing to accept diffs to recover from this? I mean > the overlooking of this from style(9), that's making my eyes bleed: > "There should be no space between the function name and > the argument

Re: install.sub: ieee80211_{scan,config}: Allow quoted SSIDs

Thanks for looking into it. On Sun, Jul 02, 2017 at 04:32:43PM +, Robert Peichaer wrote: ieee80211_scan() - Extract the needed information (nwid, bssid) using a very specific sed expression. Any line, not matching this expr is ignored. - Remove leading and trailing double-quotes in cas

arm/armv7 KNF - sad shit.

Hi, would anyone be willing to accept diffs to recover from this? I mean the overlooking of this from style(9), that's making my eyes bleed: "There should be no space between the function name and the argument list." ^about prototypes. tbh., i wanted to punch someone in the fa

Re: systemd compat for doas

Where would this be needed and make sense? I wasn't aware of this systemd behaviour until now, but a quick google search shows me that it's seen by many as a bug and I tend to agree. A type-O shouldn't imply root access. martijn@ On 07/02/17 20:59, Ted Unangst wrote: > If the username starts

lock(1): report remaining time with getitimer(2)

Hi, There's an edge case in lock(1) where, after you've reserved a terminal for a finite time, if the system time is changed you get an inaccurate report of the remaining time left on the reservation. This is because we subtract the result of time(3) at the beginning of the program from the resul

Re: file: replace fgetln with getline(3)

Brilliant, looks good to me, ok nicm On Sun, Jul 02, 2017 at 06:23:35PM +0200, Ingo Schwarze wrote: > Hi Nic, > > Nicholas Marriott wrote on Sun, Jul 02, 2017 at 08:48:28AM +0100: > > > Possibly fgetln(3) CAVEATS could benefit from some of what you have > > written here, although it is all in

systemd compat for doas

If the username starts with a digit, but isn't a number, treat it like root. Index: doas.c === RCS file: /cvs/src/usr.bin/doas/doas.c,v retrieving revision 1.72 diff -u -p -r1.72 doas.c --- doas.c 27 May 2017 09:51:07 -

armv7 bus space fixes

Hi, diff to get rid of bus_space_asm_generic.S, changing a4x bus space to use armv7 bs, which might be considered a bug fix, since generic_ misses the v7 "dsb sy"s, and does add the six missing ones to a4x. unfortunately some less useless bus_dma_tags&whitespace fixes snuck in. -Artturi diff -

Re: [patch] Remove binc from vi(1)

Any takers? On 06/22/17 21:32, Martijn van Duren wrote: > Hello tech@, > > Attached a patch to remove the binc function from vi and replace it with > recallocarray. The functions effectively do the same thing since > BINC_{GOTO,RET} already do the nlen > llen comparison. I've run > this without a

Re: add simple ifstated regression test script

> From: "Sebastian Benoit" > To: "Rob Pierce" > Cc: "tech" > Sent: Sunday, July 2, 2017 12:29:07 PM > Subject: Re: add simple ifstated regression test script > Rob Pierce(r...@2keys.ca) on 2017.07.02 12:06:25 -0400: > > I am currently using this regression script for basic ifstated sanity > >

Re: install.sub: ieee80211_{scan,config}: Allow quoted SSIDs

On Thu, Jun 15, 2017 at 12:09:20AM +0200, Klemens Nanni wrote: > Instead of ignoring SSIDs containing whitespaces, slightly adjust the > commands to take everything in between 'nwid ' and ' chan' as SSID; if > it has double quotes at start *and* end, simply remove those. > > This enables users to

Re: add simple ifstated regression test script

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, this should go into /usr/src/regress/usr.sbin/ifstated (which does not esist yet). Also

Re: file: replace fgetln with getline(3)

Hi Nic, Nicholas Marriott wrote on Sun, Jul 02, 2017 at 08:48:28AM +0100: > Possibly fgetln(3) CAVEATS could benefit from some of what you have > written here, although it is all in the manual already just in several > places. Inspecting the manual page again, i don't think it needs longer text.

add simple ifstated regression test script

I am currently using this regression script for basic ifstated sanity testing. Still a work in progress. Requesting commit for safe keeping. Regards, Rob Index: usr.sbin/ifstated/ifstated.sh === RCS file: usr.sbin/ifstated/ifstated

ifstated ifsd_config variable name change

Second diff to rename additional "always" variable in struct ifsd_config. Requested by stsp@. Rob Index: ifstated.c === RCS file: /cvs/src/usr.sbin/ifstated/ifstated.c,v retrieving revision 1.47 diff -u -p -r1.47 ifstated.c --- ifst

Re: ifstated unused variable

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 > === > RCS file: /cvs/src/usr.sbin/ifstated/ifstated.h,v > retrieving rev

Re: rename variable in ifstated

On Sun, Jul 02, 2017 at 02:56:06PM +0200, Stefan Sperling wrote: > On Sat, Jul 01, 2017 at 05:53:54PM -0400, Rob Pierce wrote: > > never.never say always.always. > > > > Rename one of the "always" variables to "body" for improved readability. > > > > No functional change. > > > > >From ifstated.

Re: ifstated unused variable

On Sun, Jul 02, 2017 at 12:32:27AM -0400, Rob Pierce wrote: > Remove unused variable from header file. ok by me. > > Index: ifstated.h > === > RCS file: /cvs/src/usr.sbin/ifstated/ifstated.h,v > retrieving revision 1.12 > diff -u -p

Re: ifstated whitespace var assignment diff

On Sat, Jul 01, 2017 at 11:59:59PM -0400, Rob Pierce wrote: > Remove variable assignment in declaration and add whitespace to improve > readability. No functional change. > > Rob Fine with me. I think either style is fine, though. > > Index: ifstated.c >

Re: rename variable in ifstated

On Sat, Jul 01, 2017 at 05:53:54PM -0400, Rob Pierce wrote: > never.never say always.always. > > Rename one of the "always" variables to "body" for improved readability. > > No functional change. > > >From ifstated.conf(5): > > "Each state consistes of an init block and a body. The init block i

Re: netstat(1) print PID for sockets.

On Sat, Jul 01, 2017 at 08:50:36PM +0200, Sebastian Benoit wrote: > Sebastian Benoit(be...@openbsd.org) on 2017.07.01 20:40:17 +0200: > > oks for this? > > /* filter listening sockets out unless -a is set */ > - if (!aflag && istcp && kf->t_state <= TCPS_LISTEN) > + if (!(aflag || l

Re: vi(1): remove stub settings

On Fri, Jun 30, 2017 at 09:19:31AM -0600, Anthony J. Bentley wrote: > > Here's a diff that updates the STANDARDS section in the manual. > I think it will be enough to talk about our POSIX compliance. POSIX > discusses its differences from historical ex/vi in great detail, > including an explicit r

Re: file: replace fgetln with getline(3)

Possibly fgetln(3) CAVEATS could benefit from some of what you have written here, although it is all in the manual already just in several places. Or at least a link to getline(3)? On Sun, Jul 02, 2017 at 12:26:19AM +0200, Ingo Schwarze wrote: > Hi, > > Nicholas Marriott wrote on Sat, Jul 01,

Re: file: replace fgetln with getline(3)

I'm not wild about magic_load calling err(), but it is probably more pain to do nicely than it is worth it for now. This is fine, ok nicm On Sun, Jul 02, 2017 at 01:03:13AM -0400, Bryan Steele wrote: > On Sat, Jul 01, 2017 at 08:48:18PM -0400, Bryan Steele wrote: > > On Sat, Jul 01, 2017 at 0

Re: file: replace fgetln with getline(3)

Right, I wasn't clear - I didn't mean "why is getline better than fgetln", I was asking "why do you want to change fgetln to getline in this code". It is fine if the answer is "I am worried that another developer might copy or modify the code and mess it up" rather than "the way file is using fget