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