Make fstat exit status like grep has

2016-05-03 Thread Vadim Zhukov
This makes fstat(1) behave more like grep(1), regarding exit status. The idea itself came from bluhm@. For manpage, I mostly copied the text from grep(1), but changed phrases talking about selected lines to matching processes. Or should those be selected processes?.. -- WBR, Vadim Zhukov

Re: libcrypto errata

2016-05-03 Thread Ted Unangst
Ted Unangst wrote: > http://ftp.openbsd.org/pub/OpenBSD/patches/5.9/common/005_crypto.patch.sig There is an additional chunk in this diff, for s3_pkt.c, that should have not been included. It adds a memset that will zero a buffer after libssl is done using it to prevent info leaks. As far as I

Re: MSI-X support

2016-05-03 Thread Mike Larkin
On Tue, May 03, 2016 at 09:40:28PM +0200, Mark Kettenis wrote: > Today mpi@ reminded me that I had written support for MSI-X some time > ago. Since he is interested in using multiple vectors, I extended the > code I had a bit to support that feature as well. This introduces a > new function: >

Re: Remove lib/libc/regex/WHATSNEW

2016-05-03 Thread Todd C. Miller
On Tue, 03 May 2016 23:21:17 +0300, Vadim Zhukov wrote: > Is there any point in keeping lib/libc/regex/WHATSNEW? Okay to rm? No point, OK millert@ to rm. - todd

Re: Return ESRCH instead of zero result for KERN_FILE sysctl

2016-05-03 Thread Ted Unangst
Alexander Bluhm wrote: > On Tue, May 03, 2016 at 12:20:34AM -0400, Ted Unangst wrote: > > Alexander Bluhm wrote: > > > > > > Can we have the same logic for KERN_FILE_BYPID and KERN_FILE_BYUID? > > > > I think this should be BYPID only. I'm not sure why. Maybe because a user > > exists even when

Re: Return ESRCH instead of zero result for KERN_FILE sysctl

2016-05-03 Thread Alexander Bluhm
On Tue, May 03, 2016 at 12:20:34AM -0400, Ted Unangst wrote: > Alexander Bluhm wrote: > > > > Can we have the same logic for KERN_FILE_BYPID and KERN_FILE_BYUID? > > I think this should be BYPID only. I'm not sure why. Maybe because a user > exists even when the user isn't running any processes.

Remove lib/libc/regex/WHATSNEW

2016-05-03 Thread Vadim Zhukov
Is there any point in keeping lib/libc/regex/WHATSNEW? Okay to rm? -- WBR, Vadim Zhukov Index: WHATSNEW === RCS file: WHATSNEW diff -N WHATSNEW --- WHATSNEW28 Apr 1997 20:44:56 - 1.3 +++ /dev/null 1 Jan 1970

Re: Return ESRCH instead of zero result for KERN_FILE sysctl

2016-05-03 Thread Alexander Bluhm
On Tue, May 03, 2016 at 10:47:51PM +0300, Vadim Zhukov wrote: > @@ -333,10 +336,12 @@ kvm_deadfile_byid(kvm_t *kd, int op, int > goto cleanup; > } > > - if (op == KERN_FILE_BYPID && arg > 0 && > - proc.p_pid != (pid_t)arg) { > -

Re: Return ESRCH instead of zero result for KERN_FILE sysctl

2016-05-03 Thread Mark Kettenis
> Date: Tue, 3 May 2016 22:47:51 +0300 (MSK) > Cc: tech@openbsd.org > > So, after initial input, here is improved version. Changelist: > > - Simplified handling of "matched" variable in sysctl_file(), > from kettenis@. > > - ERRORS section for kvm_getfiles.2 (to be committed

Re: Return ESRCH instead of zero result for KERN_FILE sysctl

2016-05-03 Thread Vadim Zhukov
So, after initial input, here is improved version. Changelist: - Simplified handling of "matched" variable in sysctl_file(), from kettenis@. - ERRORS section for kvm_getfiles.2 (to be committed separately), requested by bluhm@. I also moved beginning of RETURN VALUES section a

Re: smu(4) fan speed

2016-05-03 Thread Mark Kettenis
> Date: Tue, 3 May 2016 21:20:55 +0200 > From: Marcus Glocker > > On Mon, Apr 18, 2016 at 10:31:27AM +0200, Martin Pieuchot wrote: > > > On 18/04/16(Mon) 00:22, Marcus Glocker wrote: > > > On Sun, Apr 17, 2016 at 11:33:18PM +0200, Olivier Cherrier wrote: > > > > > > > With

Re: sqlite c api manpages?

2016-05-03 Thread Kristaps Dzonsons
>> We should provide documentation for at least the programmatic aspects >> of it. (The full extent of SQL language documentation may be beyond >> the scope of base, but C APIs are not.) > > I doubt it's worth wasting time on. Ingo, I can't speak for sqlite and inclusion (what is this `less

MSI-X support

2016-05-03 Thread Mark Kettenis
Today mpi@ reminded me that I had written support for MSI-X some time ago. Since he is interested in using multiple vectors, I extended the code I had a bit to support that feature as well. This introduces a new function: int pci_intr_map_msix(struct pci_attach_args *, int, pci_intr_handle_t

Re: smu(4) fan speed

2016-05-03 Thread Marcus Glocker
On Mon, Apr 18, 2016 at 10:31:27AM +0200, Martin Pieuchot wrote: > On 18/04/16(Mon) 00:22, Marcus Glocker wrote: > > On Sun, Apr 17, 2016 at 11:33:18PM +0200, Olivier Cherrier wrote: > > > > > With this diff, the fans go at their maximun speed and stay like that: > > > $ sysctl hw.sensors > >

Re: mklocale(1): TODIGIT > 255 clobbers type and width data

2016-05-03 Thread Ingo Schwarze
Hi, here is an alternative patch simply removing the pointless and buggy code. No functional change on OpenBSD, except that unrelated functionality is no longer clobbered, of course. Do you prefer that? Yours, Ingo Index: yacc.y

Re: xclock patch

2016-05-03 Thread Sebastien Marie
On Mon, May 02, 2016 at 05:39:36PM -0500, Edgar Pettijohn wrote: > I saw this in /var/log/messages > > May 2 17:19:07 thinkpad /bsd: xclock(81091): syscall 5 "rpath" > > Does it mean xclock needs rpath? If so here you go. generally it is what it means: at some point, the program required

libcrypto errata

2016-05-03 Thread Ted Unangst
allocation (CVE-2016-2109) Thanks to OpenSSL for providing information and patches. Refer to https://www.openssl.org/news/secadv/20160503.txt Patches for OpenBSD are available: http://ftp.openbsd.org/pub/OpenBSD/patches/5.9/common/005_crypto.patch.sig http://ftp.openbsd.org/pub/OpenBSD/patches

Re: sqlite c api manpages?

2016-05-03 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Mon, May 02, 2016 at 02:46:43PM -0400: > Kristaps Dzonsons wrote: >> If you're on OpenBSD, you started with "apropos -s3 sqlite3", were >> shocked that there's nothing there, then moved on to Google with a >> wounding confusion in your heart. > Indeed. I see this

Re: apmd: report battery time left in hour-minutes format

2016-05-03 Thread Mark Kettenis
> From: Theo de Raadt > Date: Tue, 03 May 2016 06:17:43 -0600 > > I don't like this, and especially since you choce a totally nonstandard > format to show this. > > It is fine how it is right now. I like the current way as well. I fear that if we print it the way Paul

Re: apmd: report battery time left in hour-minutes format

2016-05-03 Thread Theo de Raadt
I don't like this, and especially since you choce a totally nonstandard format to show this. It is fine how it is right now.

apmd: report battery time left in hour-minutes format

2016-05-03 Thread Paul Irofti
The following diff allows apmd to report battery time left in a user friendly hour-minutes format, instead of just minutes. Before: apmd: battery status: low. external power status: not connected. estimated battery life 50% (185 minutes) After: apmd: battery status: low. external power status:

Re: bpf device nodes

2016-05-03 Thread Theo de Raadt
> On 2016-04-27, Martin Natano wrote: > > > /dev/bpf0 is for compatibility with existing binaries and is to > > be removed after a transition period. > > What's the impact on ports going to be? > > (We still have four /dev/*random devices for compatibility.) I believe we'll

Re: bpf device nodes

2016-05-03 Thread Christian Weisgerber
On 2016-04-27, Martin Natano wrote: > /dev/bpf0 is for compatibility with existing binaries and is to > be removed after a transition period. What's the impact on ports going to be? (We still have four /dev/*random devices for compatibility.) -- Christian "naddy"

Re: Keyboard resume (zzz) diff

2016-05-03 Thread sid77
- Original Message - > - Original Message - > > Since it worked before, start bisecting diffs. Once you've narrowed it > > down, > > let us know. > > Tried to do that but I can't boot past the 4th April softraid update :-/ Small update: I spent some time on this over the weekend

Re: ndp(8) CPPFLAGS

2016-05-03 Thread Sebastian Benoit
ok J??r??mie Courr??ges-Anglas(j...@wxcvbn.org) on 2016.05.02 13:21:51 +0200: > > ndp.c doesn't have any #ifdef INET6 preprocessor directive, I can't see > how keeping that in CPPFLAGS changes anything. While here, -I${.CURDIR} > isn't needed either. Verified with sha256(1). > > ok? > >

Re: pledge: telnet should not verify if hostname is a fully qualified domain

2016-05-03 Thread Theo de Raadt
>I would like to just drop that part of code. Any OK's, comments? No kidding! Kill it. >Index: commands.c >=== >RCS file: /cvs/src/usr.bin/telnet/commands.c,v >retrieving revision 1.83 >diff -u -p -r1.83 commands.c >--- commands.c