Re: Division by zero in ehci(4)

2016-09-18 Thread Jonathan Gray
On Sat, Sep 17, 2016 at 04:36:12PM +0200, Martin Pieuchot wrote: > One of the non-checked value read from an untrusted descriptor is the > "maximum packet size" of an endpoint. If a device reports an incorrect > value most of our HC drivers wont work and if this value is 0 ehci(4) > will crash

Re: timeout_set_proc(9)

2016-09-18 Thread David Gwynne
On Fri, Sep 16, 2016 at 04:58:39PM +0200, Mark Kettenis wrote: > > Date: Thu, 15 Sep 2016 16:29:45 +0200 > > From: Martin Pieuchot > > > > After discussing with a few people about a new "timed task" API I came > > to the conclusion that mixing timeouts and tasks will result in:

Re: LibreSSL selects weak digest for (EC)DH

2016-09-18 Thread kinichiro inoguchi
Thanks, Brent. I appreciate if you commit this. Kinichiro

Re: LibreSSL selects weak digest for (EC)DH

2016-09-18 Thread Brent Cook
Looks fine to me. Shall I commit it? On Thu, Sep 15, 2016 at 2:32 AM, Kinichiro Inoguchi < kinichiro.inogu...@gmail.com> wrote: > Sorry, I attached wrong patch file. > I re-post the patch file again. > > On Thu, Sep 15, 2016 at 04:10:55PM +0900, Kinichiro Inoguchi wrote: > > Hi, > > > > I would

Re: alignment error rtadvd/armv7

2016-09-18 Thread Martin Brandenburg
On Sun, 18 Sep 2016, Jeremie Courreges-Anglas wrote: > Martin Brandenburg writes: > > > On a PandaBoard (armv7) running -current, when I run rtadvd, it crashes > > with a bus error shortly after printing (received a routing message). I > > can reproduce by sending

Re: [PATCH] www: mention /usr/src in upgrade notes

2016-09-18 Thread Theo Buehler
On Mon, Sep 19, 2016 at 01:59:20AM +0200, Simon Ruderich wrote: > I'm a relatively new OpenBSD user and just updated my system from > 5.9 to 6.0 but forgot to update the files in /usr/src. Thus when > I applied the latest patches (001-006) I actually built an old > 5.9 kernel which failed to boot.

Re: little simpler ssh code

2016-09-18 Thread Darren Tucker
On Mon, Sep 19, 2016 at 5:41 AM, Martin Natano wrote: > Two more loops that can be converted to arc4random_buf(). Ok? > [...] > + arc4random_buf(x11_fake_data, data_len); > for (i = 0; i < data_len; i++) { > I'd put that below the for loop so it

[PATCH] www: mention /usr/src in upgrade notes

2016-09-18 Thread Simon Ruderich
Hello, I'm a relatively new OpenBSD user and just updated my system from 5.9 to 6.0 but forgot to update the files in /usr/src. Thus when I applied the latest patches (001-006) I actually built an old 5.9 kernel which failed to boot. The following patch mentions /usr/src in the upgrade notes.

Re: video(1): Don't limit maximum frame size to display size

2016-09-18 Thread Jeremie Courreges-Anglas
Gregor Best writes: > Hi, > > this is another patch to video(1). It removes the logic that forces the > video grab frame size to be <= the size of the Xv adaptor. This allows > me to grab and view frames that are 1280x1024 on my 1440x900 screen. Xv > properly scales and

Re: Manpage for uctuctl(4)

2016-09-18 Thread Ingo Schwarze
Hi Rafael, Rafael Neves wrote on Sun, Sep 18, 2016 at 04:48:37PM +0100: > I have to admit that the idea of derived work in such short > content manpage is very ethereal for me. :) I think you are not alone. If i understand correctly, disputes over Copyright of small amounts of content end up

video(1): use _NET_WM_STATE_FULLSCREEN

2016-09-18 Thread Dmitrij D. Czarkoff
Hi! The diff below fixes fullscreen mode on window managers that prevent applications from resizing their windows. This is done by setting _NET_WM_STATE_FULLSCREEN atom. -- Dmitrij D. Czarkoff Index: video.c === RCS file:

Re: little simpler ssh code

2016-09-18 Thread Martin Natano
Two more loops that can be converted to arc4random_buf(). Ok? natano Index: channels.c === RCS file: /cvs/src/usr.bin/ssh/channels.c,v retrieving revision 1.352 diff -u -p -r1.352 channels.c --- channels.c 12 Sep 2016 01:22:38

Re: teach BFD how to send route messages, again

2016-09-18 Thread Peter Hessler
On 2016 Sep 17 (Sat) at 13:15:56 +0200 (+0200), Peter Hessler wrote: :On 2016 Sep 17 (Sat) at 04:50:29 -0600 (-0600), Theo de Raadt wrote: ::> route(8) for the ramdisks is not built with SMALL, so adding SMALL ::> won't help. :: ::Then you'll help it. By compiling them SMALL. : :Happilly. :

netinet/tcp_input.c syn_cache_get diff

2016-09-18 Thread David Hill
Hello - Make sure we keep TF_NOPUSH set if TCP_NOPUSH was set. FreeBSD has the same: https://github.com/freebsd/freebsd/blob/c9af4f2541fd437e0805365fbeec46d69e033310/sys/netinet/tcp_syncache.c#L860 Index: netinet/tcp_input.c ===

Re: Manpage for uctuctl(4)

2016-09-18 Thread Ingo Schwarze
Hi Rafael, Rafael Neves wrote on Sun, Sep 18, 2016 at 12:29:35PM +0100: > On Sun, Sep 18, 2016 at 03:33:00PM +0200, Ingo Schwarze wrote: >> - Put the correct manual page author into the Copyright notice. > I think that I shouldn't be in the copyright notice, because thre > is no original work

Re: Manpage for uctuctl(4)

2016-09-18 Thread Visa Hankala
On Sun, Sep 18, 2016 at 03:33:00PM +0200, Ingo Schwarze wrote: > So, here is a cleaned-up version: > > - Move the new page to the proper directory. > - Mention it in the Makefile. > - Put the correct manual page author into the Copyright notice. > - Add the architecture to the .Dt line. > -

Re: random malloc junk

2016-09-18 Thread Otto Moerbeek
On Wed, Sep 14, 2016 at 05:41:55AM +0200, Theo Buehler wrote: > On Tue, Sep 13, 2016 at 01:29:17PM +0200, Otto Moerbeek wrote: > > As often, real life came in between. Did anybody do measurements? I > > really would like to to see hard data. > > It seems that the price is relatively modest. >

Re: merge ping6(8) into ping(8)

2016-09-18 Thread Theo de Raadt
> On Sun, Sep 18, 2016 at 12:11 AM, Theo de Raadt wrote: > >> > this does 2 things: > >> > [...] > >> > >> I may recall what I have sent to you in private email, excerpt from > >> FreeBSD ping6 manpage: > >> [...] > >> When we have two binaries I have more trust when one of

Re: Manpage for uctuctl(4)

2016-09-18 Thread Rafael Neves
Hi, On Sun, Sep 18, 2016 at 03:33:00PM +0200, Ingo Schwarze wrote: > Hi, > > Mike Belopuhov wrote on Sun, Sep 18, 2016 at 01:35:45PM +0200: > > On Sun, Sep 18, 2016 at 06:15 +, Visa Hankala wrote: > >> On Sat, Sep 17, 2016 at 02:40:09PM +0100, Rafael Neves wrote: > > >>> Here follows a

video(1): Don't limit maximum frame size to display size

2016-09-18 Thread Gregor Best
Hi, this is another patch to video(1). It removes the logic that forces the video grab frame size to be <= the size of the Xv adaptor. This allows me to grab and view frames that are 1280x1024 on my 1440x900 screen. Xv properly scales and stretches the window content, so there's nothing that gets

Re: Manpage for uctuctl(4)

2016-09-18 Thread Ingo Schwarze
Hi, Mike Belopuhov wrote on Sun, Sep 18, 2016 at 01:35:45PM +0200: > On Sun, Sep 18, 2016 at 06:15 +, Visa Hankala wrote: >> On Sat, Sep 17, 2016 at 02:40:09PM +0100, Rafael Neves wrote: >>> Here follows a manpage for octuctl(4), based on dwctwo(4) manpage. >>> I am not sure if the title

Re: Manpage for uctuctl(4)

2016-09-18 Thread Mike Belopuhov
On Sun, Sep 18, 2016 at 06:15 +, Visa Hankala wrote: > On Sat, Sep 17, 2016 at 02:40:09PM +0100, Rafael Neves wrote: > > Hi, > > > > Here follows a manpage for octuctl(4), based on dwctwo(4) manpage. > > I am not sure if the title should stop in Controller or in Interface, > > so I mantained

Re: arm libunwind

2016-09-18 Thread Patrick Wildt
On Sun, Sep 18, 2016 at 08:16:42AM +0200, Mark Kettenis wrote: > This makes libunwind build on OpenBSD/arm. And it even works if you > compile the code that uses exceptions with the right compiler options. > > ok? ok patrick@ > > Index: src/AddressSpace.hpp >

Re: arm assembly syntax

2016-09-18 Thread Jonathan Gray
On Sun, Sep 18, 2016 at 12:04:48PM +0200, Mark Kettenis wrote: > Somewhere down the line the ARM people made some subtle changes to > their assembly syntax. The new syntax is called Unified Assembler > Language (UAL), and tis is what clang supports. But gas defaults to > the old pre-UAL dialect.

arm assembly syntax

2016-09-18 Thread Mark Kettenis
Somewhere down the line the ARM people made some subtle changes to their assembly syntax. The new syntax is called Unified Assembler Language (UAL), and tis is what clang supports. But gas defaults to the old pre-UAL dialect. The easiest way to make code compile with both clang and gcc is to

Re: merge ping6(8) into ping(8)

2016-09-18 Thread Stuart Henderson
On 2016/09/18 00:04, Mikhail wrote: > > this does 2 things: > > [...] > > I may recall what I have sent to you in private email, excerpt from FreeBSD > ping6 manpage: > > 8<8<8<8<8<8< > There have been many discussions on why we separate

Re: merge ping6(8) into ping(8)

2016-09-18 Thread Mikhail
On Sun, Sep 18, 2016 at 12:11 AM, Theo de Raadt wrote: >> > this does 2 things: >> > [...] >> >> I may recall what I have sent to you in private email, excerpt from >> FreeBSD ping6 manpage: >> [...] >> When we have two binaries I have more trust when one of them is working