Blame SOURCES/0010-Issue-50431-Fix-regression-from-coverity-fix.patch

232633
From ae86b58838fc3cc5978312b61613c10ce7fb6aa0 Mon Sep 17 00:00:00 2001
232633
From: Mark Reynolds <mreynolds@redhat.com>
232633
Date: Mon, 8 Jul 2019 14:00:28 -0400
232633
Subject: [PATCH 06/12] Issue 50431 - Fix regression from coverity fix
232633
232633
Description:  Fix a regression from the initial coverity commit that
232633
              caused the memebrOf groupattrs to become corrupted and
232633
              crash the server.
232633
232633
https://pagure.io/389-ds-base/issue/50431
232633
232633
Reviewed by: vashirov(Thanks!)
232633
---
232633
 ldap/servers/plugins/memberof/memberof_config.c | 5 +++--
232633
 1 file changed, 3 insertions(+), 2 deletions(-)
232633
232633
diff --git a/ldap/servers/plugins/memberof/memberof_config.c b/ldap/servers/plugins/memberof/memberof_config.c
232633
index 124217ee0..6729c3a7e 100644
232633
--- a/ldap/servers/plugins/memberof/memberof_config.c
232633
+++ b/ldap/servers/plugins/memberof/memberof_config.c
232633
@@ -550,8 +550,9 @@ memberof_apply_config(Slapi_PBlock *pb __attribute__((unused)),
232633
         }
232633
 
232633
         /* Build the new list */
232633
-        for (i = 0; theConfig.group_slapiattrs && theConfig.group_slapiattrs[i] &&
232633
-                    theConfig.groupattrs && theConfig.groupattrs[i]; i++)
232633
+        for (i = 0; theConfig.group_slapiattrs &&
232633
+                    theConfig.groupattrs &&
232633
+                    theConfig.groupattrs[i]; i++)
232633
         {
232633
             theConfig.group_slapiattrs[i] = slapi_attr_new();
232633
             slapi_attr_init(theConfig.group_slapiattrs[i], theConfig.groupattrs[i]);
232633
-- 
232633
2.21.0
232633