OpenBSD crash with pfsync

2014-05-22 Thread Loïc Blot
Hi, today i upgraded my primary router from OpenBSD 5.4 to OpenBSD 5.5 (i follow the process described here: http://www.openbsd.org/faq/upgrade55.html and this is my 5th upgrade from 5.4 to 5.5 since the release). After rebooting and doing the sysmerge without network copper cables, i rebooted

Re: OpenBSD crash with pfsync

2014-05-22 Thread Martin Pieuchot
Hello Loïc, On 22/05/14(Thu) 11:11, Loïc Blot wrote: Hi, today i upgraded my primary router from OpenBSD 5.4 to OpenBSD 5.5 (i follow the process described here: http://www.openbsd.org/faq/upgrade55.html and this is my 5th upgrade from 5.4 to 5.5 since the release). After rebooting and

move all stats under MALLOC_STATS

2014-05-22 Thread Otto Moerbeek
Hi, some stats are always computed. Move them under #ifdef MALLOC_STATS And a small thing for error reporting. ok? -Otto Index: malloc.c === RCS file: /cvs/src/lib/libc/stdlib/malloc.c,v retrieving revision 1.165 diff -u

Re: OpenBSD crash with pfsync

2014-05-22 Thread Loïc Blot
Hi Martin, i'm using IPv6 but not on the syncdev interface, on all my other interfaces (CARP OSPF). Here is the syncdev config: inet 10.XX.XX.2 255.255.255.224 NONE vlan 30 vlandev trunk1 and the pfsync config: up defer syncdev vlan30 To precise how the bug happens, i have removed some other

Re: move all stats under MALLOC_STATS

2014-05-22 Thread Kenneth Westerback
On 22 May 2014 07:29, Otto Moerbeek o...@drijf.net wrote: Hi, some stats are always computed. Move them under #ifdef MALLOC_STATS And a small thing for error reporting. ok? Makes sense to me, looks good. ok krw@ Ken -Otto Index: malloc.c

Re: [PATCH 2] followup Speedstep bug

2014-05-22 Thread Benjamin Baier
Works for me. Since this disables speedstep late in the init, wouldn't it be nice to free est_fqlist before disabling (not enabling) speedstep? Im running this diff for one week now, on amd64. Index: est.c === RCS file:

[PATCH]unnecessary return in arc4random

2014-05-22 Thread Fritjof Bornebusch
Hi tech, does this return makes any sense, because it's a void function and the return is at the end of the function. fritjof Index: arc4random.c === RCS file: /cvs/src/lib/libc/crypt/arc4random.c,v retrieving revision 1.30 diff

Re: [PATCH] libssl:Remove NULL checks before calling free()

2014-05-22 Thread Miod Vallat
Hello, After reading http://www.openbsd.org/papers/bsdcan14-libressl/mgp00015.html , I thought I'd help a bit with cleaning libssl by running this Coccinelle[1] script on src/lib/libssl: smpl @@ identifier x; @@ -if (x) { free(x); } +free(x); /smpl It removes unnecessary NULL

[PATCH] workaround buggy ACPI tables in some Intel boards

2014-05-22 Thread John D. Verne
There was some discussion of this on misc@ recently. Some Baytrail boards are setting local APIC flags to 0b11, which is a reserved value. The acpidump and other info related to the problem is capture over there, as well. Ref: http://marc.info/?l=openbsd-miscm=140043989412703w=2 Instead of

nginx + ssl breakage........

2014-05-22 Thread ianmcw01
Anybody else seeing this? cc -c -O2 -pipe -W -Wall -Wpointer-arith -Wno-unused-parameter -Werror -g - DNGX_ENABLE_SYSLOG -DHAVE_CONFIG_H -DPCRE_STATIC -DPOSIX_MALLOC_THRESHOLD=10 -I src/core -I src/event -I src/event/modules -I src/os/unix -I src/pcre -I objs -o

acpi rsdp patch

2014-05-22 Thread Daniel Dickman
The ACPI RSDP table has a checksum and an extended checksum. At the moment if the ACPI rvision is zero, the checksum is checked and if the revision is more recent only the extended checksum field is checked. However the spec says that the checksum field ...must sum to zero. At the moment this

Re: nginx + ssl breakage........

2014-05-22 Thread Ted Unangst
On Fri, May 23, 2014 at 11:16, ianmc...@tpg.com.au wrote: Anybody else seeing this? cc -c -O2 -pipe -W -Wall -Wpointer-arith -Wno-unused-parameter -Werror -g - DNGX_ENABLE_SYSLOG -DHAVE_CONFIG_H -DPCRE_STATIC -DPOSIX_MALLOC_THRESHOLD=10 -I src/core -I src/event -I src/event/modules

Re: [PATCH 2] followup Speedstep bug

2014-05-22 Thread Philip Guenther
On Thu, May 22, 2014 at 7:24 AM, Benjamin Baier program...@netzbasis.dewrote: Works for me. Since this disables speedstep late in the init, wouldn't it be nice to free est_fqlist before disabling (not enabling) speedstep? Im running this diff for one week now, on amd64. Committed (with

Re: NOINET6 by default

2014-05-22 Thread Theo de Raadt
* Mark Kettenis mark.kette...@xs4all.nl [2014-05-15 00:15]: I don't think this is a good idea; didn't we establish the other day that ifconfig if eui64 already did what your +inet6 does? almost, it's ifconfig if inet6 eui64 - but that isn't all THAT intuitive. I like +inet6 as the opposite