From: Pierangelo Masarati <ando@openldap.org>
Date: Sat, 22 Nov 2008 18:35:19 +0000 (+0000)
Subject: fix previous commit
X-Git-Tag: ACLCHECK_0~1060
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=15236be49545c89e67318d15fd40bbce73c274f2;p=openldap

fix previous commit
---

diff --git a/servers/slapd/back-bdb/modify.c b/servers/slapd/back-bdb/modify.c
index f3f1bb144a..6ef43ca76d 100644
--- a/servers/slapd/back-bdb/modify.c
+++ b/servers/slapd/back-bdb/modify.c
@@ -287,11 +287,11 @@ int bdb_modify_internal(
 				rc = bdb_index_values( op, tid, ap->a_desc,
 					vals, e->e_id, SLAP_INDEX_DELETE_OP );
 				if ( rc != LDAP_SUCCESS ) {
-					attrs_free( e->e_attrs );
-					e->e_attrs = save_attrs;
 					Debug( LDAP_DEBUG_ANY,
 						"%s: attribute \"%s\" index delete failure\n",
 						op->o_log_prefix, ap->a_desc->ad_cname.bv_val, 0 );
+					attrs_free( e->e_attrs );
+					e->e_attrs = save_attrs;
 					return rc;
 				}
 			}
@@ -306,11 +306,11 @@ int bdb_modify_internal(
 				ap->a_nvals,
 				e->e_id, SLAP_INDEX_ADD_OP );
 			if ( rc != LDAP_SUCCESS ) {
-				attrs_free( e->e_attrs );
-				e->e_attrs = save_attrs;
 				Debug( LDAP_DEBUG_ANY,
 				       "%s: attribute \"%s\" index add failure\n",
 					op->o_log_prefix, ap->a_desc->ad_cname.bv_val, 0 );
+				attrs_free( e->e_attrs );
+				e->e_attrs = save_attrs;
 				return rc;
 			}
 		}