From: Howard Chu <hyc@openldap.org>
Date: Mon, 14 Apr 2003 11:14:07 +0000 (+0000)
Subject: Fix previous commit
X-Git-Tag: OPENLDAP_REL_ENG_2_2_0ALPHA~365
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=4739b58f9980b16d4b77f085984ae2fe1a6792cc;p=openldap

Fix previous commit
---

diff --git a/servers/slapd/modify.c b/servers/slapd/modify.c
index 2ffa3767b0..b6399a9d7a 100644
--- a/servers/slapd/modify.c
+++ b/servers/slapd/modify.c
@@ -355,9 +355,8 @@ do_modify(
 	 * Calling slapi_x_modifications2ldapmods() destroyed modlist so
 	 * we don't need to free it.
 	 */
-	if ( rs->sr_err == 0 ) {
-		slapi_pblock_get( pb, SLAPI_MODIFY_MODS, (void **)&modv );
-		modlist = slapi_x_ldapmods2modifications( modv );
+	slapi_pblock_get( pb, SLAPI_MODIFY_MODS, (void **)&modv );
+	modlist = slapi_x_ldapmods2modifications( modv );
 
 	/*
 	 * NB: it is valid for the plugin to return no modifications
@@ -367,11 +366,10 @@ do_modify(
 	 * then slapi_x_ldapmods2modifications() above will return
 	 * NULL).
 	 */
-		if ( modlist == NULL ) {
-			rs->sr_err = LDAP_SUCCESS;
-			send_ldap_result( op, rs );
-			goto cleanup;
-		}
+	if ( modlist == NULL ) {
+		rs->sr_err = LDAP_SUCCESS;
+		send_ldap_result( op, rs );
+		goto cleanup;
 	}
 #endif /* defined( LDAP_SLAPI ) */