You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Jeff Trawick <tr...@gmail.com> on 2009/09/21 13:06:06 UTC

Re: svn commit: r817064 - in /httpd/httpd/branches/2.2.x: STATUS modules/aaa/mod_authnz_ldap.c

On Mon, Sep 21, 2009 at 6:54 AM, Graham Leggett <mi...@sharp.fm> wrote:

> Ruediger Pluem wrote:
>
> >> +    /* pre-scan for ldap-* requirements so we can get out of the way
> early */
> >> +    for(x=0; x < reqs_arr->nelts; x++) {
> >
> > Why do we know that reqs_arr != NULL always?
>
> Wasn't that the FIXME comment that was included in the previous version
> of this patch?
>

I hope this is the answer ;)

http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?r1=815281&r2=815280&pathrev=815281

RE: svn commit: r817064 - in /httpd/httpd/branches/2.2.x: STATUS modules/aaa/mod_authnz_ldap.c

Posted by "Plüm, Rüdiger, VF-Group" <ru...@vodafone.com>.
Yes, this answers my question. Thanks for the pointer.
 
Regards
 
Rüdiger


________________________________

	From: Jeff Trawick [mailto:trawick@gmail.com] 
	Sent: Montag, 21. September 2009 13:06
	To: dev@httpd.apache.org
	Subject: Re: svn commit: r817064 - in /httpd/httpd/branches/2.2.x: STATUS modules/aaa/mod_authnz_ldap.c
	
	
	On Mon, Sep 21, 2009 at 6:54 AM, Graham Leggett <mi...@sharp.fm> wrote:
	

		Ruediger Pluem wrote:
		
		>> +    /* pre-scan for ldap-* requirements so we can get out of the way early */
		>> +    for(x=0; x < reqs_arr->nelts; x++) {
		>
		> Why do we know that reqs_arr != NULL always?
		
		
		Wasn't that the FIXME comment that was included in the previous version
		of this patch?
		


	I hope this is the answer ;)
	
	http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?r1=815281&r2=815280&pathrev=815281