Re: [patch] httpd: don't add date header if already set

2017-07-29 Thread Florian Obser
OK florian@ p.s. whoever commits this please add extra ( ) around the && term, Apparently clang no longer supports operator precedence *sigh*. I'm wondering if it also warns for a * b + c /usr/src/usr.sbin/httpd/server_fcgi.c:665:56: warning: '&&' within '||' [-Wlogical-op-parentheses]

Re: [patch] httpd: don't add date header if already set

2017-07-29 Thread Nick Owens
ping? On Jul 18, 2017 19:01, "Nick Owens" wrote: hello tech@, here is a diff that will cause httpd's fcgi code to not set the HTTP date header if it has already been set. the code i am using for an fcgi server (https://github.com/golang/go/blob/master/src/net/http/fcgi/child.go#L102) unconditio

Re: nm in free(): chunk canary corrupted 0x3e6e6845580 0x19@0x19

2017-07-29 Thread Matthew Martin
On Sat, Jul 29, 2017 at 10:43:23AM +0100, Stuart Henderson wrote: > I was just running nm over all of /usr/lib on a system with C in malloc > flags and ran into this. > > $ MALLOC_OPTIONS=C nm -s libc.so.89.3.a The bug seems to be in mmbr_name. First add one to len so there's space for the null w

Re: efiboot boot path matching

2017-07-29 Thread Jonathan Gray
On Sat, Jul 29, 2017 at 09:01:35PM +0200, Patrick Wildt wrote: > On Sat, Jul 29, 2017 at 02:59:19PM +0200, Mark Kettenis wrote: > > This is apparently very hard. Caught this on arm64 where > > efi_device_path_depth() returned 0, which resulted in always selecting > > the first device. Clearly if

[diff] typo in tls_load_file.3

2017-07-29 Thread Carlos Cardenas
Missing 'ocsp' in the function name. +--+ Carlos diff --git lib/libtls/man/tls_load_file.3 lib/libtls/man/tls_load_file.3 index fcaa5eef029..b83f55e0fe4 100644 --- lib/libtls/man/tls_load_file.3 +++ lib/libtls/man/tls_load_file.3 @@ -254,7 +254,7 @@ sets the files from which the public certificat

[PATCH] run security(8) on first boot

2017-07-29 Thread Joe Gidi
I did a couple of fresh installs the other day, which reminded me of a minor irritation and prompted me to think about a possible solution. The first run of security(8) on a fresh install is not terribly helpful. It produces a huge email report since it diffs all the /etc/changelist files against

Re: less(1) - segmentation fault with '-g'

2017-07-29 Thread Larry Hynes
Theo Buehler wrote: > On Thu, Jul 27, 2017 at 10:31:51AM +0100, Larry Hynes wrote: > > Hi > > > > $ env | grep LESS > > LESSHISTFILE=- > > LESS="-i -M -R -g -c" > > LESSCHARSET=utf-8 > > > > $ unset LESS > > $ unset LESSCHARSET > > $ unset LESSHISTFILE > > > > $ LESS="-g" > > $ echo 'foo\nbar\n

mg: print default choice before colon in echo line prompts

2017-07-29 Thread Scott Cheloha
Hi, GNU Emacs always, or very nearly always, prints the default choice before the colon in any given prompt. For example, you'll see "Kill buffer (default foo): " not "Kill buffer: (default foo) " This patch moves all default choices before the colon in all of mg's prompts. W

Re: sysctl machdep.lidaction=suspend

2017-07-29 Thread Jason McIntyre
On Sat, Jul 29, 2017 at 10:43:55PM +0200, Ingo Schwarze wrote: > > All the same, i'd like to point out that there is a minor documentation > issue. Some time ago, we decided that having duplicate descriptions > of each and every sysctl variable in sysctl(3) and sysctl(8) is bad > for maintainabil

Re: sysctl machdep.lidaction=suspend

2017-07-29 Thread Ingo Schwarze
Hi Martin, Martin Natano wrote on Sat, Jul 29, 2017 at 08:45:05PM +0200: > On Sat, Jul 29, 2017 at 02:19:50PM +0200, Martin Natano wrote: >> On Sat, Jul 29, 2017 at 02:03:22PM +0200, Mark Kettenis wrote: >>> I don't think we want to add string parsing like this in the kernel. >>> Maybe the sysctl

[PATCH] United sys/ntfs/ntfs.h and sys/ntfs/ntfsmount.h

2017-07-29 Thread Андрей Болконский
file ntfsmount.h contains TWO defines! I moved this to ntfs.h file and remove ntfsmount.h for optinize headers. Amd64 build is ok. Index: ntfs/ntfs.h === RCS file: /cvs/src/sys/ntfs/ntfs.h,v retrieving revision 1.17 diff -u -p -u -p -

Re: Is someone interested in resuming support for socppc?

2017-07-29 Thread Denis Fondras
On Sat, Jul 29, 2017 at 01:33:03PM +0300, Андрей Болконский wrote: > https://www.openbsd.org/socppc.html > > > The OpenBSD/socppc port was discontinued after the 5.8 release. > Otherwise, will you approve removing support for socppc from src? I am still using a RB600A.

Re: efiboot boot path matching

2017-07-29 Thread Patrick Wildt
On Sat, Jul 29, 2017 at 02:59:19PM +0200, Mark Kettenis wrote: > This is apparently very hard. Caught this on arm64 where > efi_device_path_depth() returned 0, which resulted in always selecting > the first device. Clearly if the first path component (i = 0) matches > the desired type, we should

Re: [diff] httpd: tls client cert & CRL checks

2017-07-29 Thread Jan Klemkow
Hi Jack, On Fri, Jul 28, 2017 at 02:05:34AM +0930, Jack Burton wrote: > On Thu, 27 Jul 2017 13:10:14 +0200 > > > But, I found a bug in the part of the FastCGI variables. The > > following condition is always false. > > > > > Index: usr.sbin/httpd/server_fcgi.c > > >

Re: sysctl machdep.lidaction=suspend

2017-07-29 Thread Martin Natano
On Sat, Jul 29, 2017 at 02:19:50PM +0200, Martin Natano wrote: > On Sat, Jul 29, 2017 at 02:03:22PM +0200, Mark Kettenis wrote: > > > > I don't think we want to add string parsing like this in the kernel. > > Maybe the sysctl(8) frontend should do the mapping from strings to > > numbers? > > Ok,

make pkg_info -Q work with other flags

2017-07-29 Thread Aaron Bieber
Hola, Currently "pkg_info -Q" doesn't respect other flags and the way pkg_info(1) reads, it implies that they will work with it. This diff makes pkg_info function as expected when other flags are passed when using -Q. Cheers, Aaron Index: OpenBSD/PkgInfo.pm =

Raspberry Pi Ethernet MAC address

2017-07-29 Thread Mark Kettenis
Newer device trees for the Raspberry Pi have reorganized the usb part of the tree. This makes sure we can find the MAC address in those trees as well. Hopefully we can simplify this at some point when we have phased out old firmware. ok? Index: dev/usb/if_smsc.c ===

efiboot boot path matching

2017-07-29 Thread Mark Kettenis
This is apparently very hard. Caught this on arm64 where efi_device_path_depth() returned 0, which resulted in always selecting the first device. Clearly if the first path component (i = 0) matches the desired type, we should return 1, not 0. Here is the amd64 version of the diff which is easier

Re: sysctl machdep.lidaction=suspend

2017-07-29 Thread Martin Natano
On Sat, Jul 29, 2017 at 02:03:22PM +0200, Mark Kettenis wrote: > > I don't think we want to add string parsing like this in the kernel. > Maybe the sysctl(8) frontend should do the mapping from strings to > numbers? Ok, I'll try to come up with an alternative diff that does the parsing in sysctl(

Re: sysctl machdep.lidaction=suspend

2017-07-29 Thread Mark Kettenis
> Date: Sat, 29 Jul 2017 13:53:43 +0200 > From: Martin Natano > > Words are more descriptive than numbers. Let's use them! > > This diff removes > > sysctl machdep.lidaction=0 > sysctl machdep.lidaction=1 > sysctl machdep.lidaction=2 > > in favor of the more descriptive > >

sysctl machdep.lidaction=suspend

2017-07-29 Thread Martin Natano
Words are more descriptive than numbers. Let's use them! This diff removes sysctl machdep.lidaction=0 sysctl machdep.lidaction=1 sysctl machdep.lidaction=2 in favor of the more descriptive sysctl machdep.lidaction=none sysctl machdep.lidaction=suspend

Is someone interested in resuming support for socppc?

2017-07-29 Thread Андрей Болконский
https://www.openbsd.org/socppc.html > The OpenBSD/socppc port was discontinued after the 5.8 release. Otherwise, will you approve removing support for socppc from src?

nm in free(): chunk canary corrupted 0x3e6e6845580 0x19@0x19

2017-07-29 Thread Stuart Henderson
I was just running nm over all of /usr/lib on a system with C in malloc flags and ran into this. $ MALLOC_OPTIONS=C nm -s libc.so.89.3.a Archive index: _thread_sys___semctl in __semctl.so __semctl in __semctl.so ><...snip...> _libc_xdr_ypresp_master in xdr_ypresp_master.so _libc_xdr_ypresp_order

Re: [PATCH] Remove useless sys/sys/dkbad.h

2017-07-29 Thread Vadim Zhukov
2017-07-29 2:15 GMT+03:00 Андрей Болконский : > sorry. correct patch: > > 2017-07-28 23:52 GMT+03:00 Андрей Болконский : > >> This header not used since retire sparc. >> build amd64 is ok Committed, thanks. -- WBR, Vadim Zhukov

fsdb(8): remove erroneous h length modifier

2017-07-29 Thread Frederic Cambus
Hi tech@, Remove erroneous h length modifier, the argument has type 'int'. Comments? OK? Index: sbin/fsdb/fsdbutil.c === RCS file: /cvs/src/sbin/fsdb/fsdbutil.c,v retrieving revision 1.17 diff -u -p -r1.17 fsdbutil.c --- sbin/fsdb/f

Re: systat: return for non-void function

2017-07-29 Thread Florian Obser
On Fri, Jul 28, 2017 at 02:31:02PM +, Florian Obser wrote: > > pointed out by clang > > OK? > comon, don't be shy, this is an easy one... maybe with a bit more context? This is an error return: diff --git usr.bin/systat/pool.c usr.bin/systat/pool.c index b6691ae0da6..3b289446522 100644 ---