Re: Remove accents from fortunes

2017-07-11 Thread Stefan Sperling
On Wed, Jul 12, 2017 at 12:07:47AM -0600, Anthony J. Bentley wrote: > Ingo Schwarze writes: > > So here is a patch that makes putting UTF-8 characters into > > fortune/datfiles safe. > > And here is a patch that retains the accents in the datfiles as UTF-8. ok

Re: remove CPU_LIDSUSPEND/machdep.lidsuspend

2017-07-11 Thread Martin Natano
On Tue, Jul 11, 2017 at 02:40:16PM -0700, Mike Larkin wrote: > On Tue, Jul 11, 2017 at 11:19:17PM +0200, Martin Natano wrote: > > Go ahead replacing machdep.lidsuspend with cpu.lidaction. OpenBSD 6.1 > > cpu.lidaction or machdep.lidaction? machdep.lidaction; I got confused by the #define name.

Re: Remove accents from fortunes

2017-07-11 Thread Anthony J. Bentley
Ingo Schwarze writes: > So here is a patch that makes putting UTF-8 characters into > fortune/datfiles safe. And here is a patch that retains the accents in the datfiles as UTF-8. Index: Notes === RCS file: /cvs/src/games/fortune/Not

SSHFP with EDNS0/DNSSEC

2017-07-11 Thread Christian Barthel
Hi, earlier this year, jca@ worked on support for DNSSEC and the EDNS0 extension [1] and committed this work at [2] (thanks!). I tried this with SSHFP records to check authenticity of hosts with DNSSEC; but ssh reported that the hostkey fingerprints were insecure. I am using this configurati

Re: first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread David Gwynne
On Tue, Jul 11, 2017 at 11:48:47AM +, Florian Obser wrote: > On Tue, Jul 11, 2017 at 11:08:23AM +0100, Stuart Henderson wrote: > > On 2017/07/11 07:45, Florian Obser wrote: > > > The way I want to move forward with this is: > > > > > > 1) generate a random key at boot if it's not present yet (

Re: remove CPU_LIDSUSPEND/machdep.lidsuspend

2017-07-11 Thread Ted Unangst
Martin Natano wrote: > Go ahead replacing machdep.lidsuspend with cpu.lidaction. OpenBSD 6.1 > contains both MIBs, so there is a clear path for migration. i have updated my sysctl.conf. fire away!

Re: remove CPU_LIDSUSPEND/machdep.lidsuspend

2017-07-11 Thread Mike Larkin
On Tue, Jul 11, 2017 at 11:19:17PM +0200, Martin Natano wrote: > Go ahead replacing machdep.lidsuspend with cpu.lidaction. OpenBSD 6.1 cpu.lidaction or machdep.lidaction? > contains both MIBs, so there is a clear path for migration. > > ok? > > natano > > > Index: arch/amd64/amd64/machdep.c >

ypldap debugging tweak

2017-07-11 Thread Kurt Mosiejczuk
While trying out ypldap (using ypldap -dv) I got a bit confused at one point because I just saw "flattening trees" and nothing else. My LDAP source does have a *lot* of users, so thought maybe it was crunching for a bit. Nope. My config was wrong and ypldap got nothing. This diff just adds a cou

remove CPU_LIDSUSPEND/machdep.lidsuspend

2017-07-11 Thread Martin Natano
Go ahead replacing machdep.lidsuspend with cpu.lidaction. OpenBSD 6.1 contains both MIBs, so there is a clear path for migration. ok? natano Index: arch/amd64/amd64/machdep.c === RCS file: /cvs/src/sys/arch/amd64/amd64/machdep.c,v

Re: rtadvd: use fatalx like normal people

2017-07-11 Thread Florian Obser
On Tue, Jul 11, 2017 at 10:26:03PM +0200, Sebastian Benoit wrote: > ok with one question. > > Florian Obser(flor...@openbsd.org) on 2017.07.11 19:29:44 +: > > @@ -444,12 +443,11 @@ parse_iflist(struct if_msghdr ***ifmlist_p, char > > *buf, size_t bufsize) > > if (ifm->ifm

Re: rtadvd: use log_warnx when errno is not set

2017-07-11 Thread Sebastian Benoit
Florian Obser(flor...@openbsd.org) on 2017.07.11 19:30:37 +: > > OK? yes > diff --git if.c if.c > index 95f83d8e822..cc878bd5916 100644 > --- if.c > +++ if.c > @@ -435,7 +435,7 @@ parse_iflist(struct if_msghdr ***ifmlist_p, char *buf, > size_t bufsize) > lim = buf + bufsize; >

Re: rtadvd: use fatalx like normal people

2017-07-11 Thread Sebastian Benoit
ok with one question. Florian Obser(flor...@openbsd.org) on 2017.07.11 19:29:44 +: > replace > log_warnx("foo"): > exit(1); > with >fatalx("foo): > > OK? > > diff --git config.c config.c > index 85d14efedca..e6f48fffec7 100644 > --- config.c > +++ config.c > @@ -85,9 +85,8 @@ get

Re: rtadvd: don't clobber errno

2017-07-11 Thread Sebastian Benoit
ok Florian Obser(flor...@openbsd.org) on 2017.07.11 19:25:58 +: > ... we want to log_warn what went wrong. > OK? > > diff --git if.c if.c > index 4683bcffac9..c4a4fea1984 100644 > --- if.c > +++ if.c > @@ -111,7 +111,7 @@ if_nametosdl(char *name) > int > if_getmtu(char *name) > { > - i

Potential DoS attack on PF due to infinite loop

2017-07-11 Thread Jingmin Zhou
Recently we discovered a potential bug in pf_lb.c. It is in the latest code that we retrieved from here: https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys/net/pf_lb.c?annotate=1.60 The problem is at line 224. When a LB rule is configured to have 65535 as the high port, and uint16 variable tmp rea

Re: relayd ipv6 ttl check_icmp / check_tcp

2017-07-11 Thread Florian Obser
commited, thanks! On Mon, Jul 10, 2017 at 06:18:03PM +0300, Kapetanakis Giannis wrote: > On 10/07/17 17:22, Jeremie Courreges-Anglas wrote: > > Using -1 for IPV6_UNICAST_HOPS is correct. > > > > Note that you can also use -1 for IP_TTL on OpenBSD, sadly some systems > > out there don't support it

Re: rtadvd: format strings for fatal{x,}

2017-07-11 Thread Sebastian Benoit
Florian Obser(flor...@openbsd.org) on 2017.07.11 19:27:54 +: > from bgpd. > > OK? ok > benno, is this the right way to do it? I kinda lost track on your > log merging efforts... yes thats ok. I neglected rtadvd a bit. > diff --git log.c log.c > index 357ef35b79c..08afcebcac0 100644 > ---

Re: Remove accents from fortunes

2017-07-11 Thread Todd C. Miller
On Tue, 11 Jul 2017 19:47:41 +0200, Ingo Schwarze wrote: > So here is a patch that makes putting UTF-8 characters into > fortune/datfiles safe. OK millert@ - todd

rtadvd: use log_warnx when errno is not set

2017-07-11 Thread Florian Obser
OK? diff --git if.c if.c index 95f83d8e822..cc878bd5916 100644 --- if.c +++ if.c @@ -435,7 +435,7 @@ parse_iflist(struct if_msghdr ***ifmlist_p, char *buf, size_t bufsize) lim = buf + bufsize; for (ifm = (struct if_msghdr *)buf; ifm < (struct if_msghdr *)lim;) { i

rtadvd: use fatalx like normal people

2017-07-11 Thread Florian Obser
replace log_warnx("foo"): exit(1); with fatalx("foo): OK? diff --git config.c config.c index 85d14efedca..e6f48fffec7 100644 --- config.c +++ config.c @@ -85,9 +85,8 @@ getconfig(char *intface) do { \ int64_t t;

rtadvd: format strings for fatal{x,}

2017-07-11 Thread Florian Obser
from bgpd. OK? benno, is this the right way to do it? I kinda lost track on your log merging efforts... diff --git log.c log.c index 357ef35b79c..08afcebcac0 100644 --- log.c +++ log.c @@ -135,26 +135,44 @@ log_debug(const char *emsg, ...) } } -void -fatal(const char *emsg) +static vo

rtadvd: don't clobber errno

2017-07-11 Thread Florian Obser
... we want to log_warn what went wrong. OK? diff --git if.c if.c index 4683bcffac9..c4a4fea1984 100644 --- if.c +++ if.c @@ -111,7 +111,7 @@ if_nametosdl(char *name) int if_getmtu(char *name) { - int s; + int s, save_errno; struct ifreqifr;

Re: Remove accents from fortunes

2017-07-11 Thread Ted Unangst
Ingo Schwarze wrote: > Hi, > > Ted Unangst wrote on Tue, Jul 11, 2017 at 09:41:36AM -0400: > > > and it always sucks to lose information if somebody went to the > > trouble of recording the necessary accents already. > > So here is a patch that makes putting UTF-8 characters into > fortune/datfi

Re: Remove accents from fortunes

2017-07-11 Thread Ingo Schwarze
Hi, Ted Unangst wrote on Tue, Jul 11, 2017 at 09:41:36AM -0400: > and it always sucks to lose information if somebody went to the > trouble of recording the necessary accents already. So here is a patch that makes putting UTF-8 characters into fortune/datfiles safe. Of course, we cannot protect

Re: first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread Florian Obser
On Tue, Jul 11, 2017 at 03:12:58PM -, Christian Weisgerber wrote: > On 2017-07-11, Florian Obser wrote: > > > + SHA512Final(digest, &ctx); > > + > > + /* assumes sizeof(digest) > sizeof(ifid) */ > > + bcopy(digest, &in6->s6_addr[8], 8); > > ... and we're finished here. yes, forgot to

Re: Remove accents from fortunes

2017-07-11 Thread Stuart Henderson
On 2017/07/11 17:13, Ingo Schwarze wrote: > Hi Stuart, > > Stuart Henderson wrote on Tue, Jul 11, 2017 at 03:52:26PM +0100: > > On 2017/07/11 16:19, Ingo Schwarze wrote: > > >> This decade feels like a strange point in time for degrading fortune > >> and calendar files by replacing UTF-8 characte

Re: first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread Christian Weisgerber
On 2017-07-11, Florian Obser wrote: > + SHA512Final(digest, &ctx); > + > + /* assumes sizeof(digest) > sizeof(ifid) */ > + bcopy(digest, &in6->s6_addr[8], 8); ... and we're finished here. > + /* make sure to set "u" bit to local, and "g" bit to individual. */ > + in6->s6_add

Re: Remove accents from fortunes

2017-07-11 Thread Ingo Schwarze
Hi Stuart, Stuart Henderson wrote on Tue, Jul 11, 2017 at 03:52:26PM +0100: > On 2017/07/11 16:19, Ingo Schwarze wrote: >> This decade feels like a strange point in time for degrading fortune >> and calendar files by replacing UTF-8 characters with ASCII >> transcriptions. Maybe such games shoul

Re: Remove accents from fortunes

2017-07-11 Thread Stuart Henderson
On 2017/07/11 16:19, Ingo Schwarze wrote: > This decade feels like a strange point in time for degrading fortune > and calendar files by replacing UTF-8 characters with ASCII > transcriptions. Maybe such games should call > > setlocale(LC_CTYPE, ""); > char *loc = nl_langinfo(CODESET); > > a

remove prefix list and default router related sysctls

2017-07-11 Thread Florian Obser
as pointed out by mpi, the kernel no longer tracks this information. OK? diff --git sys/netinet/icmp6.h sys/netinet/icmp6.h index bfa5a196c70..bf8524c2da1 100644 --- sys/netinet/icmp6.h +++ sys/netinet/icmp6.h @@ -513,9 +513,7 @@ struct icmp6stat { #define ICMPV6CTL_MTUDISC_HIWAT16 #def

Re: Remove accents from fortunes

2017-07-11 Thread Ingo Schwarze
Hi, Anthony J. Bentley wrote on Tue, Jul 11, 2017 at 02:58:08AM -0600: > "Ted Unangst" writes: >> do we want to delete them? or replace them with proper utf-8 sequences? > I wouldn't mind doing that. But fortune(6) will happily print UTF-8 to a > non-UTF-8 terminal, which though probably harmles

Re: Remove accents from fortunes

2017-07-11 Thread Reyk Floeter
I so much hate it when people drop the umlaut in my name and write "Floter" - this usually happens while being in America because people don't know better. My name is Flöter or Floeter ("ae", "oe", or "ue" are the correct substitutions). They are different letters with different pronunciations th

Re: Remove accents from fortunes

2017-07-11 Thread Ted Unangst
Anthony J. Bentley wrote: > "Ted Unangst" writes: > > Anthony J. Bentley wrote: > > > Hi, > > > > > > "Punctuation followed by backspace" hasn't been the right way to enter > > > accents for a long time. > > > > hard to see what the diff is doing since i coudn't see the backspaces. :) > > > > do

Re: Remove accents from fortunes

2017-07-11 Thread Philippe Meunier
Anthony J. Bentley wrote: >And since nobody's complained in the past few years that they couldn't >see the accents in Jabberwocky... But then some Frenchman might suddenly complain about René Descartes's name being misspelt (not that I had ever noticed that the accent didn't show up before reading

Re: first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread Florian Obser
On Tue, Jul 11, 2017 at 11:08:23AM +0100, Stuart Henderson wrote: > On 2017/07/11 07:45, Florian Obser wrote: > > The way I want to move forward with this is: > > > > 1) generate a random key at boot if it's not present yet (like we do > > for ssh host keys and ipsec) > > 2) if /etc/netstart bring

Re: [PATCH] allow notAfter after 2038 with 32-bit time_t

2017-07-11 Thread Andrew Grillet
Hi, I have built embedded systems that have run 20 years with no update - but I doubt they are common. More of an issue is archived data. I have read data 30 years old, and may well do so again. While I think the answer is "human readable date formats" The Americans have scuppered that with M-D-Y

Re: urndis issues

2017-07-11 Thread Mike Belopuhov
On Sun, Jul 09, 2017 at 09:57 +0300, Artturi Alm wrote: > Hi, > > anyone else having issues w/urndis(android)? > victim of circumstances, i have to rely on it at times during the summer. > When i plug phone into usb, and enable usb tethering or w/e it is called, > i never get ip on first try, i ha

Re: [PATCH] allow notAfter after 2038 with 32-bit time_t

2017-07-11 Thread Stuart Henderson
On 2017/07/11 01:55, Kyle J. McKay wrote: > 2) 32-bit systems are going to be around for many years still; 32-bit ARM > platforms are everywhere .. > 4) 32-bit time_t has potentially still got over 20 years of life left in it Yes. The gamble is whether 32-bit systems will still be around then. I d

Re: first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread Stuart Henderson
On 2017/07/11 07:45, Florian Obser wrote: > > Here is the first step in implementing RFC 7217 "A Method for > Generating Semantically Opaque Interface Identifiers with IPv6 > Stateless Address Autoconfiguration (SLAAC)" > > This is orthogonal to privacy extension. The idea is to replace the > sta

Re: rip router advertisement processing and solicitation sending out of kernel

2017-07-11 Thread Martin Pieuchot
On 10/07/17(Mon) 08:04, Florian Obser wrote: > --- netinet6/in6.c > +++ netinet6/in6.c > @@ -897,25 +897,9 @@ in6_unlink_ifa(struct in6_ifaddr *ia6, struct ifnet *ifp) > { > struct ifaddr *ifa = &ia6->ia_ifa; > extern int ifatrash; > - int plen; > > NET_ASSERT_LOCKED(); >

Re: backgammon(6): remove compiled out user count checks

2017-07-11 Thread Mike Larkin
On Tue, Jul 11, 2017 at 11:01:32AM +0200, Frederic Cambus wrote: > Hi tech@, > > The user count checks have been compiled out since 1998, let's get rid > of them. > > Comments? OK? > ok mlarkin > Index: games/backgammon/backgammon/main.c > ==

backgammon(6): remove compiled out user count checks

2017-07-11 Thread Frederic Cambus
Hi tech@, The user count checks have been compiled out since 1998, let's get rid of them. Comments? OK? Index: games/backgammon/backgammon/main.c === RCS file: /cvs/src/games/backgammon/backgammon/main.c,v retrieving revision 1.23 d

Re: Remove accents from fortunes

2017-07-11 Thread Anthony J. Bentley
"Ted Unangst" writes: > Anthony J. Bentley wrote: > > Hi, > > > > "Punctuation followed by backspace" hasn't been the right way to enter > > accents for a long time. > > hard to see what the diff is doing since i coudn't see the backspaces. :) > > do we want to delete them? or replace them with

Re: [PATCH] allow notAfter after 2038 with 32-bit time_t

2017-07-11 Thread Kyle J. McKay
On Jul 5, 2017, at 22:30, Bob Beck wrote: On Thu, May 18, 2017 at 7:31 AM, Kyle J. McKay wrote: RFC 5280 section 4.1.2.5 states: To indicate that a certificate has no well-defined expiration date, the notAfter SHOULD be assigned the GeneralizedTime value of 1231235959Z. T

first stab at RFC 7217: random but stable link local addresses

2017-07-11 Thread Florian Obser
Here is the first step in implementing RFC 7217 "A Method for Generating Semantically Opaque Interface Identifiers with IPv6 Stateless Address Autoconfiguration (SLAAC)" This is orthogonal to privacy extension. The idea is to replace the stable slaac addresses formed from the MAC address of your

Re: Remove accents from fortunes

2017-07-11 Thread Ted Unangst
Anthony J. Bentley wrote: > Hi, > > "Punctuation followed by backspace" hasn't been the right way to enter > accents for a long time. hard to see what the diff is doing since i coudn't see the backspaces. :) do we want to delete them? or replace them with proper utf-8 sequences?