remove errant ifstated whitespace

2017-06-27 Thread Rob Pierce
Index: ifstated.c === RCS file: /cvs/src/usr.sbin/ifstated/ifstated.c,v retrieving revision 1.43 diff -u -p -r1.43 ifstated.c --- ifstated.c 27 Jun 2017 20:46:34 - 1.43 +++ ifstated.c 28 Jun 2017 01:30:02 - @@ -151,12

Re: Attempt to simplify file(1)

2017-06-27 Thread Bryan Steele
On Tue, Jun 27, 2017 at 11:40:49PM +0100, Nicholas Marriott wrote: > Thanks. Comments inline. > > > [..] > > One member per line in structs please. Also you could reduce the amount > of space here now to one tab. > > > [..] > > magic_load (which parses the magic file) is now before pledge and >

Re: Attempt to simplify file(1)

2017-06-27 Thread Nicholas Marriott
Thanks. Comments inline. On Tue, Jun 27, 2017 at 06:28:57AM -0400, Bryan Steele wrote: > On Tue, Jun 27, 2017 at 01:20:59AM -0400, Bryan Steele wrote: > > On Tue, Jun 27, 2017 at 12:26:08AM -0400, Bryan Steele wrote: > > Some unintentional changes crept in, here's another diff.. > > Sorry, last

Re: Attempt to simplify file(1)

2017-06-27 Thread Theo de Raadt
> OpenBSD's file(1) implementation was written by nicm@, first introduced > in 5.8, the inital design included a privileged parent process which > forked an unprivileged child which would handle potentially unsafe > file parsing. > > It also had 'sandboxing' using systrace(4), which required

Re: Attempt to simplify file(1)

2017-06-27 Thread Ted Unangst
Bryan Steele wrote: > OpenBSD's file(1) implementation was written by nicm@, first introduced > in 5.8, the inital design included a privileged parent process which > forked an unprivileged child which would handle potentially unsafe > file parsing. > > It also had 'sandboxing' using systrace(4),

Re: no depends for perl

2017-06-27 Thread Andrew Fresh
On Tue, Jun 27, 2017 at 02:13:12PM -0600, Todd C. Miller wrote: > On Mon, 26 Jun 2017 19:49:18 +0200, Marc Espie wrote: > > > Probably the most intricate yet. > > > > This is not entirely new, since I had to split manpages already a long > > time ago. The trick is to keep the "Configure stuff"

Re: [PATCH] bin: Add -v option to cp(1), mv(1), rm(1)

2017-06-27 Thread Sebastian Benoit
3 lines are longer than 80 chars. otherwise ok benno@ Job Snijders(j...@instituut.net) on 2017.06.27 11:00:55 +0200: > Dear all, > > This patch adds a '-v' option to cp(1), mv(1), and rm(1). If the '-v' > option is used, the utility will display the name of each file after it > has been

Re: ifstated.c hoist code in prep for future work

2017-06-27 Thread Sebastian Benoit
commited with the (int)(long)arg cast. Thanks! (waiting for more...) /Benno Rob Pierce(r...@2keys.ca) on 2017.06.27 12:45:43 -0400: > Hoist some privileged code in preparation for future work. > > Is this the correct use of intptr_t? > > Based on an approach in vmd with mc146818/ns8250. > >

Re: no depends for perl

2017-06-27 Thread Todd C. Miller
On Mon, 26 Jun 2017 19:49:18 +0200, Marc Espie wrote: > Probably the most intricate yet. > > This is not entirely new, since I had to split manpages already a long > time ago. The trick is to keep the "Configure stuff" in the main > Makefile.bsd-wrapper, and put everything else in

OpenBSD Errata: June 27th, 2017 (icmp6_linklocal)

2017-06-27 Thread Alexander Bluhm
An errata patch for icmp6_linklocal has been released for OpenBSD 6.1. When pinging an IPv6 link-local address, the reflected packet had ::1 as source address. The echo reply was ignored as it must be from the link-local address. Binary updates for the amd64 and i386 platforms are available via

ifstated.c hoist code in prep for future work

2017-06-27 Thread Rob Pierce
Hoist some privileged code in preparation for future work. Is this the correct use of intptr_t? Based on an approach in vmd with mc146818/ns8250. No intended functional change. Rob Index: ifstated.c === RCS file:

minor bgpd.c diff

2017-06-27 Thread Rob Pierce
rfd does not need to be global. Rob Index: bgpd.c === RCS file: /cvs/src/usr.sbin/bgpd/bgpd.c,v retrieving revision 1.189 diff -u -p -r1.189 bgpd.c --- bgpd.c 28 May 2017 15:16:33 - 1.189 +++ bgpd.c 27 Jun 2017

Re: IP in IP header chain input

2017-06-27 Thread Alexander Bluhm
On Tue, May 23, 2017 at 01:36:00PM +0200, Alexander Bluhm wrote: > The IP in IP input function strips the outer header and reinserts > the inner IP packet into the internet queue. The IPv6 local delivery > code has a loop to deal with header chains. My idea is to use this > loop and avoid the

Additional media options for ix(4)

2017-06-27 Thread Mike Belopuhov
Hi, I won't mind some broad testing of the following diff which adds some additional media options to ix(4) from FreeBSD and includes a fix for changing media from Masanobu SAITOH. The fix makes sure that when the media operation speed is selected manually, the device doesn't additionally

unbound 1.6.4

2017-06-27 Thread Stuart Henderson
If anyone's interested in newer unbound, I have an initial diff, it's 750K all-in (we are currently on 1.6.1 so there are a few releases worth of changes) so I won't send it inline, it's at https://junkpile.org/unbound-1.6.4.diff

clang integrated asm crash while building mozilla/firefox

2017-06-27 Thread Marc Espie
I've attached the files that clang gives to reproduce the crash. I also got a similar crash in firefox-esr. Funnily enough, it happened after the recent update to both, BUT the crashes seem to be unrelated ? The sysv crash is consistent, I got it during both builds. # 1 "" # 1

Re: Attempt to simplify file(1)

2017-06-27 Thread Bryan Steele
On Tue, Jun 27, 2017 at 01:20:59AM -0400, Bryan Steele wrote: > On Tue, Jun 27, 2017 at 12:26:08AM -0400, Bryan Steele wrote: > Some unintentional changes crept in, here's another diff.. Sorry, last diff broke width calculation.. 3rd times the charm. Index: Makefile

Re: sed(1): missing NUL in pattern space

2017-06-27 Thread Otto Moerbeek
On Thu, Jun 15, 2017 at 09:01:15AM +0200, Otto Moerbeek wrote: at last a followup, for the original problem. This diff incorporates your later comment. It does not cause the newly added regress test to fail, though. So that poses the question if this is what you meant. -Otto Index:

[PATCH] bin: Add -v option to cp(1), mv(1), rm(1)

2017-06-27 Thread Job Snijders
Dear all, This patch adds a '-v' option to cp(1), mv(1), and rm(1). If the '-v' option is used, the utility will display the name of each file after it has been successfully copied, moved or removed. This patch rolls in feedback from recent discussion. The manpage updates are specific as to what

Re: kernel relinking at install/upgrade time

2017-06-27 Thread Theo de Raadt
> On Mon 26/06/2017 14:35, Theo de Raadt wrote: > > There is a diff in snapshots which does kernel relinking during > > install or upgrade. > > > > Really amazing... > > Works as advertised, assuming that compXX is installed. The relinking bit uses > strip, which is part of the compiler

Re: kernel relinking at install/upgrade time

2017-06-27 Thread Leighton Sheppard
On Mon, Jun 26, 2017 at 02:35:55PM -0600, Theo de Raadt wrote: > There is a diff in snapshots which does kernel relinking during > install or upgrade. > > Really amazing... > This does sound amazing! Upgrade time

Re: kernel relinking at install/upgrade time

2017-06-27 Thread Björn Ketelaars
On Mon 26/06/2017 14:35, Theo de Raadt wrote: > There is a diff in snapshots which does kernel relinking during > install or upgrade. > > Really amazing... Works as advertised, assuming that compXX is installed. The relinking bit uses strip, which is part of the compiler collection. Does it