]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-ldbm/modrdn.c
Preliminary ldapAdd support
[openldap] / servers / slapd / back-ldbm / modrdn.c
index f7ed89fd682d7839bf861e3a13809b7fecca2563..4df7478e0cd828f53b8aeb9393c65cf8a897f320 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 1998-2003 The OpenLDAP Foundation.
+ * Copyright 1998-2005 The OpenLDAP Foundation.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -42,16 +42,14 @@ ldbm_back_modrdn(
        AttributeDescription *entry = slap_schema.si_ad_entry;
        struct ldbminfo *li = (struct ldbminfo *) op->o_bd->be_private;
        struct berval   p_dn, p_ndn;
-       struct berval   new_dn = { 0, NULL}, new_ndn = { 0, NULL };
+       struct berval   new_dn = BER_BVNULL, new_ndn = BER_BVNULL;
+       struct berval   old_ndn = BER_BVNULL;
        Entry           *e, *p = NULL;
        Entry           *matched;
        /* LDAP v2 supporting correct attribute handling. */
        LDAPRDN         new_rdn = NULL;
        LDAPRDN         old_rdn = NULL;
        int             isroot = -1;
-#define CAN_ROLLBACK   -1
-#define MUST_DESTROY   1
-       int             rc = CAN_ROLLBACK;
        int             rc_id = 0;
        ID              id = NOID;
        const char      *text = NULL;
@@ -65,18 +63,11 @@ ldbm_back_modrdn(
        Modifications   *mod = NULL;            /* Used to delete old/add new rdn */
        int             manageDSAit = get_manageDSAit( op );
 
-#ifdef NEW_LOGGING
-       LDAP_LOG( BACK_LDBM, ENTRY, 
-               "ldbm_back_modrdn: dn: %s newSuperior=%s\n", 
-               op->o_req_dn.bv_len ? op->o_req_dn.bv_val : "NULL",
-               ( op->oq_modrdn.rs_newSup && op->oq_modrdn.rs_newSup->bv_len ) ? op->oq_modrdn.rs_newSup->bv_val : "NULL",0 );
-#else
        Debug( LDAP_DEBUG_TRACE,
                "==>ldbm_back_modrdn: dn: %s newSuperior=%s\n", 
                op->o_req_dn.bv_len ? op->o_req_dn.bv_val : "NULL",
                ( op->oq_modrdn.rs_newSup && op->oq_modrdn.rs_newSup->bv_len )
                        ? op->oq_modrdn.rs_newSup->bv_val : "NULL", 0 );
-#endif
 
        /* grab giant lock for writing */
        ldap_pvt_thread_rdwr_wlock(&li->li_giant_rwlock);
@@ -93,19 +84,8 @@ ldbm_back_modrdn(
                                : NULL;
                        cache_return_entry_r( &li->li_cache, matched );
                } else {
-                       BerVarray deref = NULL;
-                       if ( !LDAP_STAILQ_EMPTY( &op->o_bd->be_syncinfo )) {
-                               syncinfo_t *si;
-                               LDAP_STAILQ_FOREACH( si, &op->o_bd->be_syncinfo, si_next ) {
-                                       struct berval tmpbv;
-                                       ber_dupbv( &tmpbv, &si->si_provideruri_bv[0] );
-                                       ber_bvarray_add( &deref, &tmpbv );
-                               }
-                       } else {
-                               deref = default_referral;
-                       }
-                       rs->sr_ref = referral_rewrite( deref, NULL, &op->o_req_dn,
-                                                       LDAP_SCOPE_DEFAULT );
+                       rs->sr_ref = referral_rewrite( default_referral, NULL,
+                                               &op->o_req_dn, LDAP_SCOPE_DEFAULT );
                }
 
                ldap_pvt_thread_rdwr_wunlock(&li->li_giant_rwlock);
@@ -124,15 +104,9 @@ ldbm_back_modrdn(
        if ( ! access_allowed( op, e,
                entry, NULL, ACL_WRITE, NULL ) )
        {
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, ERR, 
-                       "ldbm_back_modrdn: no write access to entry of (%s)\n", 
-                       op->o_req_dn.bv_val, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                        "<=- ldbm_back_modrdn: no write access to entry\n", 0,
                        0, 0 );
-#endif
 
                send_ldap_error( op, rs, LDAP_INSUFFICIENT_ACCESS,
                        "no write access to entry" );
@@ -145,13 +119,8 @@ ldbm_back_modrdn(
                /* parent is an alias, don't allow add */
                rs->sr_ref = get_entry_referrals( op, e );
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, INFO, 
-                       "ldbm_back_modrdn: entry %s is a referral\n", e->e_dn, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE, "entry %s is referral\n", e->e_dn,
                    0, 0 );
-#endif
 
                rs->sr_err = LDAP_REFERRAL;
                rs->sr_matched = e->e_name.bv_val;
@@ -164,13 +133,8 @@ ldbm_back_modrdn(
        }
 
        if ( has_children( op->o_bd, e ) ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, INFO, 
-                       "ldbm_back_modrdn: entry %s has children\n", e->e_dn, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE, "entry %s has children\n", e->e_dn,
                    0, 0 );
-#endif
 
                send_ldap_error( op, rs, LDAP_NOT_ALLOWED_ON_NONLEAF,
                    "subtree rename not supported" );
@@ -189,14 +153,8 @@ ldbm_back_modrdn(
                 */
 
                if( (p = dn2entry_w( op->o_bd, &p_ndn, NULL )) == NULL) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( BACK_LDBM, INFO, 
-                               "ldbm_back_modrdn: parent of %s does not exist\n", 
-                               e->e_ndn, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE, "parent does not exist\n",
                                0, 0, 0);
-#endif
 
                        send_ldap_error( op, rs, LDAP_OTHER,
                                "parent entry does not exist" );
@@ -208,29 +166,17 @@ ldbm_back_modrdn(
                if ( ! access_allowed( op, p,
                        children, NULL, ACL_WRITE, NULL ) )
                {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( BACK_LDBM, INFO, 
-                               "ldbm_back_modrdn: no access to parent of (%s)\n", 
-                               e->e_dn, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE, "no access to parent\n", 0,
                                0, 0 );
-#endif
 
                        send_ldap_error( op, rs, LDAP_INSUFFICIENT_ACCESS,
                                NULL );
                        goto return_results;
                }
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, DETAIL1, 
-                       "ldbm_back_modrdn: wr to children of entry %s OK\n", 
-                       p_ndn.bv_val, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                       "ldbm_back_modrdn: wr to children of entry %s OK\n",
                       p_ndn.bv_val, 0, 0 );
-#endif
 
                if ( p_ndn.bv_val == slap_empty_bv.bv_val ) {
                        p_dn = slap_empty_bv;
@@ -238,19 +184,15 @@ ldbm_back_modrdn(
                        dnParent( &e->e_name, &p_dn );
                }
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, DETAIL1, 
-                          "ldbm_back_modrdn: parent dn=%s\n", p_dn.bv_val, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE, "ldbm_back_modrdn: parent dn=%s\n",
                       p_dn.bv_val, 0, 0 );
-#endif
 
        } else {
                /* no parent, must be root to modify rdn */
-               isroot = be_isroot( op->o_bd, &op->o_ndn );
+               isroot = be_isroot( op );
                if ( ! isroot ) {
-                       if ( be_issuffix( op->o_bd, (struct berval *)&slap_empty_bv ) || be_isupdate( op->o_bd, &op->o_ndn ) ) {
+                       if ( be_issuffix( op->o_bd, (struct berval *)&slap_empty_bv )
+                               || be_shadow_update( op ) ) {
                                int     can_access;
                                p = (Entry *)&slap_entry_root;
                                
@@ -260,14 +202,9 @@ ldbm_back_modrdn(
                                                                
                                /* check parent for "children" acl */
                                if ( ! can_access ) {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( BACK_LDBM, ERR,
-                                               "ldbm_back_modrdn: no access to parent \"\"\n", 0,0,0 );
-#else
                                        Debug( LDAP_DEBUG_TRACE,
                                                "<=- ldbm_back_modrdn: no "
                                                "access to parent\n", 0, 0, 0 );
-#endif
 
                                        send_ldap_error( op, rs,
                                                LDAP_INSUFFICIENT_ACCESS,
@@ -276,15 +213,9 @@ ldbm_back_modrdn(
                                }
 
                        } else {
-#ifdef NEW_LOGGING
-                               LDAP_LOG( BACK_LDBM, ERR, 
-                                       "ldbm_back_modrdn: (%s) has no parent & not a root.\n", 
-                                       op->o_ndn, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_TRACE,
                                        "<=- ldbm_back_modrdn: no parent & "
                                        "not root\n", 0, 0, 0);
-#endif
 
                                send_ldap_error( op, rs,
                                        LDAP_INSUFFICIENT_ACCESS,
@@ -293,43 +224,26 @@ ldbm_back_modrdn(
                        }
                }
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, INFO, 
-                  "ldbm_back_modrdn: (%s) no parent, locked root.\n", e->e_dn, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                       "ldbm_back_modrdn: no parent, locked root\n",
                       0, 0, 0 );
-#endif
        }
 
        new_parent_dn = &p_dn;  /* New Parent unless newSuperior given */
 
        if ( op->oq_modrdn.rs_newSup != NULL ) {
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, DETAIL1, 
-                       "ldbm_back_modrdn: new parent \"%s\" requested\n",
-                       op->oq_modrdn.rs_newSup->bv_val, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE, 
                        "ldbm_back_modrdn: new parent \"%s\" requested...\n",
                        op->oq_modrdn.rs_newSup->bv_val, 0, 0 );
-#endif
 
                np_ndn = op->oq_modrdn.rs_nnewSup;
 
                /* newSuperior == oldParent? */
                if ( dn_match( &p_ndn, np_ndn ) ) {
-#ifdef NEW_LOGGING
-                       LDAP_LOG( BACK_LDBM, INFO, "ldbm_back_modrdn: "
-                               "new parent\"%s\" seems to be the same as the "
-                               "old parent \"%s\"\n", op->oq_modrdn.rs_newSup->bv_val, p_dn.bv_val, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE, "ldbm_back_modrdn: "
                                "new parent\"%s\" seems to be the same as the "
                                "old parent \"%s\"\n",
                                op->oq_modrdn.rs_newSup->bv_val, p_dn.bv_val, 0 );
-#endif
 
                        op->oq_modrdn.rs_newSup = NULL; /* ignore newSuperior */
                }
@@ -341,43 +255,26 @@ ldbm_back_modrdn(
 
                if ( op->oq_modrdn.rs_nnewSup->bv_len ) {
                        if( (np = dn2entry_w( op->o_bd, np_ndn, NULL )) == NULL) {
-#ifdef NEW_LOGGING
-                               LDAP_LOG( BACK_LDBM, ERR, 
-                                       "ldbm_back_modrdn: newSup(ndn=%s) not found.\n", 
-                                       np_ndn->bv_val, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_TRACE,
                                    "ldbm_back_modrdn: newSup(ndn=%s) not here!\n",
                                    np_ndn->bv_val, 0, 0);
-#endif
 
                                send_ldap_error( op, rs, LDAP_OTHER,
                                        "newSuperior not found" );
                                goto return_results;
                        }
 
-#ifdef NEW_LOGGING
-                       LDAP_LOG( BACK_LDBM, DETAIL1,
-                               "ldbm_back_modrdn: wr to new parent OK np=%p, id=%ld\n",
-                               (void *) np, np->e_id, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE,
                                "ldbm_back_modrdn: wr to new parent OK np=%p, id=%ld\n",
                                (void *) np, np->e_id, 0 );
-#endif
 
                        /* check newSuperior for "children" acl */
                        if ( !access_allowed( op, np, children, NULL,
                                              ACL_WRITE, NULL ) )
                        {
-#ifdef NEW_LOGGING
-                               LDAP_LOG( BACK_LDBM, INFO,
-                                  "ldbm_back_modrdn: no wr to newSup children.\n", 0, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_TRACE,
                                       "ldbm_back_modrdn: no wr to newSup children\n",
                                       0, 0, 0 );
-#endif
 
                                send_ldap_error( op, rs, LDAP_INSUFFICIENT_ACCESS, NULL );
                                goto return_results;
@@ -385,12 +282,7 @@ ldbm_back_modrdn(
 
                        if ( is_entry_alias( np ) ) {
                                /* parent is an alias, don't allow add */
-#ifdef NEW_LOGGING
-                               LDAP_LOG( BACK_LDBM, INFO,
-                                  "ldbm_back_modrdn: entry (%s) is an alias.\n", np->e_dn,0,0);
-#else
                                Debug( LDAP_DEBUG_TRACE, "entry is alias\n", 0, 0, 0 );
-#endif
 
 
                                send_ldap_error( op, rs, LDAP_ALIAS_PROBLEM,
@@ -401,14 +293,8 @@ ldbm_back_modrdn(
 
                        if ( is_entry_referral( np ) ) {
                                /* parent is a referral, don't allow add */
-#ifdef NEW_LOGGING
-                               LDAP_LOG( BACK_LDBM, INFO,
-                                       "ldbm_back_modrdn: entry (%s) is a referral\n",
-                                       np->e_dn, 0, 0 );
-#else
                                Debug( LDAP_DEBUG_TRACE, "entry (%s) is referral\n",
                                        np->e_dn, 0, 0 );
-#endif
 
                                send_ldap_error( op, rs, LDAP_OTHER,
                                    "newSuperior is a referral" );
@@ -420,11 +306,12 @@ ldbm_back_modrdn(
 
                        /* no parent, must be root to modify newSuperior */
                        if ( isroot == -1 ) {
-                               isroot = be_isroot( op->o_bd, &op->o_ndn );
+                               isroot = be_isroot( op );
                        }
 
                        if ( ! isroot ) {
-                               if ( be_issuffix( op->o_bd, (struct berval *)&slap_empty_bv ) || be_isupdate( op->o_bd, &op->o_ndn ) ) {
+                               if ( be_issuffix( op->o_bd, (struct berval *)&slap_empty_bv )
+                                       || be_shadow_update( op ) ) {
                                        int     can_access;
                                        np = (Entry *)&slap_entry_root;
                                
@@ -434,15 +321,9 @@ ldbm_back_modrdn(
                                                                
                                        /* check parent for "children" acl */
                                        if ( ! can_access ) {
-#ifdef NEW_LOGGING
-                                               LDAP_LOG( BACK_LDBM, ERR,
-                                                       "ldbm_back_modrdn: no access "
-                                                       "to new superior \"\"\n", 0, 0, 0 );
-#else
                                                Debug( LDAP_DEBUG_TRACE,
                                                        "<=- ldbm_back_modrdn: no "
                                                        "access to new superior\n", 0, 0, 0 );
-#endif
 
                                                send_ldap_error( op, rs,
                                                        LDAP_INSUFFICIENT_ACCESS,
@@ -451,16 +332,10 @@ ldbm_back_modrdn(
                                        }
 
                                } else {
-#ifdef NEW_LOGGING
-                                       LDAP_LOG( BACK_LDBM, ERR,
-                                               "ldbm_back_modrdn: \"\" not allowed as new superior\n",
-                                               0, 0, 0 );
-#else
                                        Debug( LDAP_DEBUG_TRACE,
                                                "<=- ldbm_back_modrdn: \"\" "
                                                "not allowed as new superior\n", 
                                                0, 0, 0);
-#endif
 
                                        send_ldap_error( op, rs,
                                                LDAP_INSUFFICIENT_ACCESS,
@@ -470,14 +345,9 @@ ldbm_back_modrdn(
                        }
                }
 
-#ifdef NEW_LOGGING
-               LDAP_LOG( BACK_LDBM, DETAIL1,
-                       "ldbm_back_modrdn: wr to new parent's children OK.\n", 0, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                    "ldbm_back_modrdn: wr to new parent's children OK\n",
                    0, 0, 0 );
-#endif
 
                new_parent_dn = op->oq_modrdn.rs_newSup;
        }
@@ -486,16 +356,12 @@ ldbm_back_modrdn(
        build_new_dn( &new_dn, new_parent_dn, &op->oq_modrdn.rs_newrdn, NULL ); 
        dnNormalize( 0, NULL, NULL, &new_dn, &new_ndn, NULL );
 
-#ifdef NEW_LOGGING
-       LDAP_LOG( BACK_LDBM, DETAIL1, "ldbm_back_modrdn: new ndn=%s\n", 
-               new_ndn.bv_val, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE, "ldbm_back_modrdn: new ndn=%s\n",
            new_ndn.bv_val, 0, 0 );
-#endif
 
        /* check for abandon */
        if ( op->o_abandon ) {
+               rs->sr_err = SLAPD_ABANDON;
                goto return_results;
        }
 
@@ -506,14 +372,9 @@ ldbm_back_modrdn(
                goto return_results;
        }
 
-#ifdef NEW_LOGGING
-       LDAP_LOG( BACK_LDBM, INFO, "ldbm_back_modrdn: new ndn (%s) does not exist\n",
-               new_ndn.bv_val, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE,
            "ldbm_back_modrdn: new ndn=%s does not exist\n",
            new_ndn.bv_val, 0, 0 );
-#endif
 
        /* Get attribute type and attribute value of our new rdn, we will
         * need to add that to our new entry
@@ -521,89 +382,53 @@ ldbm_back_modrdn(
        if ( ldap_bv2rdn( &op->oq_modrdn.rs_newrdn, &new_rdn, (char **)&rs->sr_text,
                LDAP_DN_FORMAT_LDAP ) )
        {
-#ifdef NEW_LOGGING
-               LDAP_LOG ( OPERATION, ERR, 
-                       "ldbm_back_modrdn: can't figure out "
-                       "type(s)/values(s) of newrdn\n", 
-                       0, 0, 0 );
-#else
                Debug( LDAP_DEBUG_TRACE,
                        "ldbm_back_modrdn: can't figure out "
                        "type(s)/values(s) of newrdn\n", 
                        0, 0, 0 );
-#endif
                send_ldap_error( op, rs, LDAP_INVALID_DN_SYNTAX,
                                    "unknown type(s) used in RDN" );
                goto return_results;            
        }
 
-#ifdef NEW_LOGGING
-       LDAP_LOG ( OPERATION, RESULTS, 
-               "ldbm_back_modrdn: new_rdn_type=\"%s\", "
-               "new_rdn_val=\"%s\"\n",
-               new_rdn[ 0 ]->la_attr.bv_val, 
-               new_rdn[ 0 ]->la_value.bv_val, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE,
                "ldbm_back_modrdn: new_rdn_type=\"%s\", "
                "new_rdn_val=\"%s\"\n",
                new_rdn[ 0 ]->la_attr.bv_val,
                new_rdn[ 0 ]->la_value.bv_val, 0 );
-#endif
 
        if ( op->oq_modrdn.rs_deleteoldrdn ) {
                if ( ldap_bv2rdn( &op->o_req_dn, &old_rdn, (char **)&rs->sr_text,
                        LDAP_DN_FORMAT_LDAP ) )
                {
-#ifdef NEW_LOGGING
-                       LDAP_LOG ( OPERATION, ERR, 
-                               "ldbm_back_modrdn: can't figure out "
-                               "type(s)/values(s) of old_rdn\n", 
-                               0, 0, 0 );
-#else
                        Debug( LDAP_DEBUG_TRACE,
                                "ldbm_back_modrdn: can't figure out "
                                "the old_rdn type(s)/value(s)\n", 
                                0, 0, 0 );
-#endif
                        send_ldap_error( op, rs, LDAP_OTHER,
                                    "cannot parse RDN from old DN" );
                        goto return_results;            
                }
        }
 
-#ifdef NEW_LOGGING
-       LDAP_LOG( BACK_LDBM, DETAIL1, "ldbm_back_modrdn:  DN_X500\n", 0, 0, 0 );
-#else
        Debug( LDAP_DEBUG_TRACE, "ldbm_back_modrdn: DN_X500\n",
               0, 0, 0 );
-#endif
        
        if ( slap_modrdn2mods( op, rs, e, old_rdn, new_rdn, &mod ) != LDAP_SUCCESS ) {
                send_ldap_result( op, rs );
                goto return_results;
        }
 
-
        /* check for abandon */
        if ( op->o_abandon ) {
-               goto return_results;
-       }
-
-       /* delete old one */
-       if ( dn2id_delete( op->o_bd, &e->e_nname, e->e_id ) != 0 ) {
-               send_ldap_error( op, rs, LDAP_OTHER,
-                       "DN index delete fail" );
+               rs->sr_err = SLAPD_ABANDON;
                goto return_results;
        }
 
        (void) cache_delete_entry( &li->li_cache, e );
-       rc = MUST_DESTROY;
-
-       /* XXX: there is no going back! */
 
        free( e->e_dn );
-       free( e->e_ndn );
+       old_ndn = e->e_nname;
        e->e_name = new_dn;
        e->e_nname = new_ndn;
        new_dn.bv_val = NULL;
@@ -613,13 +438,6 @@ ldbm_back_modrdn(
         * They are used by cache.
         */
 
-       /* add new one */
-       if ( dn2id_add( op->o_bd, &e->e_nname, e->e_id ) != 0 ) {
-               send_ldap_error( op, rs, LDAP_OTHER,
-                       "DN index add failed" );
-               goto return_results;
-       }
-
        /* modify memory copy of entry */
        rs->sr_err = ldbm_modify_internal( op, &mod[0], e,
                &rs->sr_text, textbuf, textlen );
@@ -627,38 +445,67 @@ ldbm_back_modrdn(
        case LDAP_SUCCESS:
                break;
 
-       case SLAPD_ABANDON:
-               /* too late ... */
-               goto return_results;
-       
        default:
-               /* here we may try to delete the newly added dn */
-               if ( dn2id_delete( op->o_bd, &e->e_nname, e->e_id ) != 0 ) {
-                       /* we already are in trouble ... */
-                       ;
-               }
                send_ldap_result( op, rs );
+               /* FALLTHRU */
+       case SLAPD_ABANDON:
                goto return_results;
        }
        
-       (void) cache_update_entry( &li->li_cache, e );
+       /*
+        * NOTE: the backend MUST delete then add the entry,
+        *              otherwise indexing may get hosed
+        * FIXME: if a new ID was used, the add could be done first.
+        *              that would be safer.
+        */
+
+       /* delete old one */
+       if ( dn2id_delete( op->o_bd, &old_ndn, e->e_id ) != 0 ) {
+               send_ldap_error( op, rs, LDAP_OTHER,
+                       "DN index delete fail" );
+               goto return_results;
+       }
+
+       /* add new one */
+       if ( dn2id_add( op->o_bd, &e->e_nname, e->e_id ) != 0 ) {
+               /* try to repair old entry - probably hopeless */
+        if( dn2id_add( op->o_bd, &old_ndn, e->e_id) != 0 ) {
+                       send_ldap_error( op, rs, LDAP_OTHER,
+                               "DN index add and repair failed" );
+               } else {
+                       send_ldap_error( op, rs, LDAP_OTHER,
+                               "DN index add failed" );
+               }
+               goto return_results;
+       }
 
        /* id2entry index */
        if ( id2entry_add( op->o_bd, e ) != 0 ) {
-               send_ldap_error( op, rs, LDAP_OTHER,
-                       "entry update failed" );
+               /* Try to undo */
+               int rc;
+               rc = dn2id_delete( op->o_bd, &e->e_nname, e->e_id );
+               rc |= dn2id_add( op->o_bd, &old_ndn, e->e_id );
+               if( rc ) {
+                       send_ldap_error( op, rs, LDAP_OTHER,
+                               "entry update and repair failed" );
+               } else {
+                       send_ldap_error( op, rs, LDAP_OTHER,
+                               "entry update failed" );
+               }
                goto return_results;
        }
 
+       (void) cache_update_entry( &li->li_cache, e );
+
        rs->sr_err = LDAP_SUCCESS;
        rs->sr_text = NULL;
        send_ldap_result( op, rs );
-       rc = 0;
        cache_entry_commit( e );
 
 return_results:
        if( new_dn.bv_val != NULL ) free( new_dn.bv_val );
        if( new_ndn.bv_val != NULL ) free( new_ndn.bv_val );
+       if( old_ndn.bv_val != NULL ) free( old_ndn.bv_val );
 
        /* LDAP v2 supporting correct attribute handling. */
        if ( new_rdn != NULL ) {
@@ -695,13 +542,7 @@ return_results:
 
        /* free entry and writer lock */
        cache_return_entry_w( &li->li_cache, e );
-       if ( rc == MUST_DESTROY ) {
-               /* if rc == MUST_DESTROY the entry is uncached 
-                * and its private data is destroyed; 
-                * the entry must be freed */
-               entry_free( e );
-       }
        ldap_pvt_thread_rdwr_wunlock(&li->li_giant_rwlock);
        rs->sr_text = NULL;
-       return( rc );
+       return( rs->sr_err );
 }