From: Luke Howard <lukeh@openldap.org>
Date: Wed, 2 Apr 2003 09:35:08 +0000 (+0000)
Subject: Do not return pointers into BerElement we do not own
X-Git-Tag: AUTOCONF_2_57~119
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=9b4bf8a973c8cfea809be9a4b658f785a6b16f2f;p=openldap

Do not return pointers into BerElement we do not own
---

diff --git a/servers/slapd/mra.c b/servers/slapd/mra.c
index 014e9d4367..a92f87bee9 100644
--- a/servers/slapd/mra.c
+++ b/servers/slapd/mra.c
@@ -38,6 +38,7 @@ get_mra(
 	ber_len_t length;
 	struct berval type = { 0, NULL };
 	struct berval value = { 0, NULL };
+	struct berval tmp = { 0, NULL };
 	MatchingRuleAssertion ma;
 
 	memset( &ma, 0, sizeof ma);
@@ -57,7 +58,7 @@ get_mra(
 	}
 
 	if ( tag == LDAP_FILTER_EXT_OID ) {
-		rtag = ber_scanf( ber, "m", &ma.ma_rule_text );
+		rtag = ber_scanf( ber, "m", &tmp );
 		if ( rtag == LBER_ERROR ) {
 #ifdef NEW_LOGGING
 			LDAP_LOG( OPERATION, ERR,
@@ -69,6 +70,7 @@ get_mra(
 			*text = "Error parsing matching rule in matching rule assertion";
 			return SLAPD_DISCONNECT;
 		}
+		ber_dupbv( &ma.ma_rule_text, &tmp );
 
 		rtag = ber_scanf( ber, "t", &tag );
 		if( rtag == LBER_ERROR ) {
@@ -226,13 +228,9 @@ get_mra(
 		ma.ma_rule,
 		SLAP_MR_EXT|SLAP_MR_VALUE_OF_ASSERTION_SYNTAX,
 		&value, &ma.ma_value, text );
-	if ( rc == LDAP_SUCCESS ) {
-		ma.ma_value = value;
-	} else
-#else
-	if( rc != LDAP_SUCCESS )
 #endif
-	{
+
+	if( rc != LDAP_SUCCESS ) {
 		return rc;
 	}