Re: remove /dev/sound*

2016-09-07 Thread Michael W. Bombardieri
Hi Alexandre, Do you know if any applications in ports use /dev/sound as default audio device. Maybe they are not smart enough to try /dev/audio if /dev/sound fails. - Michael On Thu, Sep 08, 2016 at 08:12:45AM +0200, Alexandre Ratchov wrote: > As audio(4) manual says "In all respects /dev/audi

Re: replace microtime with getmicrouptime in ip_mroute.c

2016-09-07 Thread Florian Obser
OK florian@ On Thu, Sep 08, 2016 at 11:54:01AM +1000, David Gwynne wrote: > it uses the time to rate limit the sending of assertion messages. > > there are two reasons for this change. > > firstly, using uptime instead of wall time means the intervals will > be measured against a a monotonic clo

Re: random malloc junk

2016-09-07 Thread Otto Moerbeek
On Wed, Sep 07, 2016 at 06:29:07PM -0400, Ted Unangst wrote: > Instead of always using a fixed byte pattern, I think malloc should use a > random pattern. Now, this sometimes means it's harder to identify exactly > what's used after free, so we should provide a means to get the old 0xdf > pattern

remove /dev/sound*

2016-09-07 Thread Alexandre Ratchov
As audio(4) manual says "In all respects /dev/audio and /dev/sound are identical". Only one of them is needed and this diff is to remove /dev/sound. OK? Index: etc/MAKEDEV.common === RCS file: /cvs/src/etc/MAKEDEV.common,v retrievin

Re: replace microtime with getmicrouptime in ip_mroute.c

2016-09-07 Thread Claudio Jeker
On Thu, Sep 08, 2016 at 11:54:01AM +1000, David Gwynne wrote: > it uses the time to rate limit the sending of assertion messages. > > there are two reasons for this change. > > firstly, using uptime instead of wall time means the intervals will > be measured against a a monotonic clock that isnt

Re: [patch] Some consistence

2016-09-07 Thread Jason McIntyre
On Wed, Sep 07, 2016 at 10:56:19PM -0300, Daniel Bolgheroni wrote: > Some consistence. > fixed, thanks. jmc > Index: ls.1 > === > RCS file: /cvs/src/bin/ls/ls.1,v > retrieving revision 1.77 > diff -u -p -r1.77 ls.1 > --- ls.1

[patch] Some consistence

2016-09-07 Thread Daniel Bolgheroni
Some consistence. Index: ls.1 === RCS file: /cvs/src/bin/ls/ls.1,v retrieving revision 1.77 diff -u -p -r1.77 ls.1 --- ls.110 Aug 2016 19:46:43 - 1.77 +++ ls.18 Sep 2016 01:44:18 - @@ -370,7 +370,7 @@ and

replace microtime with getmicrouptime in ip_mroute.c

2016-09-07 Thread David Gwynne
it uses the time to rate limit the sending of assertion messages. there are two reasons for this change. firstly, using uptime instead of wall time means the intervals will be measured against a a monotonic clock that isnt skewed by clock changes. it should therefore always be rate limited to 30

random malloc junk

2016-09-07 Thread Ted Unangst
Instead of always using a fixed byte pattern, I think malloc should use a random pattern. Now, this sometimes means it's harder to identify exactly what's used after free, so we should provide a means to get the old 0xdf pattern back. Since we already have two junk modes, I thought I'd carry on al

Re: Default softraid crypto PBKDF2 rounds

2016-09-07 Thread Damien Miller
On Wed, 7 Sep 2016, Andreas Bartelt wrote: > yes, due to the larger internal state of the blowfish algorithm which is > harder to efficiently realize in dedicated hardware. However, since bcrypt's > internal state effectively is of fixed size, scrypt would be an even better > option since it allow

look(1): eliminate FOLD and DICT macros

2016-09-07 Thread Todd C. Miller
There's no need to check for isascii() with ANSI ctype macros/functions. Eliminating the macros makes the code clearer. - todd Index: usr.bin/look/look.c === RCS file: /cvs/src/usr.bin/look/look.c,v retrieving revision 1.18 diff -u

netinet6 bcopy->memcpy

2016-09-07 Thread David Hill
Hello - Here is a diff to switch some bcopy's to memcpy's. Most bcopy's are on freshly alloc'd memory. Index: ip6_output.c === RCS file: /cvs/src/sys/netinet6/ip6_output.c,v retrieving revision 1.213 diff -u -p -r1.213 ip6_output.c

remove unreachable code from all audio drivers

2016-09-07 Thread Alexandre Ratchov
This diff deletes unreachable code in low-level audio drivers. As this touches all archs, tests (a quick kernel build is enough) are welcome on anything but amd64 and i386. OK? Index: share/man/man9/audio.9 === RCS file: /cvs/src/sh

static pie for arm

2016-09-07 Thread Mark Kettenis
Diff below makes static pie work on arm. While reviewing the MD_CRT0_START assembly I noticed that we don't actually need to save r4 and r5 on the stack as those arguments are unused. I also removed the useless comment around the stack alignment code. I'll probably hold off on the bsd.own.mk bit

config(8): strlen + emalloc + snprintf = asprintf

2016-09-07 Thread Michal Mazurek
ok? Index: usr.sbin/config/main.c === RCS file: /cvs/src/usr.sbin/config/main.c,v retrieving revision 1.50 diff -u -p -r1.50 main.c --- usr.sbin/config/main.c 16 Oct 2015 13:37:44 - 1.50 +++ usr.sbin/config/main.c 7

Re: mount(8): remove unneeded headers

2016-09-07 Thread Ted Unangst
Michal Mazurek wrote: > ok? > > Index: sbin/mount/getmntopts.c > === > RCS file: /cvs/src/sbin/mount/getmntopts.c,v > retrieving revision 1.12 > diff -u -p -r1.12 getmntopts.c > --- sbin/mount/getmntopts.c 16 Jan 2015 06:39:59 -

Re: Fix Wacom Intuos S 2 descriptor and make wsmouse work

2016-09-07 Thread Frank Groeneveld
On Wed, Sep 07, 2016 at 12:12:43AM +0200, Ulf Brosziewski wrote: > Hi, I was a bit hasty, I should have mentioned that calling > wsmouse_input_sync is required here. The equivalent of your > code would would be this: > > if (x != 0 || y != 0 || buttons != ms->sc_buttons) { > wsmouse_position

mount(8): remove unneeded headers

2016-09-07 Thread Michal Mazurek
ok? Index: sbin/mount/getmntopts.c === RCS file: /cvs/src/sbin/mount/getmntopts.c,v retrieving revision 1.12 diff -u -p -r1.12 getmntopts.c --- sbin/mount/getmntopts.c 16 Jan 2015 06:39:59 - 1.12 +++ sbin/mount/getmntopts

ping(8): fix multicast source address selection

2016-09-07 Thread Florian Obser
This got broken in rev 1.136 9 months ago. Sorry about that. Tests / OKs? p.s. applying + diff -bu is easier to figure out what's going on. diff --git ping.c ping.c index 9c2395a..b879d6f 100644 --- ping.c +++ ping.c @@ -121,7 +121,6 @@ int options; int moptions; #defineMULTICAST_NOLOO

Re: [PATCH] Callback-based interface to libtls

2016-09-07 Thread tobias
On 07.09.2016, at 10:45, Tobias Pape wrote: > Hi there > > On 05.09.2016, at 15:50, bust...@gmail.com wrote: > >> Hey, the typedef came in handy :) Ok bcook@ >> >> On Sep 5, 2016, at 11:52 AM, Bob Beck wrote: >> >>> I am in agreement in principle, but please coordinate with bcook@ and/or >

Re: mg: Collect forked off children from M-| command

2016-09-07 Thread Todd C. Miller
On Wed, 07 Sep 2016 12:05:22 -, Mark Lumsden wrote: > Source Joachim Nilsson: > > Collect forked off children from M-| command > > Mg left zombies from commands executed when piping a region of text to > an external command. This patch makes sure to collect for the child > b

Re: Default softraid crypto PBKDF2 rounds

2016-09-07 Thread joshua stein
On Wed, 07 Sep 2016 at 11:18:57 +0200, Andreas Bartelt wrote: > yes, due to the larger internal state of the blowfish algorithm which is > harder to efficiently realize in dedicated hardware. However, since bcrypt's > internal state effectively is of fixed size, scrypt would be an even better > opt

Re: mg: Collect forked off children from M-| command

2016-09-07 Thread Sunil Nimmagadda
On Wed, Sep 07, 2016 at 12:05:22PM +, Mark Lumsden wrote: > Source Joachim Nilsson: > > Collect forked off children from M-| command > > Mg left zombies from commands executed when piping a region of text to > an external command. This patch makes sure to collect for the child >

mg: Collect forked off children from M-| command

2016-09-07 Thread Mark Lumsden
Source Joachim Nilsson: Collect forked off children from M-| command Mg left zombies from commands executed when piping a region of text to an external command. This patch makes sure to collect for the child before returning. Looks ok to me. ok? Index: region.c

Re: Default softraid crypto PBKDF2 rounds

2016-09-07 Thread Andreas Bartelt
On 09/07/16 09:16, Damien Miller wrote: On Tue, 6 Sep 2016, David Coppa wrote: Il 6 settembre 2016 14:56:32 CEST, Filippo Valsorda ha scritto: Hello, I recently had the occasion to dive into the softraid crypto code [1] and was quite pleased with the cleanliness of it all. However, I found

Re: [PATCH] Callback-based interface to libtls

2016-09-07 Thread Tobias Pape
Hi there On 05.09.2016, at 15:50, bust...@gmail.com wrote: > Hey, the typedef came in handy :) Ok bcook@ > > On Sep 5, 2016, at 11:52 AM, Bob Beck wrote: > >> I am in agreement in principle, but please coordinate with bcook@ and/or >> jsing@ who were possibly doing >> some related adjustments

Re: mg - fix modeline segfault

2016-09-07 Thread Mark Lumsden
I think the question is how shold mg behave in a cornercase situation where mg is running but cannot be used in any meaningful fashion (as a text editor). In the case of mg being run with only two lines available to it, what should it do? 1. do nothing, just show the bottom 1 or 2 lines (minibuffe

Re: [patch] make npppd do as the man page says

2016-09-07 Thread YASUOKA Masahiko
Sorry for long delay. Using empty sting for hostname in PPTP is intentional. So I will fix the man page. Thank you for your report. On Tue, 8 Mar 2016 11:20:40 + Joe Holden wrote: > Below is a patch that corrects pptp to set the pptp hostname as the > system hostname, as per the man page a

Re: smtpd shutdown cleanup

2016-09-07 Thread Gilles Chehade
On Tue, Sep 06, 2016 at 11:31:04PM +0200, Eric Faurot wrote: > Previously, all processes would shutdown on receiving SIGINT or SIGTERM. > When going down, the parent process would kill all the other process and > waitpid() them. > > Now, only the parent process handles SIGTERM and SIGINT, other pr

Re: Default softraid crypto PBKDF2 rounds

2016-09-07 Thread Damien Miller
On Tue, 6 Sep 2016, David Coppa wrote: > Il 6 settembre 2016 14:56:32 CEST, Filippo Valsorda ha > scritto: > >Hello, > > > >I recently had the occasion to dive into the softraid crypto code [1] > >and was quite pleased with the cleanliness of it all. However, I found > >surprising the default va