Re: src/usr.sbin/slowcgi: possible bug

2017-01-13 Thread Florian Obser
On Mon, Jan 02, 2017 at 04:29:21PM +0330, temp+...@frad.ir wrote: > Hi tech@, > > I recently checked the slowcgi(8) and found that it might have an issue > when buf_pos is at the end of buffer and buf_len is zero. > > Am I right? we can simplify this even more. There is no need to remember the b

Re: traceroute(8): make progress on timeout

2017-01-13 Thread Todd C. Miller
On Fri, 13 Jan 2017 16:07:31 +, Florian Obser wrote: > traceroute(8) never sees a timeout when poll(2) returns when it receives > a packet not intended for us. E.g. a ping(8) is running in parallel. > In this case we need to account for the time we already waited. Looks good but please make w

Re: Move auth_approval in su.c before fork is lost due to pledge?

2017-01-13 Thread Todd C. Miller
One change with this diff is that the approval script will run as the invoking user, not the target user. I'm not sure that really makes a difference though. - todd

traceroute(8): make progress on timeout

2017-01-13 Thread Florian Obser
traceroute(8) never sees a timeout when poll(2) returns when it receives a packet not intended for us. E.g. a ping(8) is running in parallel. In this case we need to account for the time we already waited. Pointed out by Gabriel Nieto on bugs@, thanks! OK? diff --git traceroute.c traceroute.

Re: Move auth_approval in su.c before fork is lost due to pledge?

2017-01-13 Thread Todd C. Miller
On 07 Jan 2017 21:14:17 -0700, "Andy Bradford" wrote: > As it turns out, it is because I have an approve entry in > /etc/login.conf and this requires the ability to fork the approval > program. When su tries to run approve it fails and I find the following > in dmesg: > > su

Re: FreeType 2.7.1

2017-01-13 Thread Christian Weisgerber
Matthieu Herrb: > In my analysis of the changed symbols, only a minor bump was required, > but now that people have started installing libfreetype.so.28.0, > commit it like that. FWIW, I run such testing bulk builds without plist registration, don't upload the resulting packages, and wipe the aff

Re: find -delete

2017-01-13 Thread Dmitrij D. Czarkoff
"Ted Unangst" wrote: > This option is not posix (not like that's stopped find accumulating a > dozen extensions), but it is in gnu and freebsd (for 20 years). it's > also somewhat popular among sysadmins and blogs, etc. and perhaps most > importantly, it nicely solves one of the more troublesome

Re: pfkey vs splsoftnet()

2017-01-13 Thread Hrvoje Popovski
On 12.1.2017. 18:27, Hrvoje Popovski wrote: > On 12.1.2017. 16:20, Martin Pieuchot wrote: >> On 10/01/17(Tue) 10:37, Martin Pieuchot wrote: >>> In pfkey_sendup() we call sorwakeup() which asserts for NET_LOCK(), so >>> we are already at IPL_SOFTNET. >>> >>> pfkeyv2_send() is called via pfkey_output

Re: FreeType 2.7.1

2017-01-13 Thread Matthieu Herrb
On Thu, Jan 12, 2017 at 02:17:02PM +0100, David Coppa wrote: > On Thu, Jan 12, 2017 at 1:46 PM, Stuart Henderson > wrote: > > On 2017/01/09 15:29, David Coppa wrote: > >> Here's the update to freetype-2.7.1, both inline and as attachment. > > > >> As usual, please test it. > >> > >> And, if you c

Re: ksh(1): preserve xtrace option

2017-01-13 Thread Otto Moerbeek
On Fri, Jan 13, 2017 at 10:20:19AM +0100, Otto Moerbeek wrote: > On Fri, Jan 13, 2017 at 09:51:51AM +0100, Otto Moerbeek wrote: > > > On Fri, Jan 13, 2017 at 09:46:54AM +0100, Anton Lindqvist wrote: > > > > > Consider the following script which calculates the sum of the first N > > > integers re

Re: ksh(1): preserve xtrace option

2017-01-13 Thread Otto Moerbeek
On Fri, Jan 13, 2017 at 09:51:51AM +0100, Otto Moerbeek wrote: > On Fri, Jan 13, 2017 at 09:46:54AM +0100, Anton Lindqvist wrote: > > > Consider the following script which calculates the sum of the first N > > integers recursively: > > > > $ cat >sum.sh < > sum() { > > [ $1 -eq 0 ] && echo $2

Re: ksh(1): preserve xtrace option

2017-01-13 Thread Otto Moerbeek
On Fri, Jan 13, 2017 at 09:46:54AM +0100, Anton Lindqvist wrote: > Consider the following script which calculates the sum of the first N > integers recursively: > > $ cat >sum.sh < sum() { > [ $1 -eq 0 ] && echo $2 || sum $(($1 - 1)) $(($2 + $1)) > } > > sum 5 > ! > > Executing the script wit

ksh(1): preserve xtrace option

2017-01-13 Thread Anton Lindqvist
Consider the following script which calculates the sum of the first N integers recursively: $ cat >sum.sh