vscsi.4, wsdisplay.4: add missing Dv tags to ioctl constants

2017-09-11 Thread Scott Cheloha
Hi, Constants without the Dv styling stick out quite a bit in a browser. This patch adds them to the ioctl constants in vscsi.4 and the two lone constants without them in wsdisplay.4. I also noticed that wsdisplay.4 is perhaps the only device driver page not using Fa for its ioctl arguments, so

Re: Open /dev/mem file failed when running as a root priviledge

2017-09-11 Thread Ingo Schwarze
Hi, moving from misc@ to tech@ because you found a documentation bug. Any OKs for the patch below? Ingo Nan Xiao wrote on Tue, Sep 12, 2017 at 08:58:25AM +0800: > I want to run dmidecode (https://github.com/mirror/dmidecode) on > OpenBSD 6.1, but executing it will report following errors: >

Re: Remove DDB_STRUCTINFO

2017-09-11 Thread David Gwynne
> On 11 Sep 2017, at 20:01, Martin Pieuchot wrote: > > Now that dlg@ implemented 'show struct' using CTF, which makes it > available in GENERIC kernel we can retire DDB_STRUCTINFO. > > Diff below does that, ok? it seems noone will miss the "show offset" command below, so ok.

syslogd close *:514 sockets

2017-09-11 Thread Alexander Bluhm
Hi, In the default configuration syslogd keeps two *:514 UDP sockets open. udp 0 0 *.514 *.* udp6 0 0 *.514 *.* Several people have asked me why they are in netstat output and whether it is a

ifconfig: restore error during process_media_commands

2017-09-11 Thread Dave Voutila
Hi tech@, Was reading through ifconfig.c today and noticed something odd (to me). Very well could be my ignorance, but it looks like since rev 1.203 there's been an unused if statement around the ioctl call while processing media commands. It very well could be by design to allow execution to

Re: md5(1): use mono clock in time trial

2017-09-11 Thread Todd C. Miller
On Mon, 11 Sep 2017 16:19:02 +0200, Christian Weisgerber wrote: > FreeBSD uses getrusage() to fetch the user time used. I think that > makes more sense. > > https://svnweb.freebsd.org/base/head/sbin/md5/md5.c?revision=307658=mark > up#l293 Indeed. There's no point in including system time in

Re: malloc:check double free before canary

2017-09-11 Thread Todd C. Miller
On Mon, 11 Sep 2017 13:14:45 +0200, Otto Moerbeek wrote: > A double free of a chunk corrupts the canary, but I prefer to see a > double free error reported, Makes sense. OK millert@ - todd

Re: md5(1): use mono clock in time trial

2017-09-11 Thread Theo de Raadt
> > On Sep 11, 2017, at 9:25 AM, Theo de Raadt > wrote: > > > >> Scott Cheloha: > >> > >>> Use a monotonic clock for the elapsed time trial. > >> > >> FreeBSD uses getrusage() to fetch the user time used. I think that > > >> makes more sense. > >> > >> >

Re: md5(1): use mono clock in time trial

2017-09-11 Thread Scott Cheloha
> On Sep 11, 2017, at 9:25 AM, Theo de Raadt wrote: > >> Scott Cheloha: >> >>> Use a monotonic clock for the elapsed time trial. >> >> FreeBSD uses getrusage() to fetch the user time used. I think that >> makes more sense. >> >>

Re: md5(1): use mono clock in time trial

2017-09-11 Thread Theo de Raadt
> Scott Cheloha: > > > Use a monotonic clock for the elapsed time trial. > > FreeBSD uses getrusage() to fetch the user time used. I think that > makes more sense. > > https://svnweb.freebsd.org/base/head/sbin/md5/md5.c?revision=307658=markup#l293 The granularity is weak. However you can

Re: md5(1): use mono clock in time trial

2017-09-11 Thread Christian Weisgerber
Scott Cheloha: > Use a monotonic clock for the elapsed time trial. FreeBSD uses getrusage() to fetch the user time used. I think that makes more sense. https://svnweb.freebsd.org/base/head/sbin/md5/md5.c?revision=307658=markup#l293 -- Christian "naddy" Weisgerber

malloc:check double free before canary

2017-09-11 Thread Otto Moerbeek
Hi, A double free of a chunk corrupts the canary, but I prefer to see a double free error reported, -Otto Index: malloc.c === RCS file: /cvs/src/lib/libc/stdlib/malloc.c,v retrieving revision 1.228 diff -u -p -r1.228

Re: Remove DDB_STRUCTINFO

2017-09-11 Thread Jasper Lievisse Adriaanse
On Mon, Sep 11, 2017 at 10:01:27AM +, Martin Pieuchot wrote: > Now that dlg@ implemented 'show struct' using CTF, which makes it > available in GENERIC kernel we can retire DDB_STRUCTINFO. > > Diff below does that, ok? I suggested the same dlg, so OK with me. Cheers, Jasper > Index:

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

2017-09-11 Thread Martin Pieuchot
On 29/07/17(Sat) 21:56, Matthew Martin wrote: > 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

Remove DDB_STRUCTINFO

2017-09-11 Thread Martin Pieuchot
Now that dlg@ implemented 'show struct' using CTF, which makes it available in GENERIC kernel we can retire DDB_STRUCTINFO. Diff below does that, ok? Index: conf/files === RCS file: /cvs/src/sys/conf/files,v retrieving revision

post-clang Makefile.cross

2017-09-11 Thread Martin Pieuchot
Now that cc(1) is a link to clang(1) we need to explicitly set CC, CXX and COMPILER_VERSION to cross-compile gcc4 architecture toolchains. ok? Index: Makefile.cross === RCS file: /cvs/src/Makefile.cross,v retrieving revision 1.96