Re: siginfo_t.si_addr should be void*

2017-04-06 Thread Philip Guenther
On Sat, Apr 1, 2017 at 10:03 PM, Andrew Aldridge wrote: > --- siginfo.h 14 Apr 2015 16:40:46 - 1.11 > +++ siginfo.h 29 Mar 2017 22:17:49 - > @@ -150,7 +150,7 @@ typedef struct { ... > - caddr_t _addr; /* faulting address */ > +

Re: acpicpu(4): simplify len argument to memset

2017-04-06 Thread Philip Guenther
On Thu, Apr 6, 2017 at 12:07 PM, Anton Lindqvist wrote: > Instead of passing sizeof(struct aml_value)*4 as the len argument to > memset when writing zeros to osc_cmd, pass sizeof(osc_cmd) since osc_cmd > already is declared as 'struct aml_value osc_cmd[4]'. Committed.

patch: mv(1): Add -p flag to preserve time stamps for moved directories

2017-04-06 Thread bytevolcano
Although mv(1) will preserve atime and mtime for moved directories if moved to a different file system, it doesn't preserve them when moved in the same file system. Upon looking around, it appears to be standard POSIX behavior, as the utility is meant to perform a rename() and exit. This patch

Re: OpenBSD 6.0 relayd TLS Loadbalancer failing SSL Lab tests (Cipher Support)

2017-04-06 Thread Tom Smyth
Joel, Thanks for that ... your mail was most helpful I tried the suggested setting "HIGH:!aNULL" It worked perfect so the config now looks like this http protocol https { match request header append "X-Forwarded-For" value "$REMOTE_ADDR" match request header append

Re: OpenBSD 6.0 relayd TLS Loadbalancer failing SSL Lab tests (Cipher Support)

2017-04-06 Thread Joel Sing
On Thursday 06 April 2017 16:38:26 Tom Smyth wrote: > Hello all, > > I was installing relayd as a loadbalancer (and ssl terminator) on > OpenBSD6.0 > amd64 base install, > > I used the following configuration for my /etc/relayd.conf file > > http protocol https { > match request header

Re: gzip not exiting 2 when not compressible

2017-04-06 Thread Todd C. Miller
This was fixed after 6.0 came out. I believe it was: CVSROOT:/cvs Module name:src Changes by: mill...@cvs.openbsd.org 2016/08/17 06:02:38 Modified files: usr.bin/compress: gzopen.c Log message: When returning the file size info, only use z_total_in/z_total_out for the

Re: vm.conf.5 / parse.y inconsistency

2017-04-06 Thread Reyk Floeter
On Thu, Apr 06, 2017 at 08:58:34PM +, Pontus Lundkvist wrote: > Hi, > > Version 1.17 of vm.conf.5 changed the keyword "kernel" to "boot", but the > parser still expects "kernel". > > Regards, > Pontus You're right, I somehow missed to commit the parse.y bit. OK? Reyk Index: parse.y

Re: gzip not exiting 2 when not compressible

2017-04-06 Thread Ted Unangst
Craig Skinner wrote: > > On a default 6.0 release machine, gzip keeps on going/growing: no repro on 6.1

vm.conf.5 / parse.y inconsistency

2017-04-06 Thread Pontus Lundkvist
Hi, Version 1.17 of vm.conf.5 changed the keyword "kernel" to "boot", but the parser still expects "kernel". Regards, Pontus

gzip not exiting 2 when not compressible

2017-04-06 Thread Craig Skinner
Hello there, >From gzip(1): DESCRIPTION .. ... If compression would not reduce the size of a file, the file is ignored (unless -f is used). EXIT STATUS ... 2 At least one of the specified files was not compressed since -f was not specified and

acpicpu(4): simplify len argument to memset

2017-04-06 Thread Anton Lindqvist
Hi, Instead of passing sizeof(struct aml_value)*4 as the len argument to memset when writing zeros to osc_cmd, pass sizeof(osc_cmd) since osc_cmd already is declared as 'struct aml_value osc_cmd[4]'. Tested on amd64. Index: acpicpu.c

Re: bioctl(8): redundant mention of command line in man-page

2017-04-06 Thread Jason McIntyre
On Thu, Apr 06, 2017 at 08:35:02PM +0200, Anton Lindqvist wrote: > Hi, > The bioctl man-page uses the phrase "The following command, executed > from the command line, ..." in the examples section. A convention I > haven't seen before. I would argue that mentioning the command line is > redundant

bioctl(8): redundant mention of command line in man-page

2017-04-06 Thread Anton Lindqvist
Hi, The bioctl man-page uses the phrase "The following command, executed from the command line, ..." in the examples section. A convention I haven't seen before. I would argue that mentioning the command line is redundant since it's implied by the actual examples and could therefore be removed.

Re: malloc.3: make arguments consistent in example code

2017-04-06 Thread Otto Moerbeek
On Wed, Apr 05, 2017 at 11:21:00PM -0700, Christopher Hettrick wrote: > Hi tech@ > > The arguments to malloc() in the EXAMPLE section are not consistent. > Make a couple other consistency fixes as well. Committed, thanks, -Otto > > Index: malloc.3 >

OpenBSD 6.0 relayd TLS Loadbalancer failing SSL Lab tests (Cipher Support)

2017-04-06 Thread Tom Smyth
Hello all, I was installing relayd as a loadbalancer (and ssl terminator) on OpenBSD6.0 amd64 base install, I used the following configuration for my /etc/relayd.conf file http protocol https { match request header append "X-Forwarded-For" value "$REMOTE_ADDR" match request

ip_esp/ip_mroute bcopy->mem*

2017-04-06 Thread David Hill
Hello - Convert bcopy to memcpy where the memory does not overlap, otherwise, use memmove. While here, change some previous conversions to a simple assignment. OK? Index: ip_esp.c === RCS file: /cvs/src/sys/netinet/ip_esp.c,v

spamd.c patch

2017-04-06 Thread Edgar Pettijohn
Use errx() instead of fprintf() like the rest of the options. Originally noticed because of the missing '\' for the newline. Index: spamd.c === RCS file: /cvs/src/libexec/spamd/spamd.c,v retrieving revision 1.150 diff -u -p -u

Re: wsconsctl(8): mention decrement assignment in man-page

2017-04-06 Thread Jason McIntyre
On Thu, Apr 06, 2017 at 08:06:58AM +0100, Jason McIntyre wrote: > On Thu, Apr 06, 2017 at 08:24:58AM +0200, Anton Lindqvist wrote: > > On Wed, Apr 05, 2017 at 10:03:08PM +0100, Jason McIntyre wrote: > > > On Wed, Apr 05, 2017 at 04:37:04PM +0200, Anton Lindqvist wrote: > > > > Hi, > > > > Support

Re: wsconsctl(8): mention decrement assignment in man-page

2017-04-06 Thread Jason McIntyre
On Thu, Apr 06, 2017 at 08:24:58AM +0200, Anton Lindqvist wrote: > On Wed, Apr 05, 2017 at 10:03:08PM +0100, Jason McIntyre wrote: > > On Wed, Apr 05, 2017 at 04:37:04PM +0200, Anton Lindqvist wrote: > > > Hi, > > > Support for decrementing a variable using the '-=' operator with > > > wsconsctl

Re: wsconsctl(8): mention decrement assignment in man-page

2017-04-06 Thread Anton Lindqvist
On Wed, Apr 05, 2017 at 10:03:08PM +0100, Jason McIntyre wrote: > On Wed, Apr 05, 2017 at 04:37:04PM +0200, Anton Lindqvist wrote: > > Hi, > > Support for decrementing a variable using the '-=' operator with > > wsconsctl was added in r1.11 of wsconsctl.c. It never ended up being > > documented.

malloc.3: make arguments consistent in example code

2017-04-06 Thread Christopher Hettrick
Hi tech@ The arguments to malloc() in the EXAMPLE section are not consistent. Make a couple other consistency fixes as well. Index: malloc.3 === RCS file: /cvs/src/lib/libc/stdlib/malloc.3,v retrieving revision 1.108 diff -u -p