Re: svn commit: r965709 - in /httpd/httpd/trunk: CHANGES server/request.c

2010-07-20 Thread Stefan Fritsch
On Tuesday 20 July 2010, Nick Kew wrote: > On 20 Jul 2010, at 07:43, Ruediger Pluem wrote: > > Is this needed any longer? Shouldn't authz be expected to handle > > r->user == NULL after Stefans recent changes? I have reverted the change that the auth_checker hook is also called with r->user == NU

Re: svn commit: r965709 - in /httpd/httpd/trunk: CHANGES server/request.c

2010-07-20 Thread Nick Kew
On 20 Jul 2010, at 07:43, Ruediger Pluem wrote: > Is this needed any longer? Shouldn't authz be expected to handle r->user == > NULL after Stefans > recent changes? Hadn't considered that. An extra check here can't hurt, can it? -- Nick Kew

Re: svn commit: r965709 - in /httpd/httpd/trunk: CHANGES server/request.c

2010-07-19 Thread Ruediger Pluem
On 07/20/2010 03:34 AM, n...@apache.org wrote: > Author: niq > Date: Tue Jul 20 01:34:39 2010 > New Revision: 965709 > > URL: http://svn.apache.org/viewvc?rev=965709&view=rev > Log: > Don't risk segfault in authz if r->user is not set > PR 42995 > > Modified: > httpd/httpd/trunk/CHANGES >