]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/overlays/translucent.c
Avoid use of uninitialized variable 'e' after failed overlay_entry_get_ov().
[openldap] / servers / slapd / overlays / translucent.c
index 126a336318d92565fe6d744ce55634664754a21d..3fadbbf6b1b935031ef6fa67e5cb2702e7a9d330 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2004-2006 The OpenLDAP Foundation.
+ * Copyright 2004-2007 The OpenLDAP Foundation.
  * Portions Copyright 2005 Symas Corporation.
  * All rights reserved.
  *
 /* config block */
 typedef struct translucent_info {
        BackendDB db;                   /* captive backend */
+       AttributeName *local;   /* valid attrs for local filters */
+       AttributeName *remote;  /* valid attrs for remote filters */
        int strict;
        int no_glue;
+       int defer_db_open;
 } translucent_info;
 
 static ConfigLDAPadd translucent_ldadd;
 static ConfigCfAdd translucent_cfadd;
 
+static ConfigDriver translucent_cf_gen;
+
+enum {
+       TRANS_LOCAL = 1,
+       TRANS_REMOTE
+};
+
 static ConfigTable translucentcfg[] = {
        { "translucent_strict", "on|off", 1, 2, 0,
          ARG_ON_OFF|ARG_OFFSET,
@@ -56,6 +66,26 @@ static ConfigTable translucentcfg[] = {
          "( OLcfgOvAt:14.2 NAME 'olcTranslucentNoGlue' "
          "DESC 'Disable automatic glue records for ADD and MODRDN' "
          "SYNTAX OMsBoolean SINGLE-VALUE )", NULL, NULL },
+       { "translucent_local", "attr[,attr...]", 1, 2, 0,
+         ARG_STRING|ARG_MAGIC|TRANS_LOCAL,
+         translucent_cf_gen,
+         "( OLcfgOvAt:14.3 NAME 'olcTranslucentLocal' "
+         "DESC 'Attributes to use in local search filter' "
+         "SYNTAX OMsDirectoryString )", NULL, NULL },
+       { "translucent_remote", "attr[,attr...]", 1, 2, 0,
+         ARG_STRING|ARG_MAGIC|TRANS_REMOTE,
+         translucent_cf_gen,
+         "( OLcfgOvAt:14.4 NAME 'olcTranslucentRemote' "
+         "DESC 'Attributes to use in remote search filter' "
+         "SYNTAX OMsDirectoryString )", NULL, NULL },
+       { NULL, NULL, 0, 0, 0, ARG_IGNORED }
+};
+
+static ConfigTable transdummy[] = {
+       { "", "", 0, 0, 0, ARG_IGNORED,
+               NULL, "( OLcfgGlAt:13 NAME 'olcDatabase' "
+                       "DESC 'The backend type for a database instance' "
+                       "SUP olcBackend SINGLE-VALUE X-ORDERED 'SIBLINGS' )", NULL, NULL },
        { NULL, NULL, 0, 0, 0, ARG_IGNORED }
 };
 
@@ -64,16 +94,27 @@ static ConfigOCs translucentocs[] = {
          "NAME 'olcTranslucentConfig' "
          "DESC 'Translucent configuration' "
          "SUP olcOverlayConfig "
-         "MAY ( olcTranslucentStrict $ olcTranslucentNoGlue ) )",
+         "MAY ( olcTranslucentStrict $ olcTranslucentNoGlue $"
+         " olcTranslucentLocal $ olcTranslucentRemote ) )",
          Cft_Overlay, translucentcfg, NULL, translucent_cfadd },
        { "( OLcfgOvOc:14.2 "
          "NAME 'olcTranslucentDatabase' "
          "DESC 'Translucent target database configuration' "
-         "AUXILIARY )", Cft_Misc, translucentcfg, translucent_ldadd },
+         "AUXILIARY )", Cft_Misc, transdummy, translucent_ldadd },
        { NULL, 0, NULL }
 };
 /* for translucent_init() */
 
+static int
+translucent_ldadd_cleanup( ConfigArgs *ca )
+{
+       slap_overinst *on = ca->private;
+       translucent_info *ov = on->on_bi.bi_private;
+
+       ov->defer_db_open = 0;
+       return backend_startup_one( ca->be, &ca->reply );
+}
+
 static int
 translucent_ldadd( CfEntryInfo *cei, Entry *e, ConfigArgs *ca )
 {
@@ -89,6 +130,12 @@ translucent_ldadd( CfEntryInfo *cei, Entry *e, ConfigArgs *ca )
        on = (slap_overinst *)cei->ce_bi;
        ov = on->on_bi.bi_private;
        ca->be = &ov->db;
+       ca->private = on;
+       if ( CONFIG_ONLINE_ADD( ca ))
+               ca->cleanup = translucent_ldadd_cleanup;
+       else
+               ov->defer_db_open = 0;
+
        return LDAP_SUCCESS;
 }
 
@@ -103,10 +150,14 @@ translucent_cfadd( Operation *op, SlapReply *rs, Entry *e, ConfigArgs *ca )
        Debug(LDAP_DEBUG_TRACE, "==> translucent_cfadd\n", 0, 0, 0);
 
        /* FIXME: should not hardcode "olcDatabase" here */
-       bv.bv_len = sprintf( ca->msg, "olcDatabase=%s",
-                            ov->db.bd_info->bi_type );
-       bv.bv_val = ca->msg;
+       bv.bv_len = snprintf( ca->cr_msg, sizeof( ca->cr_msg ),
+               "olcDatabase=%s", ov->db.bd_info->bi_type );
+       if ( bv.bv_len < 0 || bv.bv_len >= sizeof( ca->cr_msg ) ) {
+               return -1;
+       }
+       bv.bv_val = ca->cr_msg;
        ca->be = &ov->db;
+       ov->defer_db_open = 0;
 
        /* We can only create this entry if the database is table-driven
         */
@@ -118,6 +169,51 @@ translucent_cfadd( Operation *op, SlapReply *rs, Entry *e, ConfigArgs *ca )
        return 0;
 }
 
+static int
+translucent_cf_gen( ConfigArgs *c )
+{
+       slap_overinst   *on = (slap_overinst *)c->bi;
+       translucent_info *ov = on->on_bi.bi_private;
+       AttributeName **an, *a2;
+       int i;
+
+       if ( c->type == TRANS_LOCAL )
+               an = &ov->local;
+       else
+               an = &ov->remote;
+
+       if ( c->op == SLAP_CONFIG_EMIT ) {
+               if ( !*an )
+                       return 1;
+               for ( i = 0; !BER_BVISNULL(&(*an)[i].an_name); i++ ) {
+                       value_add_one( &c->rvalue_vals, &(*an)[i].an_name );
+               }
+               return ( i < 1 );
+       } else if ( c->op == LDAP_MOD_DELETE ) {
+               if ( c->valx < 0 ) {
+                       anlist_free( *an, 1, NULL );
+                       *an = NULL;
+               } else {
+                       i = c->valx;
+                       ch_free( (*an)[i].an_name.bv_val );
+                       do {
+                               (*an)[i] = (*an)[i+1];
+                       } while ( !BER_BVISNULL( &(*an)[i].an_name ));
+               }
+               return 0;
+       }
+       a2 = str2anlist( *an, c->argv[1], "," );
+       if ( !a2 ) {
+               snprintf( c->cr_msg, sizeof( c->cr_msg ), "%s unable to parse attribute %s",
+                       c->argv[0], c->argv[1] );
+               Debug( LDAP_DEBUG_CONFIG|LDAP_DEBUG_NONE,
+                       "%s: %s\n", c->log, c->cr_msg, 0 );
+               return ARG_BAD_CONF;
+       }
+       *an = a2;
+       return 0;
+}
+
 static slap_overinst translucent;
 
 /*
@@ -147,6 +243,7 @@ void glue_parent(Operation *op) {
        ber_dupbv(&e->e_nname, &ndn);
 
        a = attr_alloc( slap_schema.si_ad_objectClass );
+       a->a_numvals = 2;
        a->a_vals = ch_malloc(sizeof(struct berval) * 3);
        ber_dupbv(&a->a_vals[0], &glue[0]);
        ber_dupbv(&a->a_vals[1], &glue[1]);
@@ -156,6 +253,7 @@ void glue_parent(Operation *op) {
        e->e_attrs = a;
 
        a = attr_alloc( slap_schema.si_ad_structuralObjectClass );
+       a->a_numvals = 1;
        a->a_vals = ch_malloc(sizeof(struct berval) * 2);
        ber_dupbv(&a->a_vals[0], &glue[1]);
        ber_dupbv(&a->a_vals[1], &glue[2]);
@@ -251,7 +349,11 @@ static int translucent_modrdn(Operation *op, SlapReply *rs) {
                op->o_bd->bd_info = (BackendInfo *) on;
                return(rs->sr_err);
        }
-       if(!ov->no_glue) glue_parent(op);
+       if(!ov->no_glue) {
+               op->o_tag = LDAP_REQ_ADD;
+               glue_parent(op);
+               op->o_tag = LDAP_REQ_MODRDN;
+       }
        return(SLAP_CB_CONTINUE);
 }
 
@@ -296,19 +398,24 @@ translucent_tag_cb( Operation *op, SlapReply *rs )
 
 static int translucent_modify(Operation *op, SlapReply *rs) {
        SlapReply nrs = { REP_RESULT };
-       Operation nop = *op;
 
        slap_overinst *on = (slap_overinst *) op->o_bd->bd_info;
        translucent_info *ov = on->on_bi.bi_private;
        Entry *e = NULL, *re = NULL;
        Attribute *a, *ax;
-       Modifications *m, *mm;
+       Modifications *m, **mm;
+       BackendDB *db;
        int del, rc, erc = 0;
        slap_callback cb = { 0 };
 
        Debug(LDAP_DEBUG_TRACE, "==> translucent_modify: %s\n",
                op->o_req_dn.bv_val, 0, 0);
 
+       if(ov->defer_db_open) {
+               send_ldap_error(op, rs, LDAP_UNAVAILABLE,
+                       "remote DB not available");
+               return(rs->sr_err);
+       }
 /*
 ** fetch entry from the captive backend;
 ** if it did not exist, fail;
@@ -316,14 +423,15 @@ static int translucent_modify(Operation *op, SlapReply *rs) {
 **
 */
 
-       nop.o_bd = &ov->db;
-       rc = ov->db.bd_info->bi_entry_get_rw(&nop, &nop.o_req_ndn, NULL, NULL, 0, &re);
+       db = op->o_bd;
+       op->o_bd = &ov->db;
+       rc = ov->db.bd_info->bi_entry_get_rw(op, &op->o_req_ndn, NULL, NULL, 0, &re);
        if(rc != LDAP_SUCCESS || re == NULL ) {
-               send_ldap_error((&nop), rs, LDAP_NO_SUCH_OBJECT,
+               send_ldap_error((op), rs, LDAP_NO_SUCH_OBJECT,
                        "attempt to modify nonexistent local record");
                return(rs->sr_err);
        }
-       nop = *op;
+       op->o_bd = db;
 /*
 ** fetch entry from local backend;
 ** if it exists:
@@ -343,10 +451,14 @@ static int translucent_modify(Operation *op, SlapReply *rs) {
 
        if(e && rc == LDAP_SUCCESS) {
                Debug(LDAP_DEBUG_TRACE, "=> translucent_modify: found local entry\n", 0, 0, 0);
-               for(m = op->orm_modlist; m; m = m->sml_next) {
+               for(mm = &op->orm_modlist; *mm; ) {
+                       m = *mm;
                        for(a = e->e_attrs; a; a = a->a_next)
                                if(a->a_desc == m->sml_desc) break;
-                       if(a) continue;         /* found local attr */
+                       if(a) {
+                               mm = &m->sml_next;
+                               continue;               /* found local attr */
+                       }
                        if(m->sml_op == LDAP_MOD_DELETE) {
                                for(a = re->e_attrs; a; a = a->a_next)
                                        if(a->a_desc == m->sml_desc) break;
@@ -362,21 +474,22 @@ static int translucent_modify(Operation *op, SlapReply *rs) {
                                Debug(LDAP_DEBUG_TRACE,
                                        "=> translucent_modify: silently dropping delete: %s\n",
                                        m->sml_desc->ad_cname.bv_val, 0, 0);
-                               for(mm = op->orm_modlist; mm->sml_next != m; mm = mm->sml_next);
-                               mm->sml_next = m->sml_next;
+                               *mm = m->sml_next;
                                m->sml_next = NULL;
                                slap_mods_free(m, 1);
-                               m = mm;
                                continue;
                        }
                        m->sml_op = LDAP_MOD_ADD;
+                       mm = &m->sml_next;
                }
                erc = SLAP_CB_CONTINUE;
 release:
                if(re) {
-                       if(ov->db.bd_info->bi_entry_release_rw)
-                               ov->db.bd_info->bi_entry_release_rw(&nop, re, 0);
-                       else
+                       if(ov->db.bd_info->bi_entry_release_rw) {
+                               op->o_bd = &ov->db;
+                               ov->db.bd_info->bi_entry_release_rw(op, re, 0);
+                               op->o_bd = db;
+                       } else
                                entry_free(re);
                }
                op->o_bd->bd_info = (BackendInfo *) on->on_info;
@@ -393,9 +506,11 @@ release:
 
        /* don't leak remote entry copy */
        if(re) {
-               if(ov->db.bd_info->bi_entry_release_rw)
-                       ov->db.bd_info->bi_entry_release_rw(&nop, re, 0);
-               else
+               if(ov->db.bd_info->bi_entry_release_rw) {
+                       op->o_bd = &ov->db;
+                       ov->db.bd_info->bi_entry_release_rw(op, re, 0);
+                       op->o_bd = db;
+               } else
                        entry_free(re);
        }
 /*
@@ -425,6 +540,8 @@ release:
                atmp.a_desc = m->sml_desc;
                atmp.a_vals = m->sml_values;
                atmp.a_nvals = m->sml_nvalues ? m->sml_nvalues : atmp.a_vals;
+               atmp.a_numvals = m->sml_numvals;
+               atmp.a_flags = 0;
                a = attr_dup( &atmp );
                a->a_next  = ax;
                ax = a;
@@ -454,28 +571,29 @@ release:
        ber_dupbv( &e->e_nname, &op->o_req_ndn );
        e->e_attrs = a;
 
-       nop.o_tag       = LDAP_REQ_ADD;
-       nop.oq_add.rs_e = e;
-
-       glue_parent(&nop);
-
+       op->o_tag       = LDAP_REQ_ADD;
        cb.sc_response = translucent_tag_cb;
        cb.sc_private = op->orm_modlist;
-       cb.sc_next = nop.o_callback;
-       nop.o_callback = &cb;
-       rc = on->on_info->oi_orig->bi_op_add(&nop, &nrs);
-       if ( nop.ora_e == e )
+       op->oq_add.rs_e = e;
+
+       glue_parent(op);
+
+       cb.sc_next = op->o_callback;
+       op->o_callback = &cb;
+       rc = on->on_info->oi_orig->bi_op_add(op, &nrs);
+       if ( op->ora_e == e )
                entry_free( e );
+       op->o_callback = cb.sc_next;
 
        return(rc);
 }
 
 static int translucent_compare(Operation *op, SlapReply *rs) {
-       Operation nop = *op;
        slap_overinst *on = (slap_overinst *) op->o_bd->bd_info;
        translucent_info *ov = on->on_bi.bi_private;
        AttributeAssertion *ava = op->orc_ava;
        Entry *e;
+       BackendDB *db;
        int rc;
 
        Debug(LDAP_DEBUG_TRACE, "==> translucent_compare: <%s> %s:%s\n",
@@ -486,71 +604,149 @@ static int translucent_compare(Operation *op, SlapReply *rs) {
 **     CONTINUE and let it do the compare;
 **
 */
-       op->o_bd->bd_info = (BackendInfo *) on->on_info;
-       rc = be_entry_get_rw(op, &op->o_req_ndn, NULL, ava->aa_desc, 0, &e);
-       if(e && rc == LDAP_SUCCESS) {
-               be_entry_release_r(op, e);
-               op->o_bd->bd_info = (BackendInfo *) on;
+       rc = overlay_entry_get_ov(op, &op->o_req_ndn, NULL, ava->aa_desc, 0, &e, on);
+       if(rc == LDAP_SUCCESS && e) {
+               overlay_entry_release_ov(op, e, 0, on);
                return(SLAP_CB_CONTINUE);
        }
-       op->o_bd->bd_info = (BackendInfo *) on;
 
+       if(ov->defer_db_open) {
+               send_ldap_error(op, rs, LDAP_UNAVAILABLE,
+                       "remote DB not available");
+               return(rs->sr_err);
+       }
 /*
 ** call compare() in the captive backend;
 ** return the result;
 **
 */
-       nop.o_bd = &ov->db;
-       nop.o_callback = NULL;
-       rc = ov->db.bd_info->bi_op_compare(&nop, rs);
+       db = op->o_bd;
+       op->o_bd = &ov->db;
+       rc = ov->db.bd_info->bi_op_compare(op, rs);
+       op->o_bd = db;
 
        return(rc);
 }
 
 /*
 ** translucent_search_cb()
-**     merge local data with the search result
+**     merge local data with remote data
 **
+** Four cases:
+** 1: remote search, no local filter
+**     merge data and send immediately
+** 2: remote search, with local filter
+**     merge data and save
+** 3: local search, no remote filter
+**     merge data and send immediately
+** 4: local search, with remote filter
+**     check list, merge, send, delete
 */
 
+#define        RMT_SIDE        0
+#define        LCL_SIDE        1
+#define        USE_LIST        2
+
+typedef struct trans_ctx {
+       BackendDB *db;
+       slap_overinst *on;
+       Filter *orig;
+       Avlnode *list;
+       int step;
+} trans_ctx;
+
 static int translucent_search_cb(Operation *op, SlapReply *rs) {
+       trans_ctx *tc;
+       BackendDB *db;
        slap_overinst *on;
-       Entry *e, *re = NULL;
+       translucent_info *ov;
+       Entry *le, *re;
        Attribute *a, *ax, *an, *as = NULL;
-       Operation * original_op, local_op;
        int rc;
 
+       tc = op->o_callback->sc_private;
+
+       /* Don't let the op complete while we're gathering data */
+       if ( rs->sr_type == REP_RESULT && ( tc->step & USE_LIST ))
+               return 0;
+
        if(!op || !rs || rs->sr_type != REP_SEARCH || !rs->sr_entry)
                return(SLAP_CB_CONTINUE);
 
        Debug(LDAP_DEBUG_TRACE, "==> translucent_search_cb: %s\n",
                rs->sr_entry->e_name.bv_val, 0, 0);
 
-       original_op = op->o_callback->sc_private;
-       on = (slap_overinst *) original_op->o_bd->bd_info;
-       local_op = *original_op;
+       on = tc->on;
+       ov = on->on_bi.bi_private;
 
-       local_op.o_bd->bd_info = (BackendInfo *) on->on_info->oi_orig;
-       rc = be_entry_get_rw(&local_op, &rs->sr_entry->e_nname, NULL, NULL, 0, &e);
-       local_op.o_bd->bd_info = (BackendInfo *) on;
+       db = op->o_bd;
+       re = NULL;
+
+       /* If we have local, get remote */
+       if ( tc->step & LCL_SIDE ) {
+               le = rs->sr_entry;
+               /* If entry is already on list, use it */
+               if ( tc->step & USE_LIST ) {
+                       re = tavl_delete( &tc->list, le, entry_dn_cmp );
+                       if ( re ) {
+                               if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+                                       rs->sr_flags ^= REP_ENTRY_MUSTRELEASE;
+                                       be_entry_release_r( op, rs->sr_entry );
+                               }
+                               if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
+                                       rs->sr_flags ^= REP_ENTRY_MUSTBEFREED;
+                                       entry_free( rs->sr_entry );
+                               }
+                               rc = test_filter( op, re, tc->orig );
+                               if ( rc == LDAP_COMPARE_TRUE ) {
+                                       rs->sr_flags |= REP_ENTRY_MUSTBEFREED;
+                                       rs->sr_entry = re;
+                                       return SLAP_CB_CONTINUE;
+                               } else {
+                                       entry_free( re );
+                                       rs->sr_entry = NULL;
+                                       return 0;
+                               }
+                       }
+               }
+               op->o_bd = &ov->db;
+               rc = be_entry_get_rw( op, &rs->sr_entry->e_nname, NULL, NULL, 0, &re );
+               if ( rc == LDAP_SUCCESS && re ) {
+                       Entry *tmp = entry_dup( re );
+                       be_entry_release_r( op, re );
+                       re = tmp;
+               }
+       } else {
+       /* Else we have remote, get local */
+               op->o_bd = tc->db;
+               rc = overlay_entry_get_ov(op, &rs->sr_entry->e_nname, NULL, NULL, 0, &le, on);
+               if ( rc == LDAP_SUCCESS && le ) {
+                       re = entry_dup( rs->sr_entry );
+                       if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+                               rs->sr_flags ^= REP_ENTRY_MUSTRELEASE;
+                               be_entry_release_r( op, rs->sr_entry );
+                       }
+                       if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
+                               rs->sr_flags ^= REP_ENTRY_MUSTBEFREED;
+                               entry_free( rs->sr_entry );
+                       }
+               } else {
+                       le = NULL;
+               }
+       }
 
 /*
-** if we got an entry from local backend:
-**     make a copy of this search result;
+** if we got remote and local entry:
 **     foreach local attr:
-**             foreach search result attr:
-**                     if match, result attr with local attr;
+**             foreach remote attr:
+**                     if match, remote attr with local attr;
 **                     if new local, add to list;
-**     append new local attrs to search result;
+**     append new local attrs to remote;
 **
 */
 
-       if(e && rc == LDAP_SUCCESS) {
-               re = entry_dup(rs->sr_entry);
-               for(ax = e->e_attrs; ax; ax = ax->a_next) {
-#if 0
-                       if(is_at_operational(ax->a_desc->ad_type)) continue;
-#endif
+       if ( re && le ) {
+               for(ax = le->e_attrs; ax; ax = ax->a_next) {
                        for(a = re->e_attrs; a; a = a->a_next) {
                                if(a->a_desc == ax->a_desc) {
                                        if(a->a_vals != a->a_nvals)
@@ -567,9 +763,19 @@ static int translucent_search_cb(Operation *op, SlapReply *rs) {
                        an->a_next = as;
                        as = an;
                }
-               local_op.o_bd->bd_info = (BackendInfo *) on->on_info->oi_orig;
-               be_entry_release_r(&local_op, e);
-               local_op.o_bd->bd_info = (BackendInfo *) on;
+               /* Dispose of local entry */
+               if ( tc->step & LCL_SIDE ) {
+                       if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+                               rs->sr_flags ^= REP_ENTRY_MUSTRELEASE;
+                               be_entry_release_r( op, rs->sr_entry );
+                       }
+                       if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
+                               rs->sr_flags ^= REP_ENTRY_MUSTBEFREED;
+                               entry_free( rs->sr_entry );
+                       }
+               } else {
+                       overlay_entry_release_ov(op, le, 0, on);
+               }
 
                /* literally append, so locals are always last */
                if(as) {
@@ -580,11 +786,143 @@ static int translucent_search_cb(Operation *op, SlapReply *rs) {
                                re->e_attrs = as;
                        }
                }
-               rs->sr_entry = re;
-               rs->sr_flags |= REP_ENTRY_MUSTBEFREED;
+               /* If both filters, save entry for later */
+               if ( tc->step == (USE_LIST|RMT_SIDE) ) {
+                       tavl_insert( &tc->list, re, entry_dn_cmp, avl_dup_error );
+                       rs->sr_entry = NULL;
+                       rc = 0;
+               } else {
+               /* send it now */
+                       rs->sr_entry = re;
+                       rs->sr_flags |= REP_ENTRY_MUSTBEFREED;
+                       rc = SLAP_CB_CONTINUE;
+               }
+       } else if ( le ) {
+       /* Only a local entry: remote was deleted
+        * Ought to delete the local too...
+        */
+               rc = 0;
+       } else if ( tc->step & USE_LIST ) {
+       /* Only a remote entry, but both filters:
+        * Test the complete filter
+        */
+               rc = test_filter( op, rs->sr_entry, tc->orig );
+               if ( rc == LDAP_COMPARE_TRUE ) {
+                       rc = SLAP_CB_CONTINUE;
+               } else {
+                       rc = 0;
+               }
+       } else {
+       /* Only a remote entry, only remote filter:
+        * just pass thru
+        */
+               rc = SLAP_CB_CONTINUE;
        }
 
-       return(SLAP_CB_CONTINUE);
+       op->o_bd = db;
+       return rc;
+}
+
+/* Dup the filter, excluding invalid elements */
+static Filter *
+trans_filter_dup(Operation *op, Filter *f, AttributeName *an)
+{
+       Filter *n = NULL;
+
+       if ( !f )
+               return NULL;
+
+       switch( f->f_choice & SLAPD_FILTER_MASK ) {
+       case SLAPD_FILTER_COMPUTED:
+               n = op->o_tmpalloc( sizeof(Filter), op->o_tmpmemctx );
+               n->f_choice = f->f_choice;
+               n->f_result = f->f_result;
+               n->f_next = NULL;
+               break;
+
+       case LDAP_FILTER_PRESENT:
+               if ( ad_inlist( f->f_desc, an )) {
+                       n = op->o_tmpalloc( sizeof(Filter), op->o_tmpmemctx );
+                       n->f_choice = f->f_choice;
+                       n->f_desc = f->f_desc;
+                       n->f_next = NULL;
+               }
+               break;
+
+       case LDAP_FILTER_EQUALITY:
+       case LDAP_FILTER_GE:
+       case LDAP_FILTER_LE:
+       case LDAP_FILTER_APPROX:
+       case LDAP_FILTER_SUBSTRINGS:
+       case LDAP_FILTER_EXT:
+               if ( !f->f_av_desc || ad_inlist( f->f_av_desc, an )) {
+                       n = op->o_tmpalloc( sizeof(Filter), op->o_tmpmemctx );
+                       n->f_choice = f->f_choice;
+                       n->f_ava = f->f_ava;
+                       n->f_next = NULL;
+               }
+               break;
+
+       case LDAP_FILTER_AND:
+       case LDAP_FILTER_OR:
+       case LDAP_FILTER_NOT: {
+               Filter **p;
+
+               n = op->o_tmpalloc( sizeof(Filter), op->o_tmpmemctx );
+               n->f_choice = f->f_choice;
+               n->f_next = NULL;
+
+               for ( p = &n->f_list, f = f->f_list; f; f = f->f_next ) {
+                       *p = trans_filter_dup( op, f, an );
+                       if ( !*p )
+                               continue;
+                       p = &(*p)->f_next;
+               }
+               /* nothing valid in this list */
+               if ( !n->f_list ) {
+                       op->o_tmpfree( n, op->o_tmpmemctx );
+                       return NULL;
+               }
+               /* Only 1 element in this list */
+               if ((n->f_choice & SLAPD_FILTER_MASK) != LDAP_FILTER_NOT &&
+                       !n->f_list->f_next ) {
+                       f = n->f_list;
+                       *n = *f;
+                       op->o_tmpfree( f, op->o_tmpmemctx );
+               }
+               break;
+       }
+       }
+       return n;
+}
+
+static void
+trans_filter_free( Operation *op, Filter *f )
+{
+       Filter *n, *p, *next;
+
+       f->f_choice &= SLAPD_FILTER_MASK;
+
+       switch( f->f_choice ) {
+       case LDAP_FILTER_AND:
+       case LDAP_FILTER_OR:
+       case LDAP_FILTER_NOT:
+               /* Free in reverse order */
+               n = NULL;
+               for ( p = f->f_list; p; p = next ) {
+                       next = p->f_next;
+                       p->f_next = n;
+                       n = p;
+               }
+               for ( p = n; p; p = next ) {
+                       next = p->f_next;
+                       trans_filter_free( op, p );
+               }
+               break;
+       default:
+               break;
+       }
+       op->o_tmpfree( f, op->o_tmpmemctx );
 }
 
 /*
@@ -596,20 +934,92 @@ static int translucent_search_cb(Operation *op, SlapReply *rs) {
 
 static int translucent_search(Operation *op, SlapReply *rs) {
        slap_overinst *on = (slap_overinst *) op->o_bd->bd_info;
-       Operation nop = *op;
        translucent_info *ov = on->on_bi.bi_private;
        slap_callback cb = { NULL, NULL, NULL, NULL };
+       trans_ctx tc;
+       Filter *fl, *fr;
+       struct berval fbv;
+       int rc = 0;
 
        Debug(LDAP_DEBUG_TRACE, "==> translucent_search: <%s> %s\n",
                op->o_req_dn.bv_val, op->ors_filterstr.bv_val, 0);
 
+       if(ov->defer_db_open) {
+               send_ldap_error(op, rs, LDAP_UNAVAILABLE,
+                       "remote DB not available");
+               return(rs->sr_err);
+       }
+
+       fr = ov->remote ? trans_filter_dup( op, op->ors_filter, ov->remote ) : NULL;
+       fl = ov->local ? trans_filter_dup( op, op->ors_filter, ov->local ) : NULL;
        cb.sc_response = (slap_response *) translucent_search_cb;
-       cb.sc_private = op;
-       cb.sc_next = nop.o_callback;
+       cb.sc_private = &tc;
+       cb.sc_next = op->o_callback;
+
+       tc.db = op->o_bd;
+       tc.on = on;
+       tc.orig = op->ors_filter;
+       tc.list = NULL;
+       tc.step = 0;
+       fbv = op->ors_filterstr;
+
+       op->o_callback = &cb;
+
+       if ( fr || !fl ) {
+               op->o_bd = &ov->db;
+               tc.step |= RMT_SIDE;
+               if ( fl ) {
+                       tc.step |= USE_LIST;
+                       op->ors_filter = fr;
+                       filter2bv_x( op, fr, &op->ors_filterstr );
+               }
+               rc = ov->db.bd_info->bi_op_search(op, rs);
+               op->o_bd = tc.db;
+               if ( fl ) {
+                       op->o_tmpfree( op->ors_filterstr.bv_val, op->o_tmpmemctx );
+               }
+       }
+       if ( fl && !rc ) {
+               tc.step |= LCL_SIDE;
+               op->ors_filter = fl;
+               filter2bv_x( op, fl, &op->ors_filterstr );
+               rc = overlay_op_walk( op, rs, op_search, on->on_info, on->on_next );
+               op->o_tmpfree( op->ors_filterstr.bv_val, op->o_tmpmemctx );
+       }
+       op->ors_filterstr = fbv;
+       op->ors_filter = tc.orig;
+       op->o_callback = cb.sc_next;
+       /* Send out anything remaining on the list and finish */
+       if ( tc.step & USE_LIST ) {
+               if ( tc.list ) {
+                       Avlnode *av;
+
+                       av = tavl_end( tc.list, TAVL_DIR_LEFT );
+                       while ( av ) {
+                               rs->sr_entry = av->avl_data;
+                               rc = test_filter( op, rs->sr_entry, op->ors_filter );
+                               if ( rc == LDAP_COMPARE_TRUE ) {
+                                       rs->sr_flags = REP_ENTRY_MUSTBEFREED;
+                                       rc = send_search_entry( op, rs );
+                                       if ( rc ) break;
+                               } else {
+                                       entry_free( rs->sr_entry );
+                               }
+                               av = tavl_next( av, TAVL_DIR_RIGHT );
+                       }
+                       tavl_free( tc.list, NULL );
+                       rs->sr_entry = NULL;
+               }
+               send_ldap_result( op, rs );
+       }
+
+       /* Free in reverse order */
+       if ( fl )
+               trans_filter_free( op, fl );
+       if ( fr )
+               trans_filter_free( op, fr );
 
-       nop.o_callback = &cb;
-       nop.o_bd = &ov->db;
-       return (ov->db.bd_info->bi_op_search(&nop, rs));
+       return rc;
 }
 
 
@@ -621,14 +1031,23 @@ static int translucent_search(Operation *op, SlapReply *rs) {
 
 static int translucent_bind(Operation *op, SlapReply *rs) {
        slap_overinst *on = (slap_overinst *) op->o_bd->bd_info;
-       Operation nop = *op;
        translucent_info *ov = on->on_bi.bi_private;
+       BackendDB *db;
+       int rc;
 
        Debug(LDAP_DEBUG_TRACE, "translucent_bind: <%s> method %d\n",
                op->o_req_dn.bv_val, op->orb_method, 0);
 
-       nop.o_bd = &ov->db;
-       return (ov->db.bd_info->bi_op_bind(&nop, rs));
+       if(ov->defer_db_open) {
+               send_ldap_error(op, rs, LDAP_UNAVAILABLE,
+                       "remote DB not available");
+               return(rs->sr_err);
+       }
+       db = op->o_bd;
+       op->o_bd = &ov->db;
+       rc = ov->db.bd_info->bi_op_bind(op, rs);
+       op->o_bd = db;
+       return rc;
 }
 
 /*
@@ -683,10 +1102,9 @@ static int translucent_db_config(
 **
 */
 
-static int translucent_db_init(BackendDB *be) {
+static int translucent_db_init(BackendDB *be, ConfigReply *cr) {
        slap_overinst *on = (slap_overinst *) be->bd_info;
        translucent_info *ov;
-       int rc;
 
        Debug(LDAP_DEBUG_TRACE, "==> translucent_db_init\n", 0, 0, 0);
 
@@ -695,8 +1113,9 @@ static int translucent_db_init(BackendDB *be) {
        ov->db = *be;
        ov->db.be_private = NULL;
        ov->db.be_pcl_mutexp = &ov->db.be_pcl_mutex;
+       ov->defer_db_open = 1;
 
-       if ( !backend_db_init( "ldap", &ov->db )) {
+       if ( !backend_db_init( "ldap", &ov->db, -1, NULL )) {
                Debug( LDAP_DEBUG_CONFIG, "translucent: unable to open captive back-ldap\n", 0, 0, 0);
                return 1;
        }
@@ -712,7 +1131,7 @@ static int translucent_db_init(BackendDB *be) {
 **
 */
 
-static int translucent_db_open(BackendDB *be) {
+static int translucent_db_open(BackendDB *be, ConfigReply *cr) {
        slap_overinst *on = (slap_overinst *) be->bd_info;
        translucent_info *ov = on->on_bi.bi_private;
        int rc;
@@ -725,7 +1144,10 @@ static int translucent_db_open(BackendDB *be) {
        ov->db.be_acl = be->be_acl;
        ov->db.be_dfltaccess = be->be_dfltaccess;
 
-       rc = backend_startup_one( &ov->db );
+       if ( ov->defer_db_open )
+               return 0;
+
+       rc = backend_startup_one( &ov->db, NULL );
 
        if(rc) Debug(LDAP_DEBUG_TRACE,
                "translucent: bi_db_open() returned error %d\n", rc, 0, 0);
@@ -735,13 +1157,12 @@ static int translucent_db_open(BackendDB *be) {
 
 /*
 ** translucent_db_close()
-**     if the captive backend has a close() method, call it;
-**     free any config data;
+**     if the captive backend has a close() method, call it
 **
 */
 
 static int
-translucent_db_close( BackendDB *be )
+translucent_db_close( BackendDB *be, ConfigReply *cr )
 {
        slap_overinst *on = (slap_overinst *) be->bd_info;
        translucent_info *ov = on->on_bi.bi_private;
@@ -750,7 +1171,7 @@ translucent_db_close( BackendDB *be )
        Debug(LDAP_DEBUG_TRACE, "==> translucent_db_close\n", 0, 0, 0);
 
        if ( ov && ov->db.bd_info && ov->db.bd_info->bi_db_close ) {
-               rc = ov->db.bd_info->bi_db_close(&ov->db);
+               rc = ov->db.bd_info->bi_db_close(&ov->db, NULL);
        }
 
        return(rc);
@@ -758,20 +1179,25 @@ translucent_db_close( BackendDB *be )
 
 /*
 ** translucent_db_destroy()
-**     if the captive backend has a db_destroy() method, call it
+**     if the captive backend has a db_destroy() method, call it;
+**     free any config data
 **
 */
 
 static int
-translucent_db_destroy( BackendDB *be )
+translucent_db_destroy( BackendDB *be, ConfigReply *cr )
 {
        slap_overinst *on = (slap_overinst *) be->bd_info;
        translucent_info *ov = on->on_bi.bi_private;
        int rc = 0;
 
-       Debug(LDAP_DEBUG_TRACE, "==> translucent_db_close\n", 0, 0, 0);
+       Debug(LDAP_DEBUG_TRACE, "==> translucent_db_destroy\n", 0, 0, 0);
 
        if ( ov ) {
+               if ( ov->remote )
+                       anlist_free( ov->remote, 1, NULL );
+               if ( ov->local )
+                       anlist_free( ov->local, 1, NULL );
                if ( ov->db.be_private != NULL ) {
                        backend_stopdown_one( &ov->db );
                }