andykimpe / rpms / 389-ds-base

Forked from rpms/389-ds-base 4 months ago
Clone
Blob Blame History Raw
From 7ec70c93f5cec20b8bd35b715c0aa00a913dfa63 Mon Sep 17 00:00:00 2001
From: Noriko Hosoi <nhosoi@redhat.com>
Date: Wed, 1 Jul 2015 18:16:20 -0700
Subject: [PATCH 351/363] Ticket #48212 - Dynamic nsMatchingRule changes had no
 effect on the attrinfo thus following reindexing, as well.

Description: When nsMatchingRule was dynamically updated in an index entry,
the value was set to the configuration but was not applied to the attribute
info.  On-line reindexing following the nsMatchingRule change actually ignored
the setting. On the other hand, the standalone utility dbverify independently
picked up the nsMatchingRule from the configuration and generated the attribute
info, which expected the index reindexed based upon the new nsMatchingRule. But
it was actually not and dbverify reported the index corruption.

This patch applies the changes to the attribute info when nsMatchingRule is
modified.

https://fedorahosted.org/389/ticket/48212

Reviewed by rmeggins@redhat.com (Thank you, Rich!)

(cherry picked from commit d9679725e69df1d191864ca00bad6b79b13e7362)
(cherry picked from commit d15beff66fd28902bd8ca80af12ad76a7ecbe57d)
(cherry picked from commit 96cb1511b46e35f93ff47a0ad56d6e19a8e06227)
(cherry picked from commit 84b03a890b7773fce0adb79e692b6e4d9c70d574)
---
 ldap/servers/slapd/back-ldbm/ldbm_attr.c | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/ldap/servers/slapd/back-ldbm/ldbm_attr.c b/ldap/servers/slapd/back-ldbm/ldbm_attr.c
index 1992174..db087fe 100644
--- a/ldap/servers/slapd/back-ldbm/ldbm_attr.c
+++ b/ldap/servers/slapd/back-ldbm/ldbm_attr.c
@@ -156,7 +156,16 @@ ainfo_dup(
   attrinfo_delete_idlistinfo(&a->ai_idlistinfo);
   a->ai_idlistinfo = b->ai_idlistinfo;
   b->ai_idlistinfo = NULL;
-  
+
+  /* copy cmp functions and substr lengths */
+  a->ai_key_cmp_fn = b->ai_key_cmp_fn;
+  a->ai_dup_cmp_fn = b->ai_dup_cmp_fn;
+  if (b->ai_substr_lens) {
+    size_t substrlen = sizeof(int) * INDEX_SUBSTRLEN;
+    a->ai_substr_lens = (int *)slapi_ch_calloc(1, substrlen);
+    memcpy(a->ai_substr_lens, b->ai_substr_lens, substrlen);
+  }
+
   return( 1 );
 }
 
-- 
2.4.3