]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-meta/search.c
use result of validation (ITS#4028)
[openldap] / servers / slapd / back-meta / search.c
index 888cb7af65739af540e4b200adfa2f2c956bea0b..1f1408d19c49b3c8a2054f4e2385cbff347f200d 100644 (file)
@@ -54,13 +54,13 @@ meta_back_search_start(
 )
 {
        metainfo_t      *mi = ( metainfo_t * )op->o_bd->be_private;
-       struct berval           realbase = op->o_req_dn;
-       int                     realscope = op->ors_scope;
-       ber_len_t               suffixlen = 0;
-       struct berval           mbase = BER_BVNULL; 
-       struct berval           mfilter = BER_BVNULL;
-       char                    **mapped_attrs = NULL;
-       int                     rc;
+       struct berval   realbase = op->o_req_dn;
+       int             realscope = op->ors_scope;
+       ber_len_t       suffixlen = 0;
+       struct berval   mbase = BER_BVNULL; 
+       struct berval   mfilter = BER_BVNULL;
+       char            **mapped_attrs = NULL;
+       int             rc;
 
        /* should we check return values? */
        if ( op->ors_deref != -1 ) {
@@ -76,12 +76,12 @@ meta_back_search_start(
                                ( void * )&op->ors_slimit);
        }
 
-       dc->rwmap = &mi->mi_targets[ candidate ]->mt_rwmap;
+       dc->target = &mi->mi_targets[ candidate ];
 
        /*
         * modifies the base according to the scope, if required
         */
-       suffixlen = mi->mi_targets[ candidate ]->mt_nsuffix.bv_len;
+       suffixlen = mi->mi_targets[ candidate ].mt_nsuffix.bv_len;
        if ( suffixlen > op->o_req_ndn.bv_len ) {
                switch ( op->ors_scope ) {
                case LDAP_SCOPE_SUBTREE:
@@ -93,10 +93,15 @@ meta_back_search_start(
                         * the requested searchBase already passed
                         * thru the candidate analyzer...
                         */
-                       if ( dnIsSuffix( &mi->mi_targets[ candidate ]->mt_nsuffix,
+                       if ( dnIsSuffix( &mi->mi_targets[ candidate ].mt_nsuffix,
                                        &op->o_req_ndn ) )
                        {
-                               realbase = mi->mi_targets[ candidate ]->mt_nsuffix;
+                               realbase = mi->mi_targets[ candidate ].mt_nsuffix;
+#ifdef LDAP_SCOPE_SUBORDINATE
+                               if ( mi->mi_targets[ candidate ].mt_scope == LDAP_SCOPE_SUBORDINATE ) {
+                                       realscope = LDAP_SCOPE_SUBORDINATE;
+                               }
+#endif /* LDAP_SCOPE_SUBORDINATE */
 
                        } else {
                                /*
@@ -106,20 +111,34 @@ meta_back_search_start(
                        }
                        break;
 
+#ifdef LDAP_SCOPE_SUBORDINATE
+               case LDAP_SCOPE_SUBORDINATE:
+#endif /* LDAP_SCOPE_SUBORDINATE */
                case LDAP_SCOPE_ONELEVEL:
                {
-                       struct berval   rdn = mi->mi_targets[ candidate ]->mt_nsuffix;
+                       struct berval   rdn = mi->mi_targets[ candidate ].mt_nsuffix;
                        rdn.bv_len -= op->o_req_ndn.bv_len + STRLENOF( "," );
                        if ( dnIsOneLevelRDN( &rdn )
-                                       && dnIsSuffix( &mi->mi_targets[ candidate ]->mt_nsuffix, &op->o_req_ndn ) )
+                                       && dnIsSuffix( &mi->mi_targets[ candidate ].mt_nsuffix, &op->o_req_ndn ) )
                        {
                                /*
                                 * if there is exactly one level,
                                 * make the target suffix the new
                                 * base, and make scope "base"
                                 */
-                               realbase = mi->mi_targets[ candidate ]->mt_nsuffix;
-                               realscope = LDAP_SCOPE_BASE;
+                               realbase = mi->mi_targets[ candidate ].mt_nsuffix;
+#ifdef LDAP_SCOPE_SUBORDINATE
+                               if ( op->ors_scope == LDAP_SCOPE_SUBORDINATE ) {
+                                       if ( mi->mi_targets[ candidate ].mt_scope == LDAP_SCOPE_SUBORDINATE ) {
+                                               realscope = LDAP_SCOPE_SUBORDINATE;
+                                       } else {
+                                               realscope = LDAP_SCOPE_SUBTREE;
+                                       }
+                               } else
+#endif /* LDAP_SCOPE_SUBORDINATE */
+                               {
+                                       realscope = LDAP_SCOPE_BASE;
+                               }
                                break;
                        } /* else continue with the next case */
                }
@@ -175,7 +194,7 @@ meta_back_search_start(
        /*
         * Maps required attributes
         */
-       rc = ldap_back_map_attrs( &mi->mi_targets[ candidate ]->mt_rwmap.rwm_at,
+       rc = ldap_back_map_attrs( &mi->mi_targets[ candidate ].mt_rwmap.rwm_at,
                        op->ors_attrs, BACKLDAP_MAP, &mapped_attrs );
        if ( rc != LDAP_SUCCESS ) {
                /*
@@ -218,19 +237,19 @@ done:;
 int
 meta_back_search( Operation *op, SlapReply *rs )
 {
-       metainfo_t              *mi = ( metainfo_t * )op->o_bd->be_private;
-       metaconn_t              *mc;
-       metasingleconn_t        *msc;
-       struct timeval          tv = { 0, 0 };
-       LDAPMessage             *res = NULL, *e;
-       int                     rc = 0, sres = LDAP_SUCCESS;
-       char                    *matched = NULL;
-       int                     i, last = 0, ncandidates = 0,
-                               initial_candidates = 0, candidate_match = 0;
-       dncookie                dc;
-       int                     is_ok = 0;
-       void                    *savepriv;
-       SlapReply               *candidates = meta_back_candidates_get( op );
+       metainfo_t      *mi = ( metainfo_t * )op->o_bd->be_private;
+       metaconn_t      *mc;
+       struct timeval  tv = { 0, 0 };
+       time_t          stoptime;
+       LDAPMessage     *res = NULL, *e;
+       int             rc = 0, sres = LDAP_SUCCESS;
+       char            *matched = NULL;
+       int             i, last = 0, ncandidates = 0,
+                       initial_candidates = 0, candidate_match = 0;
+       dncookie        dc;
+       int             is_ok = 0;
+       void            *savepriv;
+       SlapReply       *candidates = meta_back_candidates_get( op );
 
        /*
         * controls are set in ldap_back_dobind()
@@ -249,18 +268,21 @@ meta_back_search( Operation *op, SlapReply *rs )
        /*
         * Inits searches
         */
-       for ( i = 0, msc = &mc->mc_conns[ 0 ]; !META_LAST( msc ); ++i, ++msc ) {
-               candidates[ i ].sr_msgid = -1;
+       for ( i = 0; i < mi->mi_ntargets; i++ ) {
+               metasingleconn_t        *msc = &mc->mc_conns[ i ];
 
-               if ( candidates[ i ].sr_tag != META_CANDIDATE ) {
-                       continue;
-               }
-               candidates[ i ].sr_err = LDAP_SUCCESS;
+               candidates[ i ].sr_msgid = -1;
                candidates[ i ].sr_matched = NULL;
                candidates[ i ].sr_text = NULL;
                candidates[ i ].sr_ref = NULL;
                candidates[ i ].sr_ctrls = NULL;
 
+               if ( candidates[ i ].sr_tag != META_CANDIDATE
+                       || candidates[ i ].sr_err != LDAP_SUCCESS )
+               {
+                       continue;
+               }
+
                switch ( meta_back_search_start( op, rs, &dc, msc, i, candidates ) )
                {
                case 0:
@@ -298,10 +320,33 @@ meta_back_search( Operation *op, SlapReply *rs )
 #endif
 
        if ( initial_candidates == 0 ) {
-               send_ldap_error( op, rs, LDAP_NO_SUCH_OBJECT, NULL );
-               /* FIXME: find a way to look up the best match */
-
+               /* NOTE: here we are not sending any matchedDN;
+                * this is intended, because if the back-meta
+                * is serving this search request, but no valid
+                * candidate could be looked up, it means that
+                * there is a hole in the mapping of the targets
+                * and thus no knowledge of any remote superior
+                * is available */
+               Debug( LDAP_DEBUG_ANY, "%s meta_back_search: "
+                       "base=\"%s\" scope=%d: "
+                       "no candidate could be selected\n",
+                       op->o_log_prefix, op->o_req_dn.bv_val,
+                       op->ors_scope );
+
+               /* FIXME: we're sending the first error we encounter;
+                * maybe we should pick the worst... */
                rc = LDAP_NO_SUCH_OBJECT;
+               for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                       if ( candidates[ i ].sr_tag == META_CANDIDATE
+                               && candidates[ i ].sr_err != LDAP_SUCCESS )
+                       {
+                               rc = candidates[ i ].sr_err;
+                               break;
+                       }
+               }
+
+               send_ldap_error( op, rs, rc, NULL );
+
                goto finish;
        }
 
@@ -310,6 +355,11 @@ meta_back_search( Operation *op, SlapReply *rs )
         * but this is necessary for version matching, and for ACL processing.
         */
 
+       if ( op->ors_tlimit != SLAP_NO_LIMIT ) {
+               stoptime = op->o_time + op->ors_tlimit;
+               tv.tv_sec = 0;
+       }
+
        /*
         * In case there are no candidates, no cycle takes place...
         *
@@ -319,7 +369,9 @@ meta_back_search( Operation *op, SlapReply *rs )
        for ( rc = 0; ncandidates > 0; ) {
                int     gotit = 0, doabandon = 0;
 
-               for ( i = 0, msc = &mc->mc_conns[ 0 ]; !META_LAST( msc ); msc++, i++ ) {
+               for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                       metasingleconn_t        *msc = &mc->mc_conns[ i ];
+
                        if ( candidates[ i ].sr_msgid == -1 ) {
                                continue;
                        }
@@ -329,16 +381,6 @@ meta_back_search( Operation *op, SlapReply *rs )
                                break;
                        }
                        
-                       if ( op->ors_slimit > 0 && rs->sr_nentries == op->ors_slimit )
-                       {
-                               rs->sr_err = LDAP_SIZELIMIT_EXCEEDED;
-                               savepriv = op->o_private;
-                               op->o_private = (void *)i;
-                               send_ldap_result( op, rs );
-                               op->o_private = savepriv;
-                               goto finish;
-                       }
-
                        /*
                         * FIXME: handle time limit as well?
                         * Note that target servers are likely 
@@ -350,11 +392,22 @@ meta_back_search( Operation *op, SlapReply *rs )
                                        0, &tv, &res );
 
                        if ( rc == 0 ) {
-                               /* timeout exceeded */
-
                                /* FIXME: res should not need to be freed */
                                assert( res == NULL );
 
+                               /* check time limit */
+                               if ( op->ors_tlimit != SLAP_NO_LIMIT
+                                               && slap_get_time() > stoptime )
+                               {
+                                       doabandon = 1;
+                                       rc = rs->sr_err = LDAP_TIMELIMIT_EXCEEDED;
+                                       savepriv = op->o_private;
+                                       op->o_private = (void *)i;
+                                       send_ldap_result( op, rs );
+                                       op->o_private = savepriv;
+                                       goto finish;
+                               }
+
                                continue;
 
                        } else if ( rc == -1 ) {
@@ -375,6 +428,18 @@ really_bad:;
                                goto finish;
 
                        } else if ( rc == LDAP_RES_SEARCH_ENTRY ) {
+                               if ( --op->ors_slimit == -1 ) {
+                                       ldap_msgfree( res );
+                                       res = NULL;
+
+                                       rs->sr_err = LDAP_SIZELIMIT_EXCEEDED;
+                                       savepriv = op->o_private;
+                                       op->o_private = (void *)i;
+                                       send_ldap_result( op, rs );
+                                       op->o_private = savepriv;
+                                       goto finish;
+                               }
+
                                is_ok++;
 
                                e = ldap_first_entry( msc->msc_ld, res );
@@ -388,6 +453,7 @@ really_bad:;
 
                                gotit = 1;
 
+#if 0
                                /*
                                 * If scope is BASE, we need to jump out
                                 * as soon as one entry is found; if
@@ -395,6 +461,10 @@ really_bad:;
                                 * this should correspond to the sole
                                 * entry that has the base DN
                                 */
+                               /* FIXME: this defeats the purpose of
+                                * doing a search with scope == base and
+                                * sizelimit = 1 to determine if a
+                                * candidate is actually unique */
                                if ( op->ors_scope == LDAP_SCOPE_BASE
                                                && rs->sr_nentries > 0 )
                                {
@@ -403,6 +473,7 @@ really_bad:;
                                        sres = LDAP_SUCCESS;
                                        break;
                                }
+#endif
 
                        } else if ( rc == LDAP_RES_SEARCH_REFERENCE ) {
                                char            **references = NULL;
@@ -474,7 +545,7 @@ really_bad:;
                                                        (char **)&candidates[ i ].sr_matched,
                                                        NULL /* (char **)&candidates[ i ].sr_text */ ,
                                                        &references,
-                                                       &candidates[ i ].sr_ctrls, 1 ) )
+                                                       &candidates[ i ].sr_ctrls, 1 ) != LDAP_SUCCESS )
                                {
                                        res = NULL;
                                        ldap_get_option( msc->msc_ld,
@@ -483,6 +554,7 @@ really_bad:;
                                        sres = slap_map_api2result( rs );
                                        goto really_bad;
                                }
+
                                rs->sr_err = candidates[ i ].sr_err;
                                sres = slap_map_api2result( rs );
                                res = NULL;
@@ -500,7 +572,7 @@ really_bad:;
                                                        0, 0, &match );
 
                                                dc.ctx = "matchedDN";
-                                               dc.rwmap = &mi->mi_targets[ i ]->mt_rwmap;
+                                               dc.target = &mi->mi_targets[ i ];
 
                                                if ( !ldap_back_dn_massage( &dc, &match, &mmatch ) ) {
                                                        if ( mmatch.bv_val == match.bv_val ) {
@@ -556,6 +628,15 @@ really_bad:;
 
                                rs->sr_err = candidates[ i ].sr_err;
                                sres = slap_map_api2result( rs );
+
+                               snprintf( buf, sizeof( buf ),
+                                       "%s meta_back_search[%d] "
+                                       "match=\"%s\" err=%d\n",
+                                       op->o_log_prefix, i,
+                                       candidates[ i ].sr_matched ? candidates[ i ].sr_matched : "",
+                                       candidates[ i ].sr_err );
+                               Debug( LDAP_DEBUG_ANY, "%s", buf, 0, 0 );
+
                                switch ( sres ) {
                                case LDAP_NO_SUCH_OBJECT:
                                        /* is_ok is touched any time a valid
@@ -573,15 +654,17 @@ really_bad:;
                                case LDAP_REFERRAL:
                                        is_ok++;
                                        break;
-                               }
 
-                               snprintf( buf, sizeof( buf ),
-                                       "%s meta_back_search[%d] "
-                                       "match=\"%s\" err=%d\n",
-                                       op->o_log_prefix, i,
-                                       candidates[ i ].sr_matched ? candidates[ i ].sr_matched : "",
-                                       candidates[ i ].sr_err );
-                               Debug( LDAP_DEBUG_ANY, "%s", buf, 0, 0 );
+                               default:
+                                       if ( META_BACK_ONERR_STOP( mi ) ) {
+                                               savepriv = op->o_private;
+                                               op->o_private = (void *)i;
+                                               send_ldap_result( op, rs );
+                                               op->o_private = savepriv;
+                                               goto finish;
+                                       }
+                                       break;
+                               }
 
                                last = i;
                                rc = 0;
@@ -601,7 +684,9 @@ really_bad:;
 
                /* check for abandon */
                if ( op->o_abandon || doabandon ) {
-                       for ( i = 0, msc = mc->mc_conns; !META_LAST( msc ); msc++, i++ ) {
+                       for ( i = 0; i < mi->mi_ntargets; i++ ) {
+                               metasingleconn_t        *msc = &mc->mc_conns[ i ];
+
                                if ( candidates[ i ].sr_msgid != -1 ) {
                                        ldap_abandon_ext( msc->msc_ld,
                                                candidates[ i ].sr_msgid,
@@ -640,17 +725,52 @@ really_bad:;
         * 
         * FIXME: only the last one gets caught!
         */
-       if ( candidate_match > 0 && rs->sr_nentries > 0 ) {
+       savepriv = op->o_private;
+       op->o_private = (void *)mi->mi_ntargets;
+       if ( candidate_match > 0 ) {
+               struct berval   pmatched = BER_BVNULL;
+
                /* we use the first one */
                for ( i = 0; i < mi->mi_ntargets; i++ ) {
                        if ( candidates[ i ].sr_tag == META_CANDIDATE
                                        && candidates[ i ].sr_matched )
                        {
-                               matched = (char *)candidates[ i ].sr_matched;
-                               candidates[ i ].sr_matched = NULL;
-                               break;
+                               struct berval   bv, pbv;
+                               int             rc;
+
+                               ber_str2bv( candidates[ i ].sr_matched, 0, 0, &bv );
+                               rc = dnPretty( NULL, &bv, &pbv, op->o_tmpmemctx );
+
+                               if ( rc == LDAP_SUCCESS ) {
+
+                                       /* NOTE: if they all are superiors
+                                        * of the baseDN, the shorter is also 
+                                        * superior of the longer... */
+                                       if ( pbv.bv_len > pmatched.bv_len ) {
+                                               if ( !BER_BVISNULL( &pmatched ) ) {
+                                                       op->o_tmpfree( pmatched.bv_val, op->o_tmpmemctx );
+                                               }
+                                               pmatched = pbv;
+                                               op->o_private = (void *)i;
+
+                                       } else {
+                                               op->o_tmpfree( pbv.bv_val, op->o_tmpmemctx );
+                                       }
+                               }
+
+                               if ( candidates[ i ].sr_matched != NULL ) {
+                                       free( (char *)candidates[ i ].sr_matched );
+                                       candidates[ i ].sr_matched = NULL;
+                               }
                        }
                }
+
+               if ( !BER_BVISNULL( &pmatched ) ) {
+                       matched = pmatched.bv_val;
+               }
+
+       } else if ( sres == LDAP_NO_SUCH_OBJECT ) {
+               matched = op->o_bd->be_suffix[ 0 ].bv_val;
        }
 
 #if 0
@@ -689,16 +809,14 @@ really_bad:;
        rs->sr_err = sres;
        rs->sr_matched = matched;
        rs->sr_ref = ( sres == LDAP_REFERRAL ? rs->sr_v2ref : NULL );
-       savepriv = op->o_private;
-       op->o_private = (void *)mi->mi_ntargets;
        send_ldap_result( op, rs );
        op->o_private = savepriv;
        rs->sr_matched = NULL;
        rs->sr_ref = NULL;
 
 finish:;
-       if ( matched ) {
-               free( matched );
+       if ( matched && matched != op->o_bd->be_suffix[ 0 ].bv_val ) {
+               op->o_tmpfree( matched, op->o_tmpmemctx );
        }
 
        if ( rs->sr_v2ref ) {
@@ -731,7 +849,9 @@ finish:;
                }
        }
 
-       return rc;
+       meta_back_release_conn( op, mc );
+
+       return rs->sr_err;
 }
 
 static int
@@ -747,9 +867,11 @@ meta_send_entry(
        Entry                   ent = { 0 };
        BerElement              ber = *e->lm_ber;
        Attribute               *attr, **attrp;
-       struct berval           *bv, bdn;
+       struct berval           bdn,
+                               dn = BER_BVNULL;
        const char              *text;
        dncookie                dc;
+       int                     rc;
 
        if ( ber_scanf( &ber, "{m{", &bdn ) == LBER_ERROR ) {
                return LDAP_DECODING_ERROR;
@@ -758,12 +880,12 @@ meta_send_entry(
        /*
         * Rewrite the dn of the result, if needed
         */
-       dc.rwmap = &mi->mi_targets[ target ]->mt_rwmap;
+       dc.target = &mi->mi_targets[ target ];
        dc.conn = op->o_conn;
        dc.rs = rs;
        dc.ctx = "searchResult";
 
-       rs->sr_err = ldap_back_dn_massage( &dc, &bdn, &ent.e_name );
+       rs->sr_err = ldap_back_dn_massage( &dc, &bdn, &dn );
        if ( rs->sr_err != LDAP_SUCCESS) {
                return rs->sr_err;
        }
@@ -775,9 +897,14 @@ meta_send_entry(
         * 
         * FIXME: should we log anything, or delegate to dnNormalize?
         */
-       if ( dnNormalize( 0, NULL, NULL, &ent.e_name, &ent.e_nname,
-               op->o_tmpmemctx ) != LDAP_SUCCESS )
-       {
+       rc = dnPrettyNormal( NULL, &dn, &ent.e_name, &ent.e_nname,
+               op->o_tmpmemctx );
+       if ( dn.bv_val != bdn.bv_val ) {
+               free( dn.bv_val );
+       }
+       BER_BVZERO( &dn );
+
+       if ( rc != LDAP_SUCCESS ) {
                return LDAP_INVALID_DN_SYNTAX;
        }
 
@@ -793,9 +920,11 @@ meta_send_entry(
 
        dc.ctx = "searchAttrDN";
        while ( ber_scanf( &ber, "{m", &a ) != LBER_ERROR ) {
-               int             last = 0;
+               int                             last = 0;
+               slap_syntax_validate_func       *validate;
+               slap_syntax_transform_func      *pretty;
 
-               ldap_back_map( &mi->mi_targets[ target ]->mt_rwmap.rwm_at, 
+               ldap_back_map( &mi->mi_targets[ target ].mt_rwmap.rwm_at, 
                                &a, &mapped, BACKLDAP_REMAP );
                if ( BER_BVISNULL( &mapped ) || mapped.bv_val[0] == '\0' ) {
                        continue;
@@ -809,8 +938,8 @@ meta_send_entry(
                attr->a_desc = NULL;
                if ( slap_bv2ad( &mapped, &attr->a_desc, &text )
                                != LDAP_SUCCESS) {
-                       if ( slap_bv2undef_ad( &mapped, &attr->a_desc, &text ) 
-                                       != LDAP_SUCCESS )
+                       if ( slap_bv2undef_ad( &mapped, &attr->a_desc, &text,
+                               SLAP_AD_PROXIED ) != LDAP_SUCCESS )
                        {
                                char    buf[ SLAP_TEXT_BUFLEN ];
 
@@ -827,7 +956,9 @@ meta_send_entry(
                }
 
                /* no subschemaSubentry */
-               if ( attr->a_desc == slap_schema.si_ad_subschemaSubentry ) {
+               if ( attr->a_desc == slap_schema.si_ad_subschemaSubentry
+                       || attr->a_desc == slap_schema.si_ad_entryDN )
+               {
 
                        /* 
                         * We eat target's subschemaSubentry because
@@ -835,6 +966,10 @@ meta_send_entry(
                         * to resolve to the appropriate backend;
                         * later, the local subschemaSubentry is
                         * added.
+                        *
+                        * We also eat entryDN because the frontend
+                        * will reattach it without checking if already
+                        * present...
                         */
                        ( void )ber_scanf( &ber, "x" /* [W] */ );
 
@@ -843,16 +978,31 @@ meta_send_entry(
                }
 
                if ( ber_scanf( &ber, "[W]", &attr->a_vals ) == LBER_ERROR 
-                               || attr->a_vals == NULL ) {
+                               || attr->a_vals == NULL )
+               {
                        attr->a_vals = (struct berval *)&slap_dummy_bv;
 
-               } else if ( attr->a_desc == slap_schema.si_ad_objectClass
-                               || attr->a_desc == slap_schema.si_ad_structuralObjectClass ) {
+               } else {
+                       for ( last = 0; !BER_BVISNULL( &attr->a_vals[ last ] ); ++last )
+                               ;
+               }
 
-                       for ( last = 0; !BER_BVISNULL( &attr->a_vals[ last ] ); ++last );
+               validate = attr->a_desc->ad_type->sat_syntax->ssyn_validate;
+               pretty = attr->a_desc->ad_type->sat_syntax->ssyn_pretty;
+
+               if ( !validate && !pretty ) {
+                       attr->a_nvals = NULL;
+                       attr_free( attr );
+                       goto next_attr;
+               }
+
+               if ( attr->a_desc == slap_schema.si_ad_objectClass
+                               || attr->a_desc == slap_schema.si_ad_structuralObjectClass )
+               {
+                       struct berval   *bv;
 
                        for ( bv = attr->a_vals; !BER_BVISNULL( bv ); bv++ ) {
-                               ldap_back_map( &mi->mi_targets[ target ]->mt_rwmap.rwm_oc,
+                               ldap_back_map( &mi->mi_targets[ target ].mt_rwmap.rwm_oc,
                                                bv, &mapped, BACKLDAP_REMAP );
                                if ( BER_BVISNULL( &mapped ) || mapped.bv_val[0] == '\0') {
                                        free( bv->bv_val );
@@ -887,10 +1037,51 @@ meta_send_entry(
 
                } else if ( attr->a_desc == slap_schema.si_ad_ref ) {
                        ldap_back_referral_result_rewrite( &dc, attr->a_vals );
+
+               } else {
+                       int     i;
+
+                       for ( i = 0; i < last; i++ ) {
+                               struct berval   pval;
+                               int             rc;
+
+                               if ( pretty ) {
+                                       rc = pretty( attr->a_desc->ad_type->sat_syntax,
+                                               &attr->a_vals[i], &pval, NULL );
+
+                               } else {
+                                       rc = validate( attr->a_desc->ad_type->sat_syntax,
+                                               &attr->a_vals[i] );
+                               }
+
+                               if ( rc ) {
+                                       LBER_FREE( attr->a_vals[i].bv_val );
+                                       if ( --last == i ) {
+                                               BER_BVZERO( &attr->a_vals[ i ] );
+                                               break;
+                                       }
+                                       attr->a_vals[i] = attr->a_vals[last];
+                                       BER_BVZERO( &attr->a_vals[last] );
+                                       i--;
+                                       continue;
+                               }
+
+                               if ( pretty ) {
+                                       LBER_FREE( attr->a_vals[i].bv_val );
+                                       attr->a_vals[i] = pval;
+                               }
+                       }
+
+                       if ( last == 0 ) {
+                               ch_free( attr->a_vals );
+                               ch_free( attr );
+                               goto next_attr;
+                       }
                }
 
                if ( last && attr->a_desc->ad_type->sat_equality &&
-                       attr->a_desc->ad_type->sat_equality->smr_normalize ) {
+                       attr->a_desc->ad_type->sat_equality->smr_normalize )
+               {
                        int i;
 
                        attr->a_nvals = ch_malloc( ( last + 1 ) * sizeof( struct berval ) );
@@ -910,6 +1101,7 @@ meta_send_entry(
 
                *attrp = attr;
                attrp = &attr->a_next;
+next_attr:;
        }
        rs->sr_entry = &ent;
        rs->sr_attrs = op->ors_attrs;
@@ -917,24 +1109,16 @@ meta_send_entry(
        send_search_entry( op, rs );
        rs->sr_entry = NULL;
        rs->sr_attrs = NULL;
-       while ( ent.e_attrs ) {
-               attr = ent.e_attrs;
-               ent.e_attrs = attr->a_next;
-               if ( attr->a_vals != &slap_dummy_bv ) {
-                       if ( attr->a_nvals != attr->a_vals ) {
-                               ber_bvarray_free( attr->a_nvals );
-                       }
-                       ber_bvarray_free( attr->a_vals );
-               }
-               free( attr );
-       }
        
-       if ( ent.e_dn && ent.e_dn != bdn.bv_val ) {
-               free( ent.e_dn );
+       if ( !BER_BVISNULL( &ent.e_name ) ) {
+               free( ent.e_name.bv_val );
+               BER_BVZERO( &ent.e_name );
        }
-       if ( ent.e_ndn ) {
-               free( ent.e_ndn );
+       if ( !BER_BVISNULL( &ent.e_nname ) ) {
+               free( ent.e_nname.bv_val );
+               BER_BVZERO( &ent.e_nname );
        }
+       entry_clean( &ent );
 
        return LDAP_SUCCESS;
 }