]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/overlays/rwm.c
Partial revert of f30269f5d2e4bb5ee7486fe6542078d1b59dba6d
[openldap] / servers / slapd / overlays / rwm.c
index b07b7e18a4d056f660c12582f3f97ea4dee55885..cc6211ed2508d4084d84e088afea42fe18a5471a 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2003-2009 The OpenLDAP Foundation.
+ * Copyright 2003-2012 The OpenLDAP Foundation.
  * Portions Copyright 2003 Pierangelo Masarati.
  * All rights reserved.
  *
@@ -34,6 +34,8 @@ typedef struct rwm_op_state {
        struct berval ro_ndn;
        struct berval r_dn;
        struct berval r_ndn;
+       struct berval rx_dn;
+       struct berval rx_ndn;
        AttributeName *mapped_attrs;
        OpRequest o_request;
 } rwm_op_state;
@@ -52,30 +54,46 @@ rwm_send_entry( Operation *op, SlapReply *rs );
 static void
 rwm_op_rollback( Operation *op, SlapReply *rs, rwm_op_state *ros )
 {
-       if ( !BER_BVISNULL( &ros->ro_dn ) ) {
-               op->o_req_dn = ros->ro_dn;
-       }
-       if ( !BER_BVISNULL( &ros->ro_ndn ) ) {
-               op->o_req_ndn = ros->ro_ndn;
-       }
+       /* in case of successful extended operation cleanup
+        * gets called *after* (ITS#6632); this hack counts
+        * on others to cleanup our o_req_dn/o_req_ndn,
+        * while we cleanup theirs. */
+       if ( ros->r_tag == LDAP_REQ_EXTENDED && rs->sr_err == LDAP_SUCCESS ) {
+               if ( !BER_BVISNULL( &ros->rx_dn ) ) {
+                       ch_free( ros->rx_dn.bv_val );
+               }
+               if ( !BER_BVISNULL( &ros->rx_ndn ) ) {
+                       ch_free( ros->rx_ndn.bv_val );
+               }
 
-       if ( !BER_BVISNULL( &ros->r_dn )
-               && ros->r_dn.bv_val != ros->ro_dn.bv_val )
-       {
-               assert( ros->r_dn.bv_val != ros->r_ndn.bv_val );
-               ch_free( ros->r_dn.bv_val );
-               BER_BVZERO( &ros->r_dn );
-       }
+       } else {
+               if ( !BER_BVISNULL( &ros->ro_dn ) ) {
+                       op->o_req_dn = ros->ro_dn;
+               }
+               if ( !BER_BVISNULL( &ros->ro_ndn ) ) {
+                       op->o_req_ndn = ros->ro_ndn;
+               }
 
-       if ( !BER_BVISNULL( &ros->r_ndn )
-               && ros->r_ndn.bv_val != ros->ro_ndn.bv_val )
-       {
-               ch_free( ros->r_ndn.bv_val );
-               BER_BVZERO( &ros->r_ndn );
+               if ( !BER_BVISNULL( &ros->r_dn )
+                       && ros->r_dn.bv_val != ros->ro_dn.bv_val )
+               {
+                       assert( ros->r_dn.bv_val != ros->r_ndn.bv_val );
+                       ch_free( ros->r_dn.bv_val );
+               }
+
+               if ( !BER_BVISNULL( &ros->r_ndn )
+                       && ros->r_ndn.bv_val != ros->ro_ndn.bv_val )
+               {
+                       ch_free( ros->r_ndn.bv_val );
+               }
        }
 
+       BER_BVZERO( &ros->r_dn );
+       BER_BVZERO( &ros->r_ndn );
        BER_BVZERO( &ros->ro_dn );
        BER_BVZERO( &ros->ro_ndn );
+       BER_BVZERO( &ros->rx_dn );
+       BER_BVZERO( &ros->rx_ndn );
 
        switch( ros->r_tag ) {
        case LDAP_REQ_COMPARE:
@@ -104,9 +122,9 @@ rwm_op_rollback( Operation *op, SlapReply *rs, rwm_op_state *ros )
                }
                break;
        case LDAP_REQ_SEARCH:
-               ch_free( ros->mapped_attrs );
+               op->o_tmpfree( ros->mapped_attrs, op->o_tmpmemctx );
                filter_free_x( op, op->ors_filter, 1 );
-               ch_free( op->ors_filterstr.bv_val );
+               op->o_tmpfree( op->ors_filterstr.bv_val, op->o_tmpmemctx );
                op->ors_attrs = ros->ors_attrs;
                op->ors_filter = ros->ors_filter;
                op->ors_filterstr = ros->ors_filterstr;
@@ -122,7 +140,7 @@ rwm_op_rollback( Operation *op, SlapReply *rs, rwm_op_state *ros )
 #if 0
                        ldap_pvt_thread_mutex_lock( &op->o_conn->c_mutex );
                        /* too late, c_mutex released */
-                       fprintf( stderr, "*** DN: \"%s\" => \"%s\"\n",
+                       Debug( LDAP_DEBUG_ANY, "*** DN: \"%s\" => \"%s\"\n",
                                op->o_conn->c_ndn.bv_val,
                                op->o_req_ndn.bv_val );
                        ber_bvreplace( &op->o_conn->c_ndn,
@@ -154,9 +172,9 @@ rwm_op_cleanup( Operation *op, SlapReply *rs )
 }
 
 static rwm_op_cb *
-rwm_callback_get( Operation *op, SlapReply *rs )
+rwm_callback_get( Operation *op )
 {
-       rwm_op_cb       *roc = NULL;
+       rwm_op_cb       *roc;
 
        roc = op->o_tmpalloc( sizeof( struct rwm_op_cb ), op->o_tmpmemctx );
        roc->cb.sc_cleanup = rwm_op_cleanup;
@@ -166,9 +184,12 @@ rwm_callback_get( Operation *op, SlapReply *rs )
        roc->ros.r_tag = op->o_tag;
        roc->ros.ro_dn = op->o_req_dn;
        roc->ros.ro_ndn = op->o_req_ndn;
-       roc->ros.o_request = op->o_request;
        BER_BVZERO( &roc->ros.r_dn );
        BER_BVZERO( &roc->ros.r_ndn );
+       BER_BVZERO( &roc->ros.rx_dn );
+       BER_BVZERO( &roc->ros.rx_ndn );
+       roc->ros.mapped_attrs = NULL;
+       roc->ros.o_request = op->o_request;
 
        return roc;
 }
@@ -228,6 +249,11 @@ rwm_op_dn_massage( Operation *op, SlapReply *rs, void *cookie,
        assert( BER_BVISNULL( &ros->r_ndn ) );
        ros->r_ndn = ndn;
 
+       if ( ros->r_tag == LDAP_REQ_EXTENDED ) {
+               ros->rx_dn = ros->r_dn;
+               ros->rx_ndn = ros->r_ndn;
+       }
+
        return LDAP_SUCCESS;
 }
 
@@ -244,7 +270,7 @@ rwm_op_add( Operation *op, SlapReply *rs )
        char                    *olddn = op->o_req_dn.bv_val;
        int                     isupdate;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "addDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -387,7 +413,7 @@ rwm_op_bind( Operation *op, SlapReply *rs )
        slap_overinst           *on = (slap_overinst *) op->o_bd->bd_info;
        int                     rc;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "bindDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -423,7 +449,7 @@ rwm_op_compare( Operation *op, SlapReply *rs )
        int                     rc;
        struct berval           mapped_vals[2] = { BER_BVNULL, BER_BVNULL };
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "compareDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -508,7 +534,7 @@ rwm_op_delete( Operation *op, SlapReply *rs )
        slap_overinst           *on = (slap_overinst *) op->o_bd->bd_info;
        int                     rc;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "deleteDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -533,7 +559,7 @@ rwm_op_modify( Operation *op, SlapReply *rs )
        Modifications           **mlp;
        int                     rc;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "modifyDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -575,7 +601,7 @@ rwm_op_modify( Operation *op, SlapReply *rs )
                                        &mapping, RWM_MAP );
                        if ( drop_missing || ( mapping != NULL && BER_BVISNULL( &mapping->m_dst ) ) )
                        {
-                               goto cleanup_mod;
+                               goto skip_mod;
                        }
                }
 
@@ -670,6 +696,10 @@ next_mod:;
                mlp = &ml->sml_next;
                continue;
 
+skip_mod:;
+               *mlp = (*mlp)->sml_next;
+               continue;
+
 cleanup_mod:;
                ml = *mlp;
                *mlp = (*mlp)->sml_next;
@@ -692,7 +722,7 @@ rwm_op_modrdn( Operation *op, SlapReply *rs )
        int                     rc;
        dncookie                dc;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        if ( op->orr_newSup ) {
                struct berval   nnewSup = BER_BVNULL;
@@ -823,7 +853,7 @@ rwm_entry_release_rw( Operation *op, Entry *e, int rw )
        }
 
        /* just free entry if (probably) ours */
-       if ( e->e_private == NULL ) {
+       if ( e->e_private == NULL && BER_BVISNULL( &e->e_bv ) ) {
                entry_free( e );
                return LDAP_SUCCESS;
        }
@@ -836,17 +866,13 @@ rwm_entry_get_rw( Operation *op, struct berval *ndn,
        ObjectClass *oc, AttributeDescription *at, int rw, Entry **ep )
 {
        slap_overinst           *on = (slap_overinst *) op->o_bd->bd_info;
-       struct ldaprwmap        *rwmap = 
-                       (struct ldaprwmap *)on->on_bi.bi_private;
-
        int                     rc;
-       dncookie                dc;
-
        BackendDB               db;
        Operation               op2;
        SlapReply               rs = { REP_SEARCH };
 
        rwm_op_state            ros = { 0 };
+       struct berval           mndn = BER_BVNULL;
 
        if ( ((BackendInfo *)on->on_info->oi_orig)->bi_entry_get_rw == NULL ) {
                return SLAP_CB_CONTINUE;
@@ -862,6 +888,8 @@ rwm_entry_get_rw( Operation *op, struct berval *ndn,
                return LDAP_OTHER;
        }
 
+       mndn = BER_BVISNULL( &ros.r_ndn ) ? *ndn : ros.r_ndn;
+
        /* map attribute & objectClass */
        if ( at != NULL ) {
        }
@@ -874,20 +902,29 @@ rwm_entry_get_rw( Operation *op, struct berval *ndn,
        op2.o_bd = &db;
        op2.o_bd->bd_info = (BackendInfo *)on->on_info->oi_orig;
        op2.ors_attrs = slap_anlist_all_attributes;
-       rc = op2.o_bd->bd_info->bi_entry_get_rw( &op2, &ros.r_ndn, oc, at, rw, ep );
+       rc = op2.o_bd->bd_info->bi_entry_get_rw( &op2, &mndn, oc, at, rw, ep );
        if ( rc == LDAP_SUCCESS && *ep != NULL ) {
+               /* we assume be_entry_release() needs to be called */
+               rs.sr_flags = REP_ENTRY_MUSTRELEASE;
                rs.sr_entry = *ep;
 
                /* duplicate & release */
                op2.o_bd->bd_info = (BackendInfo *)on;
                rc = rwm_send_entry( &op2, &rs );
+               RS_ASSERT( rs.sr_flags & REP_ENTRY_MUSTFLUSH );
                if ( rc == SLAP_CB_CONTINUE ) {
                        *ep = rs.sr_entry;
                        rc = LDAP_SUCCESS;
+               } else {
+                       assert( rc != LDAP_SUCCESS && rs.sr_entry == *ep );
+                       *ep = NULL;
+                       op2.o_bd->bd_info = (BackendInfo *)on->on_info;
+                       be_entry_release_r( &op2, rs.sr_entry );
+                       op2.o_bd->bd_info = (BackendInfo *)on;
                }
        }
 
-       if ( ros.r_ndn.bv_val != ndn->bv_val ) {
+       if ( !BER_BVISNULL( &ros.r_ndn) && ros.r_ndn.bv_val != ndn->bv_val ) {
                op->o_tmpfree( ros.r_ndn.bv_val, op->o_tmpmemctx );
        }
 
@@ -911,7 +948,7 @@ rwm_op_search( Operation *op, SlapReply *rs )
 
        char                    *text = NULL;
 
-       rwm_op_cb               *roc = rwm_callback_get( op, rs );
+       rwm_op_cb               *roc = rwm_callback_get( op );
 
        rc = rewrite_session_var_set( rwmap->rwm_rw, op->o_conn,
                "searchFilter", op->ors_filterstr.bv_val );
@@ -946,7 +983,7 @@ rwm_op_search( Operation *op, SlapReply *rs )
        op->ors_filter = f;
        op->ors_filterstr = fstr;
 
-       rc = rwm_map_attrnames( &rwmap->rwm_at, &rwmap->rwm_oc,
+       rc = rwm_map_attrnames( op, &rwmap->rwm_at, &rwmap->rwm_oc,
                        op->ors_attrs, &an, RWM_MAP );
        if ( rc != LDAP_SUCCESS ) {
                text = "attribute list mapping error";
@@ -973,7 +1010,7 @@ error_return:;
        }
 
        if ( !BER_BVISNULL( &fstr ) ) {
-               ch_free( fstr.bv_val );
+               op->o_tmpfree( fstr.bv_val, op->o_tmpmemctx );
        }
 
        rwm_op_rollback( op, rs, &roc->ros );
@@ -1030,7 +1067,7 @@ rwm_exop_passwd( Operation *op, SlapReply *rs )
                ber_dupbv_x( &op->o_req_ndn, &op->o_ndn, op->o_tmpmemctx );
        }
 
-       roc = rwm_callback_get( op, rs );
+       roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "extendedDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -1102,7 +1139,7 @@ rwm_extended( Operation *op, SlapReply *rs )
                }
        }
 
-       roc = rwm_callback_get( op, rs );
+       roc = rwm_callback_get( op );
 
        rc = rwm_op_dn_massage( op, rs, "extendedDN", &roc->ros );
        if ( rc != LDAP_SUCCESS ) {
@@ -1117,7 +1154,7 @@ rwm_extended( Operation *op, SlapReply *rs )
        return SLAP_CB_CONTINUE;
 }
 
-static int
+static void
 rwm_matched( Operation *op, SlapReply *rs )
 {
        slap_overinst           *on = (slap_overinst *) op->o_bd->bd_info;
@@ -1129,7 +1166,7 @@ rwm_matched( Operation *op, SlapReply *rs )
        int                     rc;
 
        if ( rs->sr_matched == NULL ) {
-               return SLAP_CB_CONTINUE;
+               return;
        }
 
        dc.rwmap = rwmap;
@@ -1142,10 +1179,8 @@ rwm_matched( Operation *op, SlapReply *rs )
        if ( rc != LDAP_SUCCESS ) {
                rs->sr_err = rc;
                rs->sr_text = "Rewrite error";
-               return 1;
-       }
 
-       if ( mdn.bv_val != dn.bv_val ) {
+       } else if ( mdn.bv_val != dn.bv_val ) {
                if ( rs->sr_flags & REP_MATCHED_MUSTBEFREED ) {
                        ch_free( (void *)rs->sr_matched );
 
@@ -1154,8 +1189,6 @@ rwm_matched( Operation *op, SlapReply *rs )
                }
                rs->sr_matched = mdn.bv_val;
        }
-       
-       return SLAP_CB_CONTINUE;
 }
 
 static int
@@ -1197,76 +1230,77 @@ rwm_attrs( Operation *op, SlapReply *rs, Attribute** a_first, int stripEntryDN )
                int                     last = -1;
                Attribute               *a;
 
-               if ( SLAP_OPATTRS( rs->sr_attr_flags ) && is_at_operational( (*ap)->a_desc->ad_type ) )
+               if ( ( rwmap->rwm_flags & RWM_F_DROP_UNREQUESTED_ATTRS ) &&
+                               op->ors_attrs != NULL && 
+                               !SLAP_USERATTRS( rs->sr_attr_flags ) &&
+                               !ad_inlist( (*ap)->a_desc, op->ors_attrs ) )
                {
-                       /* go on */ ;
-                       
-               } else {
-                       if ( op->ors_attrs != NULL && 
-                                       !SLAP_USERATTRS( rs->sr_attr_flags ) &&
-                                       !ad_inlist( (*ap)->a_desc, op->ors_attrs ) )
-                       {
-                               goto cleanup_attr;
-                       }
+                       goto cleanup_attr;
+               }
 
-                       drop_missing = rwm_mapping( &rwmap->rwm_at,
-                                       &(*ap)->a_desc->ad_cname, &mapping, RWM_REMAP );
-                       if ( drop_missing || ( mapping != NULL && BER_BVISEMPTY( &mapping->m_dst ) ) )
-                       {
-                               goto cleanup_attr;
-                       }
-                       if ( mapping != NULL ) {
-                               assert( mapping->m_dst_ad != NULL );
+               drop_missing = rwm_mapping( &rwmap->rwm_at,
+                               &(*ap)->a_desc->ad_cname, &mapping, RWM_REMAP );
+               if ( drop_missing || ( mapping != NULL && BER_BVISEMPTY( &mapping->m_dst ) ) )
+               {
+                       goto cleanup_attr;
+               }
+               if ( mapping != NULL ) {
+                       assert( mapping->m_dst_ad != NULL );
 
-                               /* try to normalize mapped Attributes if the original 
-                                * AttributeType was not normalized */
-                               if ( (!(*ap)->a_desc->ad_type->sat_equality || 
-                                       !(*ap)->a_desc->ad_type->sat_equality->smr_normalize) &&
-                                       mapping->m_dst_ad->ad_type->sat_equality &&
-                                       mapping->m_dst_ad->ad_type->sat_equality->smr_normalize )
+                       /* try to normalize mapped Attributes if the original 
+                        * AttributeType was not normalized */
+                       if ( (!(*ap)->a_desc->ad_type->sat_equality || 
+                               !(*ap)->a_desc->ad_type->sat_equality->smr_normalize) &&
+                               mapping->m_dst_ad->ad_type->sat_equality &&
+                               mapping->m_dst_ad->ad_type->sat_equality->smr_normalize )
+                       {
+                               if ((rwmap->rwm_flags & RWM_F_NORMALIZE_MAPPED_ATTRS))
                                {
-                                       if ((rwmap->rwm_flags & RWM_F_NORMALIZE_MAPPED_ATTRS))
+                                       int i = 0;
+
+                                       last = (*ap)->a_numvals;
+                                       if ( last )
                                        {
-                                               int i = 0;
-
-                                               last = (*ap)->a_numvals;
-                                               if ( last )
-                                               {
-                                                       (*ap)->a_nvals = ch_malloc( (last+1) * sizeof(struct berval) );
-
-                                                       for ( i = 0; !BER_BVISNULL( &(*ap)->a_vals[i]); i++ ) {
-                                                               int             rc;
-                                                               /*
-                                                                * check that each value is valid per syntax
-                                                                * and pretty if appropriate
+                                               (*ap)->a_nvals = ch_malloc( (last+1) * sizeof(struct berval) );
+
+                                               for ( i = 0; !BER_BVISNULL( &(*ap)->a_vals[i]); i++ ) {
+                                                       int             rc;
+                                                       /*
+                                                        * check that each value is valid per syntax
+                                                        * and pretty if appropriate
+                                                        */
+                                                       rc = mapping->m_dst_ad->ad_type->sat_equality->smr_normalize(
+                                                               SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
+                                                               mapping->m_dst_ad->ad_type->sat_syntax,
+                                                               mapping->m_dst_ad->ad_type->sat_equality,
+                                                               &(*ap)->a_vals[i], &(*ap)->a_nvals[i],
+                                                               NULL );
+
+                                                       if ( rc != LDAP_SUCCESS ) {
+                                                               /* FIXME: this is wrong, putting a non-normalized value
+                                                                * into nvals. But when a proxy sends us bogus data,
+                                                                * we still need to give it to the client, even if it
+                                                                * violates the syntax. I.e., we don't want to silently
+                                                                * drop things and trigger an apparent data loss.
                                                                 */
-                                                               rc = mapping->m_dst_ad->ad_type->sat_equality->smr_normalize(
-                                                                       SLAP_MR_VALUE_OF_ATTRIBUTE_SYNTAX,
-                                                                       mapping->m_dst_ad->ad_type->sat_syntax,
-                                                                       mapping->m_dst_ad->ad_type->sat_equality,
-                                                                       &(*ap)->a_vals[i], &(*ap)->a_nvals[i],
-                                                                       NULL );
-
-                                                               if ( rc != LDAP_SUCCESS ) {
-                                                                       BER_BVZERO( &(*ap)->a_nvals[i] );
-                                                               }
+                                                               ber_dupbv( &(*ap)->a_nvals[i], &(*ap)->a_vals[i] );
                                                        }
-                                                       BER_BVZERO( &(*ap)->a_nvals[i] );
                                                }
-
-                                       } else {
-                                               assert( (*ap)->a_nvals == (*ap)->a_vals );
-                                               (*ap)->a_nvals = NULL;
-                                               ber_bvarray_dup_x( &(*ap)->a_nvals, (*ap)->a_vals, NULL );
+                                               BER_BVZERO( &(*ap)->a_nvals[i] );
                                        }
+
+                               } else {
+                                       assert( (*ap)->a_nvals == (*ap)->a_vals );
+                                       (*ap)->a_nvals = NULL;
+                                       ber_bvarray_dup_x( &(*ap)->a_nvals, (*ap)->a_vals, NULL );
                                }
+                       }
 
-                               /* rewrite the attribute description */
-                               (*ap)->a_desc = mapping->m_dst_ad;
+                       /* rewrite the attribute description */
+                       (*ap)->a_desc = mapping->m_dst_ad;
 
-                               /* will need to check for duplicate attrs */
-                               check_duplicate_attrs++;
-                       }
+                       /* will need to check for duplicate attrs */
+                       check_duplicate_attrs++;
                }
 
                if ( (*ap)->a_desc == slap_schema.si_ad_entryDN ) {
@@ -1431,6 +1465,7 @@ cleanup_attr:;
        return 0;
 }
 
+/* Should return SLAP_CB_CONTINUE or failure, never LDAP_SUCCESS. */
 static int
 rwm_send_entry( Operation *op, SlapReply *rs )
 {
@@ -1439,7 +1474,6 @@ rwm_send_entry( Operation *op, SlapReply *rs )
                        (struct ldaprwmap *)on->on_bi.bi_private;
 
        Entry                   *e = NULL;
-       slap_mask_t             flags;
        struct berval           dn = BER_BVNULL,
                                ndn = BER_BVNULL;
        dncookie                dc;
@@ -1456,7 +1490,6 @@ rwm_send_entry( Operation *op, SlapReply *rs )
        dc.ctx = "searchEntryDN";
 
        e = rs->sr_entry;
-       flags = rs->sr_flags;
        if ( !( rs->sr_flags & REP_ENTRY_MODIFIABLE ) ) {
                /* FIXME: all we need to duplicate are:
                 * - dn
@@ -1464,15 +1497,17 @@ rwm_send_entry( Operation *op, SlapReply *rs )
                 * - attributes that are requested
                 * - no values if attrsonly is set
                 */
-
                e = entry_dup( e );
                if ( e == NULL ) {
                        rc = LDAP_NO_MEMORY;
                        goto fail;
                }
-
-               flags &= ~REP_ENTRY_MUSTRELEASE;
-               flags |= ( REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED );
+       } else if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+               /* ITS#6423: REP_ENTRY_MUSTRELEASE incompatible
+                * with REP_ENTRY_MODIFIABLE */
+               RS_ASSERT( 0 );
+               rc = 1;
+               goto fail;
        }
 
        /*
@@ -1504,13 +1539,21 @@ rwm_send_entry( Operation *op, SlapReply *rs )
         * to return, and remap them accordingly */
        (void)rwm_attrs( op, rs, &e->e_attrs, 1 );
 
-       if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
-               overlay_entry_release_ov( op, rs->sr_entry, 0, on );
+       if ( e != rs->sr_entry ) {
+               /* Reimplementing rs_replace_entry(), I suppose to
+                * bypass our own dubious rwm_entry_release_rw() */
+               if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+                       rs->sr_flags ^= REP_ENTRY_MUSTRELEASE;
+                       op->o_bd->bd_info = (BackendInfo *)on->on_info;
+                       be_entry_release_r( op, rs->sr_entry );
+                       op->o_bd->bd_info = (BackendInfo *)on;
+               } else if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
+                       entry_free( rs->sr_entry );
+               }
+               rs->sr_entry = e;
+               rs->sr_flags |= REP_ENTRY_MODIFIABLE | REP_ENTRY_MUSTBEFREED;
        }
 
-       rs->sr_entry = e;
-       rs->sr_flags = flags;
-
        return SLAP_CB_CONTINUE;
 
 fail:;
@@ -1617,12 +1660,12 @@ rwm_suffixmassage_config(
         */
        if ( argc == 2 ) {
                if ( be->be_suffix == NULL ) {
-                       fprintf( stderr, "%s: line %d: "
+                       Debug( LDAP_DEBUG_ANY, "%s: line %d: "
                                       " \"suffixMassage [<suffix>]"
                                       " <massaged suffix>\" without "
                                       "<suffix> part requires database "
                                       "suffix be defined first.\n",
-                               fname, lineno );
+                               fname, lineno, 0 );
                        return 1;
                }
                bvnc = be->be_suffix[ 0 ];
@@ -1633,22 +1676,22 @@ rwm_suffixmassage_config(
                massaged = 2;
 
        } else  {
-               fprintf( stderr, "%s: line %d: syntax is"
+               Debug( LDAP_DEBUG_ANY, "%s: line %d: syntax is"
                               " \"suffixMassage [<suffix>]"
                               " <massaged suffix>\"\n",
-                       fname, lineno );
+                       fname, lineno, 0 );
                return 1;
        }
 
        if ( dnPrettyNormal( NULL, &bvnc, &pvnc, &nvnc, NULL ) != LDAP_SUCCESS ) {
-               fprintf( stderr, "%s: line %d: suffix DN %s is invalid\n",
+               Debug( LDAP_DEBUG_ANY, "%s: line %d: suffix DN %s is invalid\n",
                        fname, lineno, bvnc.bv_val );
                return 1;
        }
 
        ber_str2bv( argv[ massaged ], 0, 0, &brnc );
        if ( dnPrettyNormal( NULL, &brnc, &prnc, &nrnc, NULL ) != LDAP_SUCCESS ) {
-               fprintf( stderr, "%s: line %d: suffix DN %s is invalid\n",
+               Debug( LDAP_DEBUG_ANY, "%s: line %d: suffix DN %s is invalid\n",
                                fname, lineno, brnc.bv_val );
                free( nvnc.bv_val );
                free( pvnc.bv_val );
@@ -1720,20 +1763,18 @@ rwm_response( Operation *op, SlapReply *rs )
                        dc.rs = NULL; 
                        dc.ctx = "referralDN";
                        rc = rwm_referral_result_rewrite( &dc, rs->sr_ref );
+                       /* FIXME: impossible, so far */
                        if ( rc != LDAP_SUCCESS ) {
-                               rc = 1;
+                               rs->sr_err = rc;
                                break;
                        }
                }
-               rc = rwm_matched( op, rs );
-               break;
 
-       default:
-               rc = SLAP_CB_CONTINUE;
+               rwm_matched( op, rs );
                break;
        }
 
-       return rc;
+       return SLAP_CB_CONTINUE;
 }
 
 static int
@@ -1767,9 +1808,9 @@ rwm_db_config(
 
        } else if ( strcasecmp( argv[0], "t-f-support" ) == 0 ) {
                if ( argc != 2 ) {
-                       fprintf( stderr,
+                       Debug( LDAP_DEBUG_ANY,
                "%s: line %d: \"t-f-support {no|yes|discover}\" needs 1 argument.\n",
-                                       fname, lineno );
+                                       fname, lineno, 0 );
                        return( 1 );
                }
 
@@ -1781,17 +1822,17 @@ rwm_db_config(
 
                /* TODO: not implemented yet */
                } else if ( strcasecmp( argv[ 1 ], "discover" ) == 0 ) {
-                       fprintf( stderr,
+                       Debug( LDAP_DEBUG_ANY,
                "%s: line %d: \"discover\" not supported yet "
                "in \"t-f-support {no|yes|discover}\".\n",
-                                       fname, lineno );
+                                       fname, lineno, 0 );
                        return( 1 );
 #if 0
                        rwmap->rwm_flags |= RWM_F_SUPPORT_T_F_DISCOVER;
 #endif
 
                } else {
-                       fprintf( stderr,
+                       Debug( LDAP_DEBUG_ANY,
        "%s: line %d: unknown value \"%s\" for \"t-f-support {no|yes|discover}\".\n",
                                fname, lineno, argv[ 1 ] );
                        return 1;
@@ -1799,9 +1840,9 @@ rwm_db_config(
 
        } else if ( strcasecmp( argv[0], "normalize-mapped-attrs" ) ==  0 ) {
                if ( argc !=2 ) { 
-                       fprintf( stderr,
+                       Debug( LDAP_DEBUG_ANY,
                "%s: line %d: \"normalize-mapped-attrs {no|yes}\" needs 1 argument.\n",
-                                       fname, lineno );
+                                       fname, lineno, 0 );
                        return( 1 );
                }
 
@@ -1835,6 +1876,7 @@ enum {
        RWM_CF_MAP,
        RWM_CF_T_F_SUPPORT,
        RWM_CF_NORMALIZE_MAPPED,
+       RWM_CF_DROP_UNREQUESTED,
 
        RWM_CF_LAST
 };
@@ -1877,6 +1919,7 @@ static ConfigTable rwmcfg[] = {
                2, 4, 0, ARG_MAGIC|RWM_CF_MAP, rwm_cf_gen,
                "( OLcfgOvAt:16.3 NAME 'olcRwmMap' "
                        "DESC 'maps attributes/objectClasses' "
+                       "EQUALITY caseIgnoreMatch "
                        "SYNTAX OMsDirectoryString "
                        "X-ORDERED 'VALUES' )",
                NULL, NULL },
@@ -1889,6 +1932,14 @@ static ConfigTable rwmcfg[] = {
                        "SINGLE-VALUE )",
                NULL, NULL },
 
+       { "rwm-drop-unrequested-attrs", "true|false",
+               2, 2, 0, ARG_MAGIC|ARG_ON_OFF|RWM_CF_DROP_UNREQUESTED, rwm_cf_gen,
+               "( OLcfgOvAt:16.5 NAME 'olcRwmDropUnrequested' "
+                       "DESC 'Drop unrequested attributes' "
+                       "SYNTAX OMsBoolean "
+                       "SINGLE-VALUE )",
+               NULL, NULL },
+
        { NULL, NULL, 0, 0, 0, ARG_IGNORED }
 };
 
@@ -1908,7 +1959,7 @@ static ConfigOCs rwmocs[] = {
 };
 
 static void
-slap_rewrite_unparse( BerVarray in, BerVarray *out )
+slap_bv_x_ordered_unparse( BerVarray in, BerVarray *out )
 {
        int             i;
        BerVarray       bva = NULL;
@@ -1948,10 +1999,10 @@ slap_rewrite_unparse( BerVarray in, BerVarray *out )
 }
 
 static int
-rwm_bva_rewrite_add(
-       struct ldaprwmap        *rwmap,
+rwm_bva_add(
+       BerVarray               *bva,
        int                     idx,
-       const char              *argv[] )
+       char                    **argv )
 {
        char            *line;
        struct berval   bv;
@@ -1966,16 +2017,38 @@ rwm_bva_rewrite_add(
                bv.bv_val[ bv.bv_len - 1 ] = '"';
 
                if ( idx == -1 ) {
-                       ber_bvarray_add( &rwmap->rwm_bva_rewrite, &bv );
+                       ber_bvarray_add( bva, &bv );
 
                } else {
-                       rwmap->rwm_bva_rewrite[ idx ] = bv;
+                       (*bva)[ idx ] = bv;
                }
+
+               return 0;
        }
 
-       return 0;
+       return -1;
+}
+
+static int
+rwm_bva_rewrite_add(
+       struct ldaprwmap        *rwmap,
+       int                     idx,
+       char                    **argv )
+{
+       return rwm_bva_add( &rwmap->rwm_bva_rewrite, idx, argv );
 }
 
+#ifdef unused
+static int
+rwm_bva_map_add(
+       struct ldaprwmap        *rwmap,
+       int                     idx,
+       char                    **argv )
+{
+       return rwm_bva_add( &rwmap->rwm_bva_map, idx, argv );
+}
+#endif /* unused */
+
 static int
 rwm_info_init( struct rewrite_info ** rwm_rw )
 {
@@ -2025,7 +2098,7 @@ rwm_cf_gen( ConfigArgs *c )
                                rc = 1;
 
                        } else {
-                               slap_rewrite_unparse( rwmap->rwm_bva_rewrite, &c->rvalue_vals );
+                               slap_bv_x_ordered_unparse( rwmap->rwm_bva_rewrite, &c->rvalue_vals );
                                if ( !c->rvalue_vals ) {
                                        rc = 1;
                                }
@@ -2049,7 +2122,10 @@ rwm_cf_gen( ConfigArgs *c )
                                rc = 1;
 
                        } else {
-                               value_add( &c->rvalue_vals, rwmap->rwm_bva_map );
+                               slap_bv_x_ordered_unparse( rwmap->rwm_bva_map, &c->rvalue_vals );
+                               if ( !c->rvalue_vals ) {
+                                       rc = 1;
+                               }
                        }
                        break;
 
@@ -2057,6 +2133,10 @@ rwm_cf_gen( ConfigArgs *c )
                        c->value_int = ( rwmap->rwm_flags & RWM_F_NORMALIZE_MAPPED_ATTRS );
                        break;
 
+               case RWM_CF_DROP_UNREQUESTED:
+                       c->value_int = ( rwmap->rwm_flags & RWM_F_DROP_UNREQUESTED_ATTRS );
+                       break;
+
                default:
                        assert( 0 );
                        rc = 1;
@@ -2068,13 +2148,12 @@ rwm_cf_gen( ConfigArgs *c )
                switch ( c->type ) {
                case RWM_CF_REWRITE:
                        if ( c->valx >= 0 ) {
-                               ConfigArgs ca = { 0 };
                                int i;
 
                                for ( i = 0; !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ i ] ); i++ )
                                        /* count'em */ ;
 
-                               if ( i >= c->valx ) {
+                               if ( c->valx >= i ) {
                                        rc = 1;
                                        break;
                                }
@@ -2093,6 +2172,8 @@ rwm_cf_gen( ConfigArgs *c )
 
                                for ( i = 0; !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ i ] ); i++ )
                                {
+                                       ConfigArgs ca = { 0 };
+
                                        ca.line = rwmap->rwm_bva_rewrite[ i ].bv_val;
                                        ca.argc = 0;
                                        config_fp_parse_line( &ca );
@@ -2107,6 +2188,7 @@ rwm_cf_gen( ConfigArgs *c )
                                        }
 
                                        ch_free( ca.tline );
+                                       ch_free( ca.argv );
 
                                        assert( rc == 0 );
                                }
@@ -2128,8 +2210,74 @@ rwm_cf_gen( ConfigArgs *c )
 
                case RWM_CF_MAP:
                        if ( c->valx >= 0 ) {
-                               /* single modification is not allowed */
-                               rc = 1;
+                               struct ldapmap rwm_oc = rwmap->rwm_oc;
+                               struct ldapmap rwm_at = rwmap->rwm_at;
+                               char *argv[5];
+                               int cnt = 0;
+
+                               if ( rwmap->rwm_bva_map ) {
+                                       for ( ; !BER_BVISNULL( &rwmap->rwm_bva_map[ cnt ] ); cnt++ )
+                                               /* count */ ;
+                               }
+
+                               if ( c->valx >= cnt ) {
+                                       rc = 1;
+                                       break;
+                               }
+
+                               memset( &rwmap->rwm_oc, 0, sizeof( rwmap->rwm_oc ) );
+                               memset( &rwmap->rwm_at, 0, sizeof( rwmap->rwm_at ) );
+
+                               /* re-parse all mappings except the one
+                                * that needs to be eliminated */
+                               argv[0] = "map";
+                               for ( cnt = 0; !BER_BVISNULL( &rwmap->rwm_bva_map[ cnt ] ); cnt++ ) {
+                                       ConfigArgs ca = { 0 };
+
+                                       if ( cnt == c->valx ) {
+                                               continue;
+                                       }
+
+                                       ca.line = rwmap->rwm_bva_map[ cnt ].bv_val;
+                                       ca.argc = 0;
+                                       config_fp_parse_line( &ca );
+                                       
+                                       argv[1] = ca.argv[0];
+                                       argv[2] = ca.argv[1];
+                                       argv[3] = ca.argv[2];
+                                       argv[4] = ca.argv[3];
+                       
+                                       rc = rwm_m_config( &db, c->fname, c->lineno, ca.argc + 1, argv );
+
+                                       ch_free( ca.tline );
+                                       ch_free( ca.argv );
+
+                                       /* in case of failure, restore
+                                        * the existing mapping */
+                                       if ( rc ) {
+                                               avl_free( rwmap->rwm_oc.remap, rwm_mapping_dst_free );
+                                               avl_free( rwmap->rwm_oc.map, rwm_mapping_free );
+                                               avl_free( rwmap->rwm_at.remap, rwm_mapping_dst_free );
+                                               avl_free( rwmap->rwm_at.map, rwm_mapping_free );
+                                               rwmap->rwm_oc = rwm_oc;
+                                               rwmap->rwm_at = rwm_at;
+                                               break;
+                                       }
+                               }
+
+                               /* in case of success, destroy the old mapping
+                                * and eliminate the deleted one */
+                               if ( rc == 0 ) {
+                                       avl_free( rwm_oc.remap, rwm_mapping_dst_free );
+                                       avl_free( rwm_oc.map, rwm_mapping_free );
+                                       avl_free( rwm_at.remap, rwm_mapping_dst_free );
+                                       avl_free( rwm_at.map, rwm_mapping_free );
+
+                                       ber_memfree( rwmap->rwm_bva_map[ c->valx ].bv_val );
+                                       for ( cnt = c->valx; !BER_BVISNULL( &rwmap->rwm_bva_map[ cnt ] ); cnt++ ) {
+                                               rwmap->rwm_bva_map[ cnt ] = rwmap->rwm_bva_map[ cnt + 1 ];
+                                       }
+                               }
 
                        } else {
                                avl_free( rwmap->rwm_oc.remap, rwm_mapping_dst_free );
@@ -2151,6 +2299,10 @@ rwm_cf_gen( ConfigArgs *c )
                        rwmap->rwm_flags &= ~RWM_F_NORMALIZE_MAPPED_ATTRS;
                        break;
 
+               case RWM_CF_DROP_UNREQUESTED:
+                       rwmap->rwm_flags &= ~RWM_F_DROP_UNREQUESTED_ATTRS;
+                       break;
+
                default:
                        return 1;
                }
@@ -2165,10 +2317,9 @@ rwm_cf_gen( ConfigArgs *c )
        case RWM_CF_REWRITE:
                if ( c->valx >= 0 ) {
                        struct rewrite_info *rwm_rw = rwmap->rwm_rw;
-                       ConfigArgs ca = { 0 };
                        int i, last;
 
-                       for ( last = 0; !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ last ] ); last++ )
+                       for ( last = 0; rwmap->rwm_bva_rewrite && !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ last ] ); last++ )
                                /* count'em */ ;
 
                        if ( c->valx > last ) {
@@ -2179,6 +2330,8 @@ rwm_cf_gen( ConfigArgs *c )
                        rc = rwm_info_init( &rwmap->rwm_rw );
 
                        for ( i = 0; i < c->valx; i++ ) {
+                               ConfigArgs ca = { 0 };
+
                                ca.line = rwmap->rwm_bva_rewrite[ i ].bv_val;
                                ca.argc = 0;
                                config_fp_parse_line( &ca );
@@ -2198,6 +2351,7 @@ rwm_cf_gen( ConfigArgs *c )
                                ca.argv[ 0 ] = argv0;
 
                                ch_free( ca.tline );
+                               ch_free( ca.argv );
 
                                assert( rc == 0 );
                        }
@@ -2224,8 +2378,10 @@ rwm_cf_gen( ConfigArgs *c )
                                return 1;
                        }
 
-                       for ( i = c->valx; !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ i ] ); i++ )
+                       for ( i = c->valx; rwmap->rwm_bva_rewrite && !BER_BVISNULL( &rwmap->rwm_bva_rewrite[ i ] ); i++ )
                        {
+                               ConfigArgs ca = { 0 };
+
                                ca.line = rwmap->rwm_bva_rewrite[ i ].bv_val;
                                ca.argc = 0;
                                config_fp_parse_line( &ca );
@@ -2245,12 +2401,14 @@ rwm_cf_gen( ConfigArgs *c )
                                ca.argv[ 0 ] = argv0;
 
                                ch_free( ca.tline );
+                               ch_free( ca.argv );
 
                                assert( rc == 0 );
                        }
 
                        rwmap->rwm_bva_rewrite = ch_realloc( rwmap->rwm_bva_rewrite,
                                ( last + 2 )*sizeof( struct berval ) );
+                       BER_BVZERO( &rwmap->rwm_bva_rewrite[last+1] );
 
                        for ( i = last - 1; i >= c->valx; i-- )
                        {
@@ -2300,7 +2458,126 @@ rwm_cf_gen( ConfigArgs *c )
 
        case RWM_CF_MAP:
                if ( c->valx >= 0 ) {
-                       return 1;
+                       struct ldapmap rwm_oc = rwmap->rwm_oc;
+                       struct ldapmap rwm_at = rwmap->rwm_at;
+                       char *argv[5];
+                       int cnt = 0;
+
+                       if ( rwmap->rwm_bva_map ) {
+                               for ( ; !BER_BVISNULL( &rwmap->rwm_bva_map[ cnt ] ); cnt++ )
+                                       /* count */ ;
+                       }
+
+                       if ( c->valx >= cnt ) {
+                               c->valx = cnt;
+                       }
+
+                       memset( &rwmap->rwm_oc, 0, sizeof( rwmap->rwm_oc ) );
+                       memset( &rwmap->rwm_at, 0, sizeof( rwmap->rwm_at ) );
+
+                       /* re-parse all mappings, including the one
+                        * that needs to be added */
+                       argv[0] = "map";
+                       for ( cnt = 0; cnt < c->valx; cnt++ ) {
+                               ConfigArgs ca = { 0 };
+
+                               ca.line = rwmap->rwm_bva_map[ cnt ].bv_val;
+                               ca.argc = 0;
+                               config_fp_parse_line( &ca );
+
+                               argv[1] = ca.argv[0];
+                               argv[2] = ca.argv[1];
+                               argv[3] = ca.argv[2];
+                               argv[4] = ca.argv[3];
+                       
+                               rc = rwm_m_config( &db, c->fname, c->lineno, ca.argc + 1, argv );
+
+                               ch_free( ca.tline );
+                               ch_free( ca.argv );
+
+                               /* in case of failure, restore
+                                * the existing mapping */
+                               if ( rc ) {
+                                       goto rwmmap_fail;
+                               }
+                       }
+
+                       argv0 = c->argv[0];
+                       c->argv[0] = "map";
+                       rc = rwm_m_config( &db, c->fname, c->lineno, c->argc, c->argv );
+                       c->argv[0] = argv0;
+                       if ( rc ) {
+                               goto rwmmap_fail;
+                       }
+
+                       if ( rwmap->rwm_bva_map ) {
+                               for ( ; !BER_BVISNULL( &rwmap->rwm_bva_map[ cnt ] ); cnt++ ) {
+                                       ConfigArgs ca = { 0 };
+
+                                       ca.line = rwmap->rwm_bva_map[ cnt ].bv_val;
+                                       ca.argc = 0;
+                                       config_fp_parse_line( &ca );
+                       
+                                       argv[1] = ca.argv[0];
+                                       argv[2] = ca.argv[1];
+                                       argv[3] = ca.argv[2];
+                                       argv[4] = ca.argv[3];
+                       
+                                       rc = rwm_m_config( &db, c->fname, c->lineno, ca.argc + 1, argv );
+
+                                       ch_free( ca.tline );
+                                       ch_free( ca.argv );
+
+                                       /* in case of failure, restore
+                                        * the existing mapping */
+                                       if ( rc ) {
+                                               goto rwmmap_fail;
+                                       }
+                               }
+                       }
+
+                       /* in case of success, destroy the old mapping
+                        * and add the new one */
+                       if ( rc == 0 ) {
+                               BerVarray tmp;
+                               struct berval bv, *bvp = &bv;
+
+                               if ( rwm_bva_add( &bvp, 0, &c->argv[ idx0 ] ) ) {
+                                       rc = 1;
+                                       goto rwmmap_fail;
+                               }
+                                       
+                               tmp = ber_memrealloc( rwmap->rwm_bva_map,
+                                       sizeof( struct berval )*( cnt + 2 ) );
+                               if ( tmp == NULL ) {
+                                       ber_memfree( bv.bv_val );
+                                       rc = 1;
+                                       goto rwmmap_fail;
+                               }
+                               rwmap->rwm_bva_map = tmp;
+                               BER_BVZERO( &rwmap->rwm_bva_map[ cnt + 1 ] );
+
+                               avl_free( rwm_oc.remap, rwm_mapping_dst_free );
+                               avl_free( rwm_oc.map, rwm_mapping_free );
+                               avl_free( rwm_at.remap, rwm_mapping_dst_free );
+                               avl_free( rwm_at.map, rwm_mapping_free );
+
+                               for ( ; cnt-- > c->valx; ) {
+                                       rwmap->rwm_bva_map[ cnt + 1 ] = rwmap->rwm_bva_map[ cnt ];
+                               }
+                               rwmap->rwm_bva_map[ c->valx ] = bv;
+
+                       } else {
+rwmmap_fail:;
+                               avl_free( rwmap->rwm_oc.remap, rwm_mapping_dst_free );
+                               avl_free( rwmap->rwm_oc.map, rwm_mapping_free );
+                               avl_free( rwmap->rwm_at.remap, rwm_mapping_dst_free );
+                               avl_free( rwmap->rwm_at.map, rwm_mapping_free );
+                               rwmap->rwm_oc = rwm_oc;
+                               rwmap->rwm_at = rwm_at;
+                       }
+
+                       break;
                }
 
                argv0 = c->argv[ 0 ];
@@ -2330,6 +2607,14 @@ rwm_cf_gen( ConfigArgs *c )
                }
                break;
 
+       case RWM_CF_DROP_UNREQUESTED:
+               if ( c->value_int ) {
+                       rwmap->rwm_flags |= RWM_F_DROP_UNREQUESTED_ATTRS;
+               } else {
+                       rwmap->rwm_flags &= ~RWM_F_DROP_UNREQUESTED_ATTRS;
+               }
+               break;
+
        default:
                assert( 0 );
                return 1;
@@ -2349,9 +2634,11 @@ rwm_db_init(
 
        rwmap = (struct ldaprwmap *)ch_calloc( 1, sizeof( struct ldaprwmap ) );
 
+       /* default */
+       rwmap->rwm_flags = RWM_F_DROP_UNREQUESTED_ATTRS;
+
        rc = rwm_info_init( &rwmap->rwm_rw );
 
-error_return:;
        on->on_bi.bi_private = (void *)rwmap;
 
        if ( rc ) {
@@ -2375,7 +2662,8 @@ rwm_db_destroy(
 
                if ( rwmap->rwm_rw ) {
                        rewrite_info_delete( &rwmap->rwm_rw );
-                       ber_bvarray_free( rwmap->rwm_bva_rewrite );
+                       if ( rwmap->rwm_bva_rewrite )
+                               ber_bvarray_free( rwmap->rwm_bva_rewrite );
                }
 
                avl_free( rwmap->rwm_oc.remap, rwm_mapping_dst_free );