Re: Do not hide actual error for setusercontext

2015-07-28 Thread Ted Unangst
Vadim Zhukov wrote: I see no point in hidding underlying error message from user. Found while preparing chroot for doas run tests. Okay? setusercontext(3) doesn't say anything about setting errno, so it's not clear it will always be set or to a correct value.

Re: Do not hide actual error for setusercontext

2015-07-28 Thread Marc Espie
On Tue, Jul 28, 2015 at 04:10:37PM +0300, Vadim Zhukov wrote: I see no point in hidding underlying error message from user. Found while preparing chroot for doas run tests. Okay? -- WBR, Vadim Zhukov Index: doas.c

Do not hide actual error for setusercontext

2015-07-28 Thread Vadim Zhukov
I see no point in hidding underlying error message from user. Found while preparing chroot for doas run tests. Okay? -- WBR, Vadim Zhukov Index: doas.c === RCS file: /cvs/src/usr.bin/doas/doas.c,v retrieving revision 1.27 diff

Re: Do not hide actual error for setusercontext

2015-07-28 Thread Vadim Zhukov
2015-07-28 16:59 GMT+03:00 Ted Unangst t...@tedunangst.com: Vadim Zhukov wrote: I see no point in hidding underlying error message from user. Found while preparing chroot for doas run tests. Okay? setusercontext(3) doesn't say anything about setting errno, so it's not clear it will always