]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/overlays/dynlist.c
remove REP_ENTRY_MUSTRELEASE when copying (ITS#6423)
[openldap] / servers / slapd / overlays / dynlist.c
index ba9a9d2ac51772064c4a027733d09169e0e0ddb9..1b83f9f41d01e219c63057058138d0d88eb2373e 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2003-2008 The OpenLDAP Foundation.
+ * Copyright 2003-2009 The OpenLDAP Foundation.
  * Portions Copyright 2004-2005 Pierangelo Masarati.
  * Portions Copyright 2008 Emmanuel Dreyfus.
  * All rights reserved.
@@ -62,21 +62,25 @@ static AttributeName *slap_anlist_no_attrs = anlist_no_attrs;
 static AttributeDescription *ad_dgIdentity, *ad_dgAuthz;
 
 typedef struct dynlist_map_t {
-       AttributeDescription *dlm_member_ad;
-       AttributeDescription *dlm_mapped_ad;
-       struct dynlist_map_t *dlm_next;
+       AttributeDescription    *dlm_member_ad;
+       AttributeDescription    *dlm_mapped_ad;
+       struct dynlist_map_t    *dlm_next;
 } dynlist_map_t;
 
 typedef struct dynlist_info_t {
        ObjectClass             *dli_oc;
        AttributeDescription    *dli_ad;
        struct dynlist_map_t    *dli_dlm;
+       struct berval           dli_uri;
+       LDAPURLDesc             *dli_lud;
+       struct berval           dli_uri_nbase;
+       Filter                  *dli_uri_filter;
        struct berval           dli_default_filter;
        struct dynlist_info_t   *dli_next;
 } dynlist_info_t;
 
 #define DYNLIST_USAGE \
-       "\"dynlist-attrset <oc> <URL-ad> [[<mapped-ad>:]<member-ad> ...]\": "
+       "\"dynlist-attrset <oc> [uri] <URL-ad> [[<mapped-ad>:]<member-ad> ...]\": "
 
 static dynlist_info_t *
 dynlist_is_dynlist_next( Operation *op, SlapReply *rs, dynlist_info_t *old_dli )
@@ -102,6 +106,56 @@ dynlist_is_dynlist_next( Operation *op, SlapReply *rs, dynlist_info_t *old_dli )
        }
 
        for ( ; dli; dli = dli->dli_next ) {
+               if ( dli->dli_lud != NULL ) {
+                       /* check base and scope */
+                       if ( !BER_BVISNULL( &dli->dli_uri_nbase ) ) {
+                               int d = rs->sr_entry->e_nname.bv_len - dli->dli_uri_nbase.bv_len;
+
+                               if ( d < 0 ) {
+                                       continue;
+                               }
+
+                               if ( !dnIsSuffix( &rs->sr_entry->e_nname, &dli->dli_uri_nbase ) ) {
+                                       continue;
+                               }
+
+                               switch ( dli->dli_lud->lud_scope ) {
+                               case LDAP_SCOPE_BASE:
+                                       if ( d != 0 ) {
+                                               continue;
+                                       }
+                                       break;
+
+                               case LDAP_SCOPE_ONELEVEL: {
+                                       struct berval pdn;
+
+                                       dnParent( &rs->sr_entry->e_nname, &pdn );
+                                       if ( pdn.bv_len != dli->dli_uri_nbase.bv_len ) {
+                                               continue;
+                                       }
+                                       } break;
+
+                               case LDAP_SCOPE_SUBORDINATE:
+                                       if ( d == 0 ) {
+                                               continue;
+                                       }
+                                       break;
+
+                               case LDAP_SCOPE_SUBTREE:
+                               case LDAP_SCOPE_DEFAULT:
+                                       break;
+
+                               default:
+                                       continue;
+                               }
+                       }
+
+                       /* check filter */
+                       if ( dli->dli_uri_filter && test_filter( op, rs->sr_entry, dli->dli_uri_filter ) != LDAP_COMPARE_TRUE ) {
+                               continue;
+                       }
+               }
+
                if ( attr_valfind( a,
                                SLAP_MR_ATTRIBUTE_VALUE_NORMALIZED_MATCH |
                                SLAP_MR_ASSERTED_VALUE_NORMALIZED_MATCH,
@@ -116,20 +170,27 @@ dynlist_is_dynlist_next( Operation *op, SlapReply *rs, dynlist_info_t *old_dli )
 }
 
 static int
-dynlist_make_filter( Operation *op, struct berval *oldf, struct berval *newf )
+dynlist_make_filter( Operation *op, Entry *e, const char *url, struct berval *oldf, struct berval *newf )
 {
        slap_overinst   *on = (slap_overinst *)op->o_bd->bd_info;
        dynlist_info_t  *dli = (dynlist_info_t *)on->on_bi.bi_private;
 
        char            *ptr;
+       int             needBrackets = 0;
 
        assert( oldf != NULL );
        assert( newf != NULL );
        assert( !BER_BVISNULL( oldf ) );
        assert( !BER_BVISEMPTY( oldf ) );
 
+       if ( oldf->bv_val[0] != '(' ) {
+               Debug( LDAP_DEBUG_ANY, "%s: dynlist, DN=\"%s\": missing brackets in URI=\"%s\" filter\n",
+                       op->o_log_prefix, e->e_name.bv_val, url );
+               needBrackets = 2;
+       }
+
        newf->bv_len = STRLENOF( "(&(!(objectClass=" "))" ")" )
-               + dli->dli_oc->soc_cname.bv_len + oldf->bv_len;
+               + dli->dli_oc->soc_cname.bv_len + oldf->bv_len + needBrackets;
        newf->bv_val = op->o_tmpalloc( newf->bv_len + 1, op->o_tmpmemctx );
        if ( newf->bv_val == NULL ) {
                return -1;
@@ -137,7 +198,9 @@ dynlist_make_filter( Operation *op, struct berval *oldf, struct berval *newf )
        ptr = lutil_strcopy( newf->bv_val, "(&(!(objectClass=" );
        ptr = lutil_strcopy( ptr, dli->dli_oc->soc_cname.bv_val );
        ptr = lutil_strcopy( ptr, "))" );
+       if ( needBrackets ) *ptr++ = '(';
        ptr = lutil_strcopy( ptr, oldf->bv_val );
+       if ( needBrackets ) *ptr++ = ')';
        ptr = lutil_strcopy( ptr, ")" );
        newf->bv_len = ptr - newf->bv_val;
 
@@ -212,8 +275,8 @@ dynlist_sc_update( Operation *op, SlapReply *rs )
        }
 
 #ifndef SLAP_OPATTRS
-       opattrs = ( rs->sr_attrs == NULL ) ? 0 : an_find( rs->sr_attrs, &AllOper );
-       userattrs = ( rs->sr_attrs == NULL ) ? 1 : an_find( rs->sr_attrs, &AllUser );
+       opattrs = ( rs->sr_attrs == NULL ) ? 0 : an_find( rs->sr_attrs, slap_bv_operational_attrs );
+       userattrs = ( rs->sr_attrs == NULL ) ? 1 : an_find( rs->sr_attrs, slap_bv_user_attrs );
 #else /* SLAP_OPATTRS */
        opattrs = SLAP_OPATTRS( rs->sr_attr_flags );
        userattrs = SLAP_USERATTRS( rs->sr_attr_flags );
@@ -306,7 +369,9 @@ dynlist_sc_update( Operation *op, SlapReply *rs )
                        ad = a->a_desc;
                        for ( dlm = dlc->dlc_dli->dli_dlm; dlm; dlm = dlm->dlm_next ) {
                                if ( dlm->dlm_member_ad == a->a_desc ) {
-                                       ad = dlm->dlm_mapped_ad;
+                                       if ( dlm->dlm_mapped_ad ) {
+                                               ad = dlm->dlm_mapped_ad;
+                                       }
                                        break;
                                }
                        }
@@ -360,8 +425,8 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
        }
 
 #ifndef SLAP_OPATTRS
-       opattrs = ( rs->sr_attrs == NULL ) ? 0 : an_find( rs->sr_attrs, &AllOper );
-       userattrs = ( rs->sr_attrs == NULL ) ? 1 : an_find( rs->sr_attrs, &AllUser );
+       opattrs = ( rs->sr_attrs == NULL ) ? 0 : an_find( rs->sr_attrs, slap_bv_operational_attrs );
+       userattrs = ( rs->sr_attrs == NULL ) ? 1 : an_find( rs->sr_attrs, slap_bv_user_attrs );
 #else /* SLAP_OPATTRS */
        opattrs = SLAP_OPATTRS( rs->sr_attr_flags );
        userattrs = SLAP_USERATTRS( rs->sr_attr_flags );
@@ -369,8 +434,8 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
 
        /* Don't generate member list if it wasn't requested */
        for ( dlm = dli->dli_dlm; dlm; dlm = dlm->dlm_next ) {
-               if ( userattrs ||
-                    ad_inlist( dlm->dlm_member_ad, rs->sr_attrs ) ) 
+               AttributeDescription *ad = dlm->dlm_mapped_ad ? dlm->dlm_mapped_ad : dlm->dlm_member_ad;
+               if ( userattrs || ad_inlist( ad, rs->sr_attrs ) ) 
                        break;
        }
        if ( dli->dli_dlm && !dlm )
@@ -396,12 +461,14 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
                o.o_groups = NULL;
        }
 
+       e_flags = rs->sr_flags;
        if ( !( rs->sr_flags & REP_ENTRY_MODIFIABLE ) ) {
                e = entry_dup( rs->sr_entry );
+               e_flags |= ( REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED );
+               e_flags &= ~REP_ENTRY_MUSTRELEASE;
        } else {
                e = rs->sr_entry;
        }
-       e_flags = rs->sr_flags | ( REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED );
 
        dlc.dlc_e = e;
        dlc.dlc_dli = dli;
@@ -421,7 +488,6 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
                int             i, j;
                struct berval   dn;
                int             rc;
-               dynlist_map_t   *dlm;
 
                BER_BVZERO( &o.o_req_dn );
                BER_BVZERO( &o.o_req_ndn );
@@ -498,17 +564,42 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
                                        if ( o.ors_attrs[j].an_desc != NULL &&
                                                        is_at_operational( o.ors_attrs[j].an_desc->ad_type ) )
                                        {
-                                               if ( !opattrs && !ad_inlist( o.ors_attrs[j].an_desc, rs->sr_attrs ) )
-                                               {
+                                               if ( !opattrs ) {
                                                        continue;
                                                }
 
+                                               if ( !ad_inlist( o.ors_attrs[j].an_desc, rs->sr_attrs ) ) {
+                                                       /* lookup if mapped -- linear search,
+                                                        * not very efficient unless list
+                                                        * is very short */
+                                                       for ( dlm = dli->dli_dlm; dlm; dlm = dlm->dlm_next ) {
+                                                               if ( dlm->dlm_member_ad == o.ors_attrs[j].an_desc ) {
+                                                                       break;
+                                                               }
+                                                       }
+
+                                                       if ( dlm == NULL ) {
+                                                               continue;
+                                                       }
+                                               }
+
                                        } else {
                                                if ( !userattrs && 
                                                                o.ors_attrs[j].an_desc != NULL &&
                                                                !ad_inlist( o.ors_attrs[j].an_desc, rs->sr_attrs ) )
                                                {
-                                                       continue;
+                                                       /* lookup if mapped -- linear search,
+                                                        * not very efficient unless list
+                                                        * is very short */
+                                                       for ( dlm = dli->dli_dlm; dlm; dlm = dlm->dlm_next ) {
+                                                               if ( dlm->dlm_member_ad == o.ors_attrs[j].an_desc ) {
+                                                                       break;
+                                                               }
+                                                       }
+
+                                                       if ( dlm == NULL ) {
+                                                               continue;
+                                                       }
                                                }
                                        }
                                }
@@ -530,7 +621,7 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
                } else {
                        struct berval   flt;
                        ber_str2bv( lud->lud_filter, 0, 0, &flt );
-                       if ( dynlist_make_filter( op, &flt, &o.ors_filterstr ) ) {
+                       if ( dynlist_make_filter( op, rs->sr_entry, url->bv_val, &flt, &o.ors_filterstr ) ) {
                                /* error */
                                goto cleanup;
                        }
@@ -553,7 +644,7 @@ cleanup:;
                        slap_op_groups_free( &o );
                }
                if ( o.ors_filter ) {
-                       filter_free_x( &o, o.ors_filter );
+                       filter_free_x( &o, o.ors_filter, 1 );
                }
                if ( o.ors_attrs && o.ors_attrs != rs->sr_attrs
                                && o.ors_attrs != slap_anlist_no_attrs )
@@ -775,6 +866,8 @@ done:;
 
                if ( r.sr_flags & REP_ENTRY_MUSTBEFREED ) {
                        entry_free( r.sr_entry );
+                       r.sr_entry = NULL;
+                       r.sr_flags ^= REP_ENTRY_MUSTBEFREED;
                }
        }
 
@@ -851,7 +944,7 @@ dynlist_build_def_filter( dynlist_info_t *dli )
        ptr = lutil_strcopy( ptr, dli->dli_oc->soc_cname.bv_val );
        ptr = lutil_strcopy( ptr, "))" );
 
-       assert( dli->dli_default_filter.bv_len == ptr - dli->dli_default_filter.bv_val );
+       assert( ptr == &dli->dli_default_filter.bv_val[dli->dli_default_filter.bv_len] );
 
        return 0;
 }
@@ -874,7 +967,7 @@ dynlist_db_config(
                ObjectClass             *oc;
                AttributeDescription    *ad = NULL,
                                        *member_ad = NULL;
-               dynlist_map_t           *dlm = NULL;
+               dynlist_map_t           *dlm = NULL, *dlml = NULL;
                const char              *text;
 
                if ( argc < 3 ) {
@@ -914,7 +1007,6 @@ dynlist_db_config(
                        AttributeDescription *member_ad = NULL;
                        AttributeDescription *mapped_ad = NULL;
                        dynlist_map_t *dlmp;
-                       dynlist_map_t *dlml;
 
 
                        /*
@@ -949,7 +1041,6 @@ dynlist_db_config(
                        dlmp = (dynlist_map_t *)ch_calloc( 1, sizeof( dynlist_map_t ) );
                        if ( dlm == NULL ) {
                                dlm = dlmp;
-                               dlml = NULL;
                        }
                        dlmp->dlm_member_ad = member_ad;
                        dlmp->dlm_mapped_ad = mapped_ad;
@@ -1117,9 +1208,9 @@ static ConfigDriver       dl_cfgen;
 
 /* XXXmanu 255 is the maximum arguments we allow. Can we go beyond? */
 static ConfigTable dlcfg[] = {
-       { "dynlist-attrset", "group-oc> <URL-ad> <member-ad",
-               3, 255, 0, ARG_MAGIC|DL_ATTRSET, dl_cfgen,
-               "( OLcfgOvAt:8.1 NAME 'olcDLattrSet' "
+       { "dynlist-attrset", "group-oc> [uri] <URL-ad> <[mapped:]member-ad> [...]",
+               3, 0, 0, ARG_MAGIC|DL_ATTRSET, dl_cfgen,
+               "( OLcfgOvAt:8.1 NAME 'olcDlAttrSet' "
                        "DESC 'Dynamic list: <group objectClass>, <URL attributeDescription>, <member attributeDescription>' "
                        "EQUALITY caseIgnoreMatch "
                        "SYNTAX OMsDirectoryString "
@@ -1165,10 +1256,22 @@ dl_cfgen( ConfigArgs *c )
                                assert( dli->dli_oc != NULL );
                                assert( dli->dli_ad != NULL );
 
+                               /* FIXME: check buffer overflow! */
                                ptr += snprintf( c->cr_msg, sizeof( c->cr_msg ),
-                                       SLAP_X_ORDERED_FMT "%s %s", i,
-                                       dli->dli_oc->soc_cname.bv_val,
-                                       dli->dli_ad->ad_cname.bv_val );
+                                       SLAP_X_ORDERED_FMT "%s", i,
+                                       dli->dli_oc->soc_cname.bv_val );
+
+                               if ( !BER_BVISNULL( &dli->dli_uri ) ) {
+                                       *ptr++ = ' ';
+                                       *ptr++ = '"';
+                                       ptr = lutil_strncopy( ptr, dli->dli_uri.bv_val,
+                                               dli->dli_uri.bv_len );
+                                       *ptr++ = '"';
+                               }
+
+                               *ptr++ = ' ';
+                               ptr = lutil_strncopy( ptr, dli->dli_ad->ad_cname.bv_val,
+                                       dli->dli_ad->ad_cname.bv_len );
 
                                for ( dlm = dli->dli_dlm; dlm; dlm = dlm->dlm_next ) {
                                        ptr[ 0 ] = ' ';
@@ -1212,6 +1315,22 @@ dl_cfgen( ConfigArgs *c )
 
                                        dli_next = dli->dli_next;
 
+                                       if ( !BER_BVISNULL( &dli->dli_uri ) ) {
+                                               ch_free( dli->dli_uri.bv_val );
+                                       }
+
+                                       if ( dli->dli_lud != NULL ) {
+                                               ldap_free_urldesc( dli->dli_lud );
+                                       }
+
+                                       if ( !BER_BVISNULL( &dli->dli_uri_nbase ) ) {
+                                               ber_memfree( dli->dli_uri_nbase.bv_val );
+                                       }
+
+                                       if ( dli->dli_uri_filter != NULL ) {
+                                               filter_free( dli->dli_uri_filter );
+                                       }
+
                                        ch_free( dli->dli_default_filter.bv_val );
 
                                        while ( dlm != NULL ) {
@@ -1240,6 +1359,23 @@ dl_cfgen( ConfigArgs *c )
 
                                dli = *dlip;
                                *dlip = dli->dli_next;
+
+                               if ( !BER_BVISNULL( &dli->dli_uri ) ) {
+                                       ch_free( dli->dli_uri.bv_val );
+                               }
+
+                               if ( dli->dli_lud != NULL ) {
+                                       ldap_free_urldesc( dli->dli_lud );
+                               }
+
+                               if ( !BER_BVISNULL( &dli->dli_uri_nbase ) ) {
+                                       ber_memfree( dli->dli_uri_nbase.bv_val );
+                               }
+
+                               if ( dli->dli_uri_filter != NULL ) {
+                                       filter_free( dli->dli_uri_filter );
+                               }
+
                                ch_free( dli->dli_default_filter.bv_val );
 
                                dlm = dli->dli_dlm;
@@ -1273,7 +1409,12 @@ dl_cfgen( ConfigArgs *c )
                                        *dli_next = NULL;
                ObjectClass             *oc = NULL;
                AttributeDescription    *ad = NULL;
-               dynlist_map_t           *dlm = NULL;
+               int                     attridx = 2;
+               LDAPURLDesc             *lud = NULL;
+               struct berval           nbase = BER_BVNULL;
+               Filter                  *filter = NULL;
+               struct berval           uri = BER_BVNULL;
+               dynlist_map_t           *dlm = NULL, *dlml = NULL;
                const char              *text;
 
                oc = oc_find( c->argv[ 1 ] );
@@ -1286,11 +1427,98 @@ dl_cfgen( ConfigArgs *c )
                        return 1;
                }
 
-               rc = slap_str2ad( c->argv[ 2 ], &ad, &text );
+               if ( strncasecmp( c->argv[ attridx ], "ldap://", STRLENOF("ldap://") ) == 0 ) {
+                       if ( ldap_url_parse( c->argv[ attridx ], &lud ) != LDAP_URL_SUCCESS ) {
+                               snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                       "unable to parse URI \"%s\"",
+                                       c->argv[ attridx ] );
+                               rc = 1;
+                               goto done_uri;
+                       }
+
+                       if ( lud->lud_host != NULL ) {
+                               if ( lud->lud_host[0] == '\0' ) {
+                                       ch_free( lud->lud_host );
+                                       lud->lud_host = NULL;
+
+                               } else {
+                                       snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                               "host not allowed in URI \"%s\"",
+                                               c->argv[ attridx ] );
+                                       rc = 1;
+                                       goto done_uri;
+                               }
+                       }
+
+                       if ( lud->lud_attrs != NULL ) {
+                               snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                       "attrs not allowed in URI \"%s\"",
+                                       c->argv[ attridx ] );
+                               rc = 1;
+                               goto done_uri;
+                       }
+
+                       if ( lud->lud_exts != NULL ) {
+                               snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                       "extensions not allowed in URI \"%s\"",
+                                       c->argv[ attridx ] );
+                               rc = 1;
+                               goto done_uri;
+                       }
+
+                       if ( lud->lud_dn != NULL && lud->lud_dn[ 0 ] != '\0' ) {
+                               struct berval dn;
+                               ber_str2bv( lud->lud_dn, 0, 0, &dn );
+                               rc = dnNormalize( 0, NULL, NULL, &dn, &nbase, NULL );
+                               if ( rc != LDAP_SUCCESS ) {
+                                       snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                               "DN normalization failed in URI \"%s\"",
+                                               c->argv[ attridx ] );
+                                       goto done_uri;
+                               }
+                       }
+
+                       if ( lud->lud_filter != NULL && lud->lud_filter[ 0 ] != '\0' ) {
+                               filter = str2filter( lud->lud_filter );
+                               if ( filter == NULL ) {
+                                       snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
+                                               "filter parsing failed in URI \"%s\"",
+                                               c->argv[ attridx ] );
+                                       rc = 1;
+                                       goto done_uri;
+                               }
+                       }
+
+                       ber_str2bv( c->argv[ attridx ], 0, 1, &uri );
+
+done_uri:;
+                       if ( rc ) {
+                               if ( lud ) {
+                                       ldap_free_urldesc( lud );
+                               }
+
+                               if ( !BER_BVISNULL( &nbase ) ) {
+                                       ber_memfree( nbase.bv_val );
+                               }
+
+                               if ( filter != NULL ) {
+                                       filter_free( filter );
+                               }
+
+                               Debug( LDAP_DEBUG_ANY, "%s: %s.\n",
+                                       c->log, c->cr_msg, 0 );
+
+                               return rc;
+                       }
+
+                       attridx++;
+               }
+
+               rc = slap_str2ad( c->argv[ attridx ], &ad, &text );
                if ( rc != LDAP_SUCCESS ) {
                        snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
                                "unable to find AttributeDescription \"%s\"",
-                               c->argv[ 2 ] );
+                               c->argv[ attridx ] );
                        Debug( LDAP_DEBUG_ANY, "%s: %s.\n",
                                c->log, c->cr_msg, 0 );
                        return 1;
@@ -1300,19 +1528,20 @@ dl_cfgen( ConfigArgs *c )
                        snprintf( c->cr_msg, sizeof( c->cr_msg ), DYNLIST_USAGE
                                "AttributeDescription \"%s\" "
                                "must be a subtype of \"labeledURI\"",
-                               c->argv[ 2 ] );
+                               c->argv[ attridx ] );
                        Debug( LDAP_DEBUG_ANY, "%s: %s.\n",
                                c->log, c->cr_msg, 0 );
                        return 1;
                }
 
-               for ( i = 3; i < c->argc; i++ ) {
+               attridx++;
+
+               for ( i = attridx; i < c->argc; i++ ) {
                        char *arg; 
                        char *cp;
                        AttributeDescription *member_ad = NULL;
                        AttributeDescription *mapped_ad = NULL;
                        dynlist_map_t *dlmp;
-                       dynlist_map_t *dlml;
 
 
                        /*
@@ -1350,7 +1579,6 @@ dl_cfgen( ConfigArgs *c )
                        dlmp = (dynlist_map_t *)ch_calloc( 1, sizeof( dynlist_map_t ) );
                        if ( dlm == NULL ) {
                                dlm = dlmp;
-                               dlml = NULL;
                        }
                        dlmp->dlm_member_ad = member_ad;
                        dlmp->dlm_mapped_ad = mapped_ad;
@@ -1393,6 +1621,11 @@ dl_cfgen( ConfigArgs *c )
                (*dlip)->dli_dlm = dlm;
                (*dlip)->dli_next = dli_next;
 
+               (*dlip)->dli_lud = lud;
+               (*dlip)->dli_uri_nbase = nbase;
+               (*dlip)->dli_uri_filter = filter;
+               (*dlip)->dli_uri = uri;
+
                rc = dynlist_build_def_filter( *dlip );
 
                } break;
@@ -1557,22 +1790,26 @@ dynlist_db_open(
                }
        }
 
-       rc = slap_str2ad( "dgIdentity", &ad_dgIdentity, &text );
-       if ( rc != LDAP_SUCCESS ) {
-               snprintf( cr->msg, sizeof( cr->msg),
-                       "unable to fetch attributeDescription \"dgIdentity\": %d (%s)",
-                       rc, text );
-               Debug( LDAP_DEBUG_ANY, "dynlist_db_open: %s\n", cr->msg, 0, 0 );
-               /* Just a warning */
+       if ( ad_dgIdentity == NULL ) {
+               rc = slap_str2ad( "dgIdentity", &ad_dgIdentity, &text );
+               if ( rc != LDAP_SUCCESS ) {
+                       snprintf( cr->msg, sizeof( cr->msg),
+                               "unable to fetch attributeDescription \"dgIdentity\": %d (%s)",
+                               rc, text );
+                       Debug( LDAP_DEBUG_ANY, "dynlist_db_open: %s\n", cr->msg, 0, 0 );
+                       /* Just a warning */
+               }
        }
 
-       rc = slap_str2ad( "dgAuthz", &ad_dgAuthz, &text );
-       if ( rc != LDAP_SUCCESS ) {
-               snprintf( cr->msg, sizeof( cr->msg),
-                       "unable to fetch attributeDescription \"dgAuthz\": %d (%s)",
-                       rc, text );
-               Debug( LDAP_DEBUG_ANY, "dynlist_db_open: %s\n", cr->msg, 0, 0 );
-               /* Just a warning */
+       if ( ad_dgAuthz == NULL ) {
+               rc = slap_str2ad( "dgAuthz", &ad_dgAuthz, &text );
+               if ( rc != LDAP_SUCCESS ) {
+                       snprintf( cr->msg, sizeof( cr->msg),
+                               "unable to fetch attributeDescription \"dgAuthz\": %d (%s)",
+                               rc, text );
+                       Debug( LDAP_DEBUG_ANY, "dynlist_db_open: %s\n", cr->msg, 0, 0 );
+                       /* Just a warning */
+               }
        }
 
        return 0;
@@ -1595,7 +1832,24 @@ dynlist_db_destroy(
 
                        dli_next = dli->dli_next;
 
+                       if ( !BER_BVISNULL( &dli->dli_uri ) ) {
+                               ch_free( dli->dli_uri.bv_val );
+                       }
+
+                       if ( dli->dli_lud != NULL ) {
+                               ldap_free_urldesc( dli->dli_lud );
+                       }
+
+                       if ( !BER_BVISNULL( &dli->dli_uri_nbase ) ) {
+                               ber_memfree( dli->dli_uri_nbase.bv_val );
+                       }
+
+                       if ( dli->dli_uri_filter != NULL ) {
+                               filter_free( dli->dli_uri_filter );
+                       }
+
                        ch_free( dli->dli_default_filter.bv_val );
+
                        dlm = dli->dli_dlm;
                        while ( dlm != NULL ) {
                                dlm_next = dlm->dlm_next;