EDNS0 and getrrsetbyname(3)

It is my understanding of the code that ssh(1) no longer supports DNSSEC. I say this because it calls getrrsetbyname(3) to check the secure flag, which calls into the ASR machine. In asr.c, the `pass0' function only parses two options, tcp and ndots:, ignoring edns0. Indeed, the asr_run(3) man

Documentation fix for cwmrc(5)

This is essentially the opposite of this fix by Holger Mikolon, which was never merged: http://marc.info/?l=openbsd-techm=127765978812199 cwmrc(5) autogroup takes the windowname and windowclass in the opposite order than specified in the man page. Fix the man page. Index: app/cwm/cwmrc.5

Re: Documentation fix for cwmrc(5)

On 2015-05-16 16.54.18 -0400, Okan Demirmen wrote: On Fri, May 15, 2015 at 3:11 AM, Mike Burns mike+open...@mike-burns.com wrote: cwmrc(5) autogroup takes the windowname and windowclass in the opposite order than specified in the man page. Fix the man page. The man page should

Re: OpenSMTPD with fewer semicolons

On 2015-04-04 18.24.38 -0700, Philip Guenther wrote: the original diff is fine and should be applied. Thanks. Any other OKs/anyone want to apply this? -Mike

OpenSMTPD with fewer semicolons

Running into issues with the extra semicolons in OpenSMTPD's Received header. I see that these semicolons are now removed in -portable[1]. Any reason not to get rid of them in base, too? [1]: https://github.com/OpenSMTPD/OpenSMTPD/pull/520 Index: usr.sbin/smtpd/smtp_session.c

Re: OpenSMTPD with fewer semicolons

On 2015-04-04 17.27.47 -0700, Philip Guenther wrote: On Sat, Apr 4, 2015 at 5:19 PM, Mike Burns mike+open...@mike-burns.com wrote: Running into issues with the extra semicolons in OpenSMTPD's Received header. I see that these semicolons are now removed in -portable[1]. Any reason

Re: Documentation fix for cwmrc(5)

On 2015-05-16 23.49.34 -0500, Kent R. Spillner wrote: On Sat, May 16, 2015 at 05:48:24PM -0400, Okan Demirmen wrote: Ah, I suspect if you looked at the errors, it would show a syntax error with the above. Like the example in the man page, use: autogroup 4 xterm,UXTerm and see if

Re: [Patch] New item to the Migrating to OpenBSD guide

On 2015-06-29 09.52.19 +0100, Raf Czlonka wrote: That being said, FAQ is *not* an OpenBSD for former Linux users guide, FAQ 9 is titled Migrating to OpenBSD, four of the five sections have Linux in the title, and the one section that does not mentions Linux twice in the first paragraph. Are you

Re: [Patch] New item to the Migrating to OpenBSD guide

On 2015-06-29 00.03.09 +0300, li...@wrant.com wrote: And you consider this a service to other Linux long time users? Or a way to try push some notion of yours - criticise and try to lobby for some other entity's interests. Do you have a patch that achieves the same goal (that is, the goal he

Re: NTRU Open Source Project / Post-quantum era

On 2015-05-23 05.24.30 -0400, ertetlen barmok wrote: https://github.com/NTRUOpenSourceProject https://github.com/NTRUOpenSourceProject/ntru-crypto/blob/master/LICENSE.md NTRU cryptographic IP and reference software may be used and modified to the needs of the user as long as the user adheres to

Re: Another use of areallocarray() in ksh

On 2015-11-01 10.44.45 -0500, Michael McConville wrote: > Index: history.c > === > RCS file: /cvs/src/bin/ksh/history.c,v > retrieving revision 1.52 > diff -u -p -r1.52 history.c > --- history.c 1 Nov 2015 15:38:53 - 1.52 >

Re: wc does not count last line if no trailing newline present

On 2015-11-04 01.05.38 -0500, dan mclaughlin wrote: > On Tue, 3 Nov 2015 21:01:36 -0800 Philip Guenther wrote: > > ...but it's not a bug. wc's line count is required by POSIX to be a > > count of the number of newline characters in the involved input. > > out of curiosity,

Re: tame userland diff

Before, from the tame patch in snapshots: $ dc /dev/null Killed $ After the below patch: $ dc /dev/null $ Found via the regress tests. Index: dc.c === RCS file: /cvs/src/usr.bin/dc/dc.c,v retrieving revision 1.13 diff -u -p

Re: tame userland diff

On 2015-10-03 09.53.54 -0600, Theo de Raadt wrote: > > Before, from the tame patch in snapshots: > > > > $ dc /dev/null > > Killed > > $ > > > > After the below patch: > > > > $ dc /dev/null > > $ > > > > Found via the regress tests. > > I don't know why you added "proc". I don't see a need

Re: tame userland diff

Fix tame(2) for patch(1). To recreate: /usr/src/regress/usr.bin/diff$ cat t2.1 Below is an example license to be used for new code in OpenBSD, modeled after the ISC license. It is important to specify the year of the copyright. Additional years should be separated by a comma, e.g. Copyright

tame(2) nologin(8)

- Use stdio instead of rw because of mprotect(2) in atexit. - Pass the path to the file that can be opened. - Switch to return instead of exit(3) in the nearby code. I suspect that the paths argument is unused or not yet ready, but I include in here regardless merely so that I can ask about it.

Re: tame(2) nologin(8)

On 2015-10-04 01.33.12 +0200, Mike Burns wrote: > I suspect that the paths argument is unused or not yet ready, but I > include in here regardless merely so that I can ask about it. Consolidated feedback from off-list: The paths argument is not yet to be used. Also, it must be NULL-terminated.

Re: tame userland diff

On 2015-10-04 07.15.47 +0200, Sebastien Marie wrote: > On Sat, Oct 03, 2015 at 09:52:13PM +0200, Mike Burns wrote: > > On 2015-10-03 09.53.54 -0600, Theo de Raadt wrote: > > > > > > I don't know why you added "proc". I don't see a need for it. Do

Re: tame userland diff

I see that I am too late: http://marc.info/?l=openbsd-cvs=144388023505289=2 http://marc.info/?l=openbsd-cvs=144388037405304=2 On 2015-10-03 22.44.22 +0200, Mike Burns wrote: > Fix tame(2) for patch(1). To recreate: > > /usr/src/regress/usr.bin/diff$ cat t2.1 > Below is an exa

Re: multitouch support again

On 2016-03-24 22.34.12 +0100, Ulf Brosziewski wrote: > Are there any reviews, tests, OKs, or NOs pending? I'll add on to the list of: no regressions. I do not have the applicable harware to try the multitouch. (My status is unchanged: the touch screen works fine until suspend/resume, and then X

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

On 2016-09-11 16.14.44 +0200, Frank Groeneveld wrote: > I've also made some progress regarding the scaling bug. It seems the > device is not properly detached from Xorg when it is unplugged, because > the log starts getting spammed with loads of messages like this: > > [ 202.482] (EE) ws:

Re: background scan for iwn(4)

On 2017-12-13 17.17.41 +0100, Stefan Sperling wrote: > Since nobody is reporting problems with iwm(4), I took some time to write the > corresponding diff for iwn(4) as well. I hope this increases test coverage :) > > Works for me on: > iwn0 at pci3 dev 0 function 0 "Intel Centrino Advanced-N

Re: enable middle button scrolling by default in ws(4) ?

On 2018-06-16 18.19.35 +0200, Matthieu Herrb wrote: > the ws(4) mouse driver has the option of emulating the mouse wheel > on motion + a given button is generated. > > This is active by default on button 2 on recent Linux systems (those > using libinput). > > Should OpenBSD's ws(4) activate it

Re: [PATCH] usr.bin/calendar/calendars/calendar.uk - Burns' Night -> Burns Night

On 2018-01-12 20.11.37 +, Jason McIntyre wrote: > > Burn's Hog Weighing Method: > > hilariously, i just realised the apostrophe in the quote above is in the > wrong place! maybe it should be "Burns Hog Weighing Method" ;) The apostrophe should come after the

Re: arc4random unexpected results

On 2018-03-24 21.05.56 -0500, Edgar Pettijohn wrote: > #include > #include > #include > > int > main(void) > { > uint32_t n = arc4random(); > > printf("arc4random %d\n", n); > printf("abs %d\n", abs(n)); > > return 0; > } > > Which is well and good. However, I get the

signify: -z implies -q

Poking around signify and learned that `verifyzdata` calls `verifymsg` with `1` hardcoded in the `quiet` parameter. I do appreciate that there is a distinction between case 'z' setting `quiet = 1` vs `verifyzdata` passing `1` as an argument, so maybe this diff doesn't quiet capture a truth:

Re: signify: -z implies -q

On 2019-07-26 03.02.43 +, Mike Burns wrote: > Poking around signify and learned that `verifyzdata` calls `verifymsg` > with `1` hardcoded in the `quiet` parameter. > > I do appreciate that there is a distinction between case 'z' setting > `quiet = 1` vs `verifyzd