andykimpe / rpms / 389-ds-base

Forked from rpms/389-ds-base 4 months ago
Clone

Blame 0336-Ticket-47981-COS-cache-doesn-t-properly-mark-vattr-c.patch

dc8c34
From 9d80397041c2f2051ed9247193c87b324e511d46 Mon Sep 17 00:00:00 2001
dc8c34
From: Noriko Hosoi <nhosoi@redhat.com>
dc8c34
Date: Wed, 2 Sep 2015 18:04:27 -0700
dc8c34
Subject: [PATCH] Ticket #47981 - COS cache doesn't properly mark vattr cache
dc8c34
 as invalid when there are multiple suffixes
dc8c34
dc8c34
Description: commit 42e2df3858a4e14706d57b5c907d1d3768f4d970 for fixing
dc8c34
icket 47981 accidentally added "break" to the while loop when a
dc8c34
condition is satisfied:
dc8c34
  if(!cos_cache_add_dn_defs(suffixVals[valIndex]->bv_val ,pDefs))
dc8c34
which skips the rest of the definitions.  This patch removes the
dc8c34
"break".
dc8c34
dc8c34
https://fedorahosted.org/389/ticket/47981
dc8c34
dc8c34
Reviewed by mreynolds@redhat.com (Thank you, Mark!!)
dc8c34
dc8c34
(cherry picked from commit 6557b820dca7980067afc2a33184197b2d154a51)
dc8c34
(cherry picked from commit c1721f1d6e2344eefaec817ed47119c15c43fcfc)
dc8c34
(cherry picked from commit 546aa6b1f4db1eefc426d3dcb6788f5251fd1e82)
dc8c34
(cherry picked from commit d2b69d51c2d1d7d53be881b0e816b5088295a9c1)
dc8c34
---
dc8c34
 ldap/servers/plugins/cos/cos_cache.c | 1 -
dc8c34
 1 file changed, 1 deletion(-)
dc8c34
dc8c34
diff --git a/ldap/servers/plugins/cos/cos_cache.c b/ldap/servers/plugins/cos/cos_cache.c
dc8c34
index f41c3de..94c8d77 100644
dc8c34
--- a/ldap/servers/plugins/cos/cos_cache.c
dc8c34
+++ b/ldap/servers/plugins/cos/cos_cache.c
dc8c34
@@ -716,7 +716,6 @@ static int cos_cache_build_definition_list(cosDefinitions **pDefs, int *vattr_ca
dc8c34
 									{
dc8c34
 										*vattr_cacheable = -1;
dc8c34
 										cos_def_available = 1;
dc8c34
-										break;
dc8c34
 									}
dc8c34
 								}
dc8c34
 								valIndex++;
dc8c34
-- 
dc8c34
1.9.3
dc8c34