]> git.sur5r.net Git - openldap/commitdiff
ITS#7580 Fix autogroup when no attrs in URI.
authorLandry Breuil <breuil@craig.fr>
Thu, 13 Jun 2013 12:14:54 +0000 (14:14 +0200)
committerHallvard Furuseth <hallvard@openldap.org>
Thu, 13 Jun 2013 13:16:54 +0000 (15:16 +0200)
Do not use lvals[], lnvals[] out of scope.

contrib/slapd-modules/autogroup/autogroup.c

index 06e140789f1ac65558c2c91282122fea3153cec3..0be0f281f3fd38b8b22caa4d77a5a625610ee25b 100644 (file)
@@ -331,6 +331,7 @@ autogroup_member_search_cb( Operation *op, SlapReply *rs )
                const char              *text = NULL;
                char                    textbuf[1024];
                struct berval           *vals, *nvals;
+               struct berval           lvals[ 2 ], lnvals[ 2 ];
                int                     numvals;
 
                Debug(LDAP_DEBUG_TRACE, "==> autogroup_member_search_cb <%s>\n",
@@ -347,7 +348,6 @@ autogroup_member_search_cb( Operation *op, SlapReply *rs )
                                return 0;
                        }
                } else {
-                       struct berval           lvals[ 2 ], lnvals[ 2 ];
                        lvals[ 0 ] = rs->sr_entry->e_name;
                        BER_BVZERO( &lvals[ 1 ] );
                        lnvals[ 0 ] = rs->sr_entry->e_nname;
@@ -387,6 +387,7 @@ autogroup_member_search_modify_cb( Operation *op, SlapReply *rs )
                autogroup_filter_t      *agf = agg->agg_filter;
                Modifications           *modlist;
                struct berval           *vals, *nvals;
+               struct berval           lvals[ 2 ], lnvals[ 2 ];
                int                     numvals;
 
                Debug(LDAP_DEBUG_TRACE, "==> autogroup_member_search_modify_cb <%s>\n",
@@ -403,7 +404,6 @@ autogroup_member_search_modify_cb( Operation *op, SlapReply *rs )
                                return 0;
                        }
                } else {
-                       struct berval           lvals[ 2 ], lnvals[ 2 ];
                        lvals[ 0 ] = rs->sr_entry->e_name;
                        BER_BVZERO( &lvals[ 1 ] );
                        lnvals[ 0 ] = rs->sr_entry->e_nname;