From: Pierangelo Masarati <ando@openldap.org>
Date: Tue, 8 Dec 2009 20:53:39 +0000 (+0000)
Subject: guard against simultaneous REP_ENTRY_MUSTRELEASE and REP_ENTRY_MODIFIABLE (related... 
X-Git-Tag: ACLCHECK_0~9
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=85621c882a23d2ca120d762121d99952dd020c45;p=openldap

guard against simultaneous REP_ENTRY_MUSTRELEASE and REP_ENTRY_MODIFIABLE (related to ITS#6423)
---

diff --git a/servers/slapd/overlays/collect.c b/servers/slapd/overlays/collect.c
index 3e5ee67db3..90867fdc8b 100644
--- a/servers/slapd/overlays/collect.c
+++ b/servers/slapd/overlays/collect.c
@@ -386,10 +386,18 @@ collect_response( Operation *op, SlapReply *rs )
 			* don't modify it directly. Make a copy and
 			* work with that instead.
 			*/
-			if ( !( rs->sr_flags & REP_ENTRY_MODIFIABLE )) {
-				rs->sr_entry = entry_dup( rs->sr_entry );
-				rs->sr_flags |= REP_ENTRY_MODIFIABLE |
-					REP_ENTRY_MUSTBEFREED;
+			if ( !( rs->sr_flags & REP_ENTRY_MODIFIABLE ) ) {
+				Entry *e;
+
+				e = entry_dup( rs->sr_entry );
+				if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+					overlay_entry_release_ov( op, rs->sr_entry, 0, on );
+					rs->sr_flags &= ~REP_ENTRY_MUSTRELEASE;
+				} 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;
 			}
 
 			/* Loop for each attribute in this collectinfo */
diff --git a/servers/slapd/overlays/valsort.c b/servers/slapd/overlays/valsort.c
index c5716eee3e..53a2e91925 100644
--- a/servers/slapd/overlays/valsort.c
+++ b/servers/slapd/overlays/valsort.c
@@ -297,9 +297,19 @@ valsort_response( Operation *op, SlapReply *rs )
 		a = attr_find( rs->sr_entry->e_attrs, vi->vi_ad );
 		if ( !a ) continue;
 
-		if (( rs->sr_flags & ( REP_ENTRY_MODIFIABLE|REP_ENTRY_MUSTBEFREED )) !=
-			( REP_ENTRY_MODIFIABLE|REP_ENTRY_MUSTBEFREED )) {
-			rs->sr_entry = entry_dup( rs->sr_entry );
+		if (( rs->sr_flags & ( REP_ENTRY_MODIFIABLE|REP_ENTRY_MUSTBEFREED ) ) !=
+			( REP_ENTRY_MODIFIABLE|REP_ENTRY_MUSTBEFREED ) )
+		{
+			Entry *e;
+
+			e = entry_dup( rs->sr_entry );
+			if ( rs->sr_flags & REP_ENTRY_MUSTRELEASE ) {
+				overlay_entry_release_ov( op, rs->sr_entry, 0, on );
+				rs->sr_flags &= ~REP_ENTRY_MUSTRELEASE;
+			} 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;
 			a = attr_find( rs->sr_entry->e_attrs, vi->vi_ad );
 		}