Re: rcmdsh(3): use getaddrinfo() not gethostbyname2()

2016-05-27 Thread Theo de Raadt
> rmcd(3) was converted ages ago but apparently this was missed. Nice. That is the last gethostbyname2() in base or X, except for a regress test. One difference in behaviour: inet vs inet6 ordering is now imposed by the back end, rather than the code order. And that is a bug fix. > Index:

rcmdsh(3): use getaddrinfo() not gethostbyname2()

2016-05-27 Thread Todd C. Miller
rmcd(3) was converted ages ago but apparently this was missed. - todd Index: lib/libc/net/rcmdsh.c === RCS file: /cvs/src/lib/libc/net/rcmdsh.c,v retrieving revision 1.18 diff -u -p -u -r1.18 rcmdsh.c --- lib/libc/net/rcmdsh.c

Re: kernel panic - rtable_match

2016-05-27 Thread David Hill
Great! I appreciate the commit hint. Thanks On Fri, May 27, 2016 at 10:46:32PM +0100, Stuart Henderson wrote: > On 2016/05/27 17:29, David Hill wrote: > > Hello - > > > > Running openbsd current via cvs from today.. > > > > My machine keeps panic'ing with a uvm_fault in rtable_match. > > It

Re: utvfu driver port

2016-05-27 Thread Mike Belopuhov
On Thu, May 26, 2016 at 16:09 -0700, patrick keshishian wrote: > Hi, > > Included is my initial effort to port the dual-licensed driver > for Fushicai Audio-Video Grabber (vendor 0x1b71 product 0x3002). > > As I mentioned in previous email it is "mostly working". I have > tested it on amd64 and

Re: ksh 641 should be read-only

2016-05-27 Thread Anthony Coulter
> ksh won't start after this change. The problem is that it will try to > assign the return value of getppid() to PPID *after* PPID was made read > only. Such is the price of not testing last-minute changes. When ksh processes a typeset command, it sets the read-only flag before attempting to

Re: Spelling errors

2016-05-27 Thread Stuart Henderson
On 2016/05/26 23:51, Anthony Coulter wrote: > The long line in asr.c barely squeaks in at 79 characters. > > > Index: lib/libtls/tls_init.3 > Index: lib/libc/asr/asr.c I've committed these two > Index: usr.sbin/bind/lib/isc/include/isc/hash.h Skipped this one. I'm still hopeful that at some

Re: ksh 641 should be read-only

2016-05-27 Thread Theo Buehler
On Thu, May 26, 2016 at 11:00:39PM -0400, Anthony Coulter wrote: > The man page for ksh says that ${PPID} should be read-only but > apparently it is not. There is also a line nearby that is more > than 80 characters long but I'm not comfortable enough with KNF > to select the best way to fix that.

Re: use time_uptime for route expiry times, not time_second

2016-05-27 Thread Martin Pieuchot
On 27/05/16(Fri) 17:06, David Gwynne wrote: > time_second is unix time, can jump with clock changes. time_uptime > is better cos it is monotonic. > > userland expects the expiry to be in unix time though, so it's > translated before it appears in route messages. Could you at least keep ND in

ksh 641 should be read-only

2016-05-27 Thread Anthony Coulter
The man page for ksh says that ${PPID} should be read-only but apparently it is not. There is also a line nearby that is more than 80 characters long but I'm not comfortable enough with KNF to select the best way to fix that. Index: main.c

Spelling errors

2016-05-27 Thread Anthony Coulter
The long line in asr.c barely squeaks in at 79 characters. Index: lib/libtls/tls_init.3 === RCS file: /cvs/src/lib/libtls/tls_init.3,v retrieving revision 1.59 diff -u -p -r1.59 tls_init.3 --- lib/libtls/tls_init.3 28 Apr 2016

Re: utvfu driver port

2016-05-27 Thread Marcus Glocker
Hello Patrick, On Thu, May 26, 2016 at 04:09:26PM -0700, patrick keshishian wrote: > Hi, > > Included is my initial effort to port the dual-licensed driver > for Fushicai Audio-Video Grabber (vendor 0x1b71 product 0x3002). > > As I mentioned in previous email it is "mostly working". I have >

kernel panic - rtable_match

2016-05-27 Thread David Hill
Hello - Running openbsd current via cvs from today.. My machine keeps panic'ing with a uvm_fault in rtable_match. It locks my whole machine up, so I am unable to type anything. Perhaps someone can provide me some debugging printf's? uvm_fault(0x81945e00, 0x1, 0, 1) -> e kernel: page

Re: replace bpf open loops

2016-05-27 Thread Jeremie Courreges-Anglas
Martin Natano writes: > I think it's time to get rid of all the bpf open() loops in base. > dhclient and libpcap do a plain open("/dev/bpf0", ...) since a couple of > weeks now and the upgrade issue (/dev/bpf vs. /dev/bpf0) has been fixed. > I didn't hear any other complaints

Re: ptrace PT_IO write bug

2016-05-27 Thread Jeremie Courreges-Anglas
Mathieu - writes: > Hello everyone, > > While playing a bit with ptrace to do some debugging I stumbled upon > something that looks like a bug. > While trying to write to the ptrace'd process using PT_IO in combinaison > with PIOD_WRITE_D I kept getting EFAULTs. > PIOD_READ_D

Re: kernel panic - rtable_match

2016-05-27 Thread Stuart Henderson
On 2016/05/27 17:29, David Hill wrote: > Hello - > > Running openbsd current via cvs from today.. > > My machine keeps panic'ing with a uvm_fault in rtable_match. > It locks my whole machine up, so I am unable to type anything. Perhaps > someone can provide me some debugging printf's? I sent

mklocale(1): ignore blanks after VARIABLE

2016-05-27 Thread Todd C. Miller
Currently, mklocale only ignores the first blank after the VARIABLE definition. This means that we store the variable definition along with the leading blanks. The lexer should eat the blanks after VARIABLE before storing the variable definition. - todd Index: usr.bin/mklocale/lex.l

Re: utvfu driver port

2016-05-27 Thread patrick keshishian
On Fri, May 27, 2016 at 08:41:33PM +0200, Marcus Glocker wrote: > Hello Patrick, > > On Thu, May 26, 2016 at 04:09:26PM -0700, patrick keshishian wrote: > > > Hi, > > > > Included is my initial effort to port the dual-licensed driver > > for Fushicai Audio-Video Grabber (vendor 0x1b71 product

use time_uptime for route expiry times, not time_second

2016-05-27 Thread David Gwynne
time_second is unix time, can jump with clock changes. time_uptime is better cos it is monotonic. userland expects the expiry to be in unix time though, so it's translated before it appears in route messages. ok? Index: net/route.c

Re: uvideo patches: Overview [0/4]

2016-05-27 Thread Marcus Glocker
On Thu, May 26, 2016 at 04:18:58PM -0700, patrick keshishian wrote: > On Thu, May 26, 2016 at 10:55:47PM +0200, Marcus Glocker wrote: > > On Tue, May 17, 2016 at 05:15:51PM -0700, patrick keshishian wrote: > > > > > Greetings, > > > > > > I have been looking at uvideo trying to model a new