Re: inteldrm(4) diff that needs testing

2015-10-24 Thread Philip Guenther
On Sat, Oct 24, 2015 at 2:48 PM, Mark Kettenis wrote: > The diff below makes inteldrm(4) attach directly to pci(4) instead of > vga(1). Because inteldrm(4) depends on intagp(4), this also make > intagp(4) a child of inteldrm(4). Ultimately I'd like to integrate > intagp(4) into inteldrm(4), but

Re: telnet: kill dead IP source route

2015-10-24 Thread Philip Guenther
On Sat, Oct 24, 2015 at 9:08 AM, Jérémie Courrèges-Anglas wrote: > until today I did not know that some of our code did check for IP SSRR > and LSRR IP options in incoming packets. > > That made me look for software that tried to use these options in > outgoing packets. That code, in telnet(1), h

Re: inteldrm(4) diff that needs testing

2015-10-24 Thread Jonathan Gray
On Sat, Oct 24, 2015 at 11:48:01PM +0200, Mark Kettenis wrote: > The diff below makes inteldrm(4) attach directly to pci(4) instead of > vga(1). Because inteldrm(4) depends on intagp(4), this also make > intagp(4) a child of inteldrm(4). Ultimately I'd like to integrate > intagp(4) into inteldrm(

Re: use machdep.lidsuspend=2 for hibernate

2015-10-24 Thread Glenn Faustino
This patch works fine on my Thinkpad x220. Hibernate/unhibernate works when closing/opening the lid. Thanks, Glenn dmesg: OpenBSD 5.8-current (GENERIC.MP ) #0: Sat Oct 24 15:07:40 PHT 2015 root@X220.domain.local:/usr/src/sys/arch/amd64/compile/GENERIC.MP

inteldrm(4) diff that needs testing

2015-10-24 Thread Mark Kettenis
The diff below makes inteldrm(4) attach directly to pci(4) instead of vga(1). Because inteldrm(4) depends on intagp(4), this also make intagp(4) a child of inteldrm(4). Ultimately I'd like to integrate intagp(4) into inteldrm(4), but that's going to be a bit more work. This diff is needed to mak

pledge(2) spamdb(8)

2015-10-24 Thread Ricardo Mestre
Hi tech@, spamdb(8) can pledge(2) for "stdio rpath wpath flock" and "flock" permission can be dropped after "/var/db/spamd" is opened with dbopen(3). Index: spamdb.c === RCS file: /cvs/src/usr.sbin/spamdb/spamdb.c,v retrieving rev

httpd and FastCGI - incorrect assumption?

2015-10-24 Thread Adam Wysocki
Hi tech@ reader, I am running OpenBSD 5.8 on amd64 and I have encountered a non-deterministic and incorrect behavior of httpd and slowcgi. I traced the problem to be that httpd is assuming that only first FastCGI FCGI_STDOUT record will contain headers and all subsequent records contain body (r

Re: pledge(2) date(1) earlier

2015-10-24 Thread Ricardo Mestre
I meant pledge(2) of course :) On 24/10/2015 18:52, Ricardo Mestre wrote: Hi tech@ Now that settime is available in pledge(6) then date(1) can pledge earlier, and afterwards drop the settime privilege (this is already in): Index: date.c ===

Re: pledge(2) in quiz(6)

2015-10-24 Thread Jan Stary
Any comments/OK? On Oct 22 13:49:28, h...@stare.cz wrote: > games/quiz.c popen()s a PAGER to display the help message. > Throw that away and let the use pipe into a PAGER if needed. > Then we can just pledge "stdio rpath". > > Jan > > > Index: quiz.c >

[patch] tcpdump - better BGP UPDATE AS_PATH size calculations

2015-10-24 Thread Kevin Reay
Adopt an updated version of the tcpdump.org ASN size calculation for BGP UPDATE message AS_PATHs. This corrects some bad behaviour due to incorrect ASN size calculations. I believe that the current way of calculating the ASN size for an UPDATE AS_PATH attribute is flawed. Currently, the ASN lengt

pledge(6) date(1) earlier

2015-10-24 Thread Ricardo Mestre
Hi tech@ Now that settime is available in pledge(6) then date(1) can pledge earlier, and afterwards drop the settime privilege (this is already in): Index: date.c === RCS file: /cvs/src/bin/date/date.c,v retrieving revision 1.49 d

rt_missmsg takes interface index

2015-10-24 Thread Alexander Bluhm
Hi, Instead of doing the the if_get() dance for rt_missmsg(), change the function to take an interface index. ok? bluhm Index: net/route.c === RCS file: /data/mirror/openbsd/cvs/src/sys/net/route.c,v retrieving revision 1.260 diff

Re: [patch] tcpdump segfault on invalid DECnet packet

2015-10-24 Thread Sebastian Benoit
Stuart Henderson(st...@openbsd.org) on 2015.10.20 16:37:58 +0100: > On 2015/10/14 11:11, Kevin Reay wrote: > > Thanks for the review and feedback. > > Updated patch with removed whitespace changes included. > > This is fine with me. Any OKs to commit it? yes, ok > > Index: print-decnet.c > > ==

Re: Introduce if_rtrequest()

2015-10-24 Thread Martin Pieuchot
On 24/10/15(Sat) 18:01, Martin Pieuchot wrote: > Revamping the network stack design continue here at u2k15... > > This times it's about the L2 resolution mechanism plugged into the > routing table. While discussing with Alexander we figured out that > the actual ifa_rtrequest() is always called

Re: The router doesn't know the size of the internet...

2015-10-24 Thread Florian Obser
On Sat, Oct 24, 2015 at 06:15:48PM +0200, Sebastian Benoit wrote: > Alexander Bluhm(alexander.bl...@gmx.net) on 2015.10.24 17:21:27 +0200: > > On Sat, Oct 24, 2015 at 04:02:59PM +0200, Martin Pieuchot wrote: > > > ...at least better than OpenBSD's source code. > > > > > > This diff gets rid of the

Re: The router doesn't know the size of the internet...

2015-10-24 Thread Sebastian Benoit
Alexander Bluhm(alexander.bl...@gmx.net) on 2015.10.24 17:21:27 +0200: > On Sat, Oct 24, 2015 at 04:02:59PM +0200, Martin Pieuchot wrote: > > ...at least better than OpenBSD's source code. > > > > This diff gets rid of the horrible per-ifp autoconf'd-ndp only hoplimit. > > Alexander verified that

telnet: kill dead IP source route

2015-10-24 Thread Jérémie Courrèges-Anglas
Hi, until today I did not know that some of our code did check for IP SSRR and LSRR IP options in incoming packets. That made me look for software that tried to use these options in outgoing packets. That code, in telnet(1), has been broken since 1998, says cvs log. I doubt that anyone will mi

Introduce if_rtrequest()

2015-10-24 Thread Martin Pieuchot
Revamping the network stack design continue here at u2k15... This times it's about the L2 resolution mechanism plugged into the routing table. While discussing with Alexander we figured out that the actual ifa_rtrequest() is always called with a valid ``ifp''... So this diff change this functio

Re: two simple netinet6 rt_ifidx conversion

2015-10-24 Thread Alexander Bluhm
OK bluhm@ On Sat, Oct 24, 2015 at 03:25:46PM +0200, Martin Pieuchot wrote: > Index: netinet6/in6_ifattach.c > === > RCS file: /cvs/src/sys/netinet6/in6_ifattach.c,v > retrieving revision 1.97 > diff -u -p -r1.97 in6_ifattach.c > --- n

Re: The router doesn't know the size of the internet...

2015-10-24 Thread Alexander Bluhm
On Sat, Oct 24, 2015 at 04:02:59PM +0200, Martin Pieuchot wrote: > ...at least better than OpenBSD's source code. > > This diff gets rid of the horrible per-ifp autoconf'd-ndp only hoplimit. > Alexander verified that this is not mandatory in the corresponding RFCs > and what really matters is the

pair(4) is not vether(4)

2015-10-24 Thread Reyk Floeter
Hi, some people asked me if pair(4) is a superset of vether(4). It is not (anymore). The most important difference is the link state handling: while the link of vether(4) is always active as soon as it is configured up, the link state of pair(4) is up when it is patched into a second pair(4). And

Re: kill NLS (native language support) libc errno message

2015-10-24 Thread Stefan Sperling
On Sat, Oct 24, 2015 at 04:07:59PM +0200, Alexander Bluhm wrote: > Hi, > > The only thing that is translated into multiple languages in OpenBSD > are the errno messages and signal names. Everything else is in > English. We are not planning to translate more text. Running a > mixed system with l

in6_selectroute regression (from c2k15)

2015-10-24 Thread Martin Pieuchot
When I changed rt_ifp to no longer point to lo0 for RTF_LOCAL routes I forgot the chunk below and introduced a regression. Checking for (rt_ifp->if_flags & IFF_LOOPBACK) was the old way to check for RTF_LOCAL. ok? Index: netinet6/in6_src.c

kill NLS (native language support) libc errno message

2015-10-24 Thread Alexander Bluhm
Hi, The only thing that is translated into multiple languages in OpenBSD are the errno messages and signal names. Everything else is in English. We are not planning to translate more text. Running a mixed system with less than 1% of the text in native language makes no sense. So I suggest to r

The router doesn't know the size of the internet...

2015-10-24 Thread Martin Pieuchot
...at least better than OpenBSD's source code. This diff gets rid of the horrible per-ifp autoconf'd-ndp only hoplimit. Alexander verified that this is not mandatory in the corresponding RFCs and what really matters is the per-PCB specified hop limit. See how this simplifies a lot of rt_ifp usage

two simple netinet6 rt_ifidx conversion

2015-10-24 Thread Martin Pieuchot
Index: netinet6/in6_ifattach.c === RCS file: /cvs/src/sys/netinet6/in6_ifattach.c,v retrieving revision 1.97 diff -u -p -r1.97 in6_ifattach.c --- netinet6/in6_ifattach.c 22 Oct 2015 15:37:47 - 1.97 +++ netinet6/in6_ifattac

STANDARDS for nl_langinfo(3)

2015-10-24 Thread Anthony J. Bentley
ok? Index: nl_langinfo.3 === RCS file: /cvs/src/lib/libc/locale/nl_langinfo.3,v retrieving revision 1.10 diff -u -p -r1.10 nl_langinfo.3 --- nl_langinfo.3 21 Jan 2014 03:15:45 - 1.10 +++ nl_langinfo.3 24 Oct 2015

More easy rt_ifix conversions

2015-10-24 Thread Martin Pieuchot
ok? Index: netinet/if_ether.c === RCS file: /cvs/src/sys/netinet/if_ether.c,v retrieving revision 1.176 diff -u -p -r1.176 if_ether.c --- netinet/if_ether.c 22 Oct 2015 18:14:53 - 1.176 +++ netinet/if_ether.c 24 Oct 2015 12

Re: unify errno messages

2015-10-24 Thread Todd C. Miller
On Sat, 24 Oct 2015 00:41:18 +0200, Alexander Bluhm wrote: > Before removig nls, I would like to unify all error messages in > - include comment > - libc comment > - nls C > - man page > > The EPROGUNAVAIL change is just beautification. Any objections > against that? OK millert@ - todd

Re: syslogd pledge rpath cacert

2015-10-24 Thread Todd C. Miller
On Sat, 24 Oct 2015 01:03:28 +0200, Alexander Bluhm wrote: > When loading the CA certificates at startup fails, the syslogd child > tries to load the default CA file when it connects to a TLS server. > This always fails, as the child is chrooted to /var/empty. Set the > CA storage to an empty str

if_ref -> if_get

2015-10-24 Thread Martin Pieuchot
Some trivial conversions, ok? Index: netinet/ip_output.c === RCS file: /cvs/src/sys/netinet/ip_output.c,v retrieving revision 1.303 diff -u -p -r1.303 ip_output.c --- netinet/ip_output.c 20 Oct 2015 20:22:42 - 1.303 +++ netin

Re: pair(4) (was: connect routing domains on layer 2)

2015-10-24 Thread Reyk Floeter
On Sat, Oct 24, 2015 at 06:12:44AM -0400, Ted Unangst wrote: > Reyk Floeter wrote: > > Hi, > > > > as requested by Theo and discussed with many, the following diff moves > > it into a new driver. This also allowed to improve the logic of link > > states related to the connection (as discussed wit

Re: unify errno messages

2015-10-24 Thread Alexander Bluhm
On Sat, Oct 24, 2015 at 02:54:54AM +0159, Jiri Navratil wrote: > Hi Alexander, > > On Sat, Oct 24, 2015 at 12:41:18AM +0200, Alexander Bluhm wrote: > > -#defineEOVERFLOW 87 /* Conversion overflow */ > > +#defineEOVERFLOW 87 /* Value too large to be stored in d

Re: pair(4) (was: connect routing domains on layer 2)

2015-10-24 Thread Ted Unangst
Reyk Floeter wrote: > Hi, > > as requested by Theo and discussed with many, the following diff moves > it into a new driver. This also allowed to improve the logic of link > states related to the connection (as discussed with Claudio). > > The new driver is called pair(4). > > # ifconfig

Re: unify errno messages

2015-10-24 Thread Ted Unangst
Alexander Bluhm wrote: > Hi, > > Before removig nls, I would like to unify all error messages in > - include comment > - libc comment > - nls C > - man page > > The EPROGUNAVAIL change is just beautification. Any objections > against that? ok