ldap/servers

[Noriko Hosoi]

ldap/servers/plugins/cos/cos_cache.c |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit 6557b820dca7980067afc2a33184197b2d154a51
Author: Noriko Hosoi 
Date:   Wed Sep 2 18:04:27 2015 -0700

Ticket #47981 - COS cache doesn't properly mark vattr cache as invalid when there are multiple suffixes

Description: commit 42e2df3858a4e14706d57b5c907d1d3768f4d970 for fixing
    icket 47981 accidentally added "break" to the while loop when a
    condition is satisfied:
      if(!cos_cache_add_dn_defs(suffixVals[valIndex]->bv_val ,pDefs))
    which skips the rest of the definitions.  This patch removes the
    "break".

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

Reviewed by [e-mail] (Thank you, Mark!!)

diff --git a/ldap/servers/plugins/cos/cos_cache.c b/ldap/servers/plugins/cos/cos_cache.c
index ddb85ab..e0b841d 100644
--- a/ldap/servers/plugins/cos/cos_cache.c
+++ b/ldap/servers/plugins/cos/cos_cache.c
@@ -647,7 +647,6 @@ static int cos_cache_build_definition_list(cosDefinitions **pDefs, int *vattr_ca
    								{
    									*vattr_cacheable = -1;
    									cos_def_available = 1;
-										break;
    								}
    							}
    							valIndex++;

--
389 commits mailing list
389-commits@%(host_name)s
http://lists.fedoraproject.org/postorius/[e-mail]