Re: svn commit: r1913977 - /httpd/httpd/trunk/modules/aaa/mod_authnz_ldap.c

[Ruediger Pluem]

On 11/24/23 10:45 AM, Graham Leggett via dev wrote:

> 
>  
> 
>          /* Make sure that the filtered search returned a single dn */
> 
>          if (result == LDAP_SUCCESS && dn) {
> 
> +            req->dn = dn;

Don't we need to save the vals as well?

Index: modules/aaa/mod_authnz_ldap.c
===================================================================
--- modules/aaa/mod_authnz_ldap.c       (revision 1914069)
+++ modules/aaa/mod_authnz_ldap.c       (working copy)
@@ -1482,10 +1482,12 @@

         /* Make sure that the filtered search returned a single dn */
         if (result == LDAP_SUCCESS && dn) {
+            req->dn = dn;
+            req->vals = vals;
             ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(02631)
                           "auth_ldap authorize: require ldap-search: "
                           "authorization successful");




Regards

Rüdiger