commit 70b86908ea8f246e5d7b38664aefed7808940a0c
author emmanuel lecharny <elecharny@apache.org> Mon Apr 01 00:42:47 2024 +0200
committer emmanuel lecharny <elecharny@apache.org> Mon Apr 01 00:42:47 2024 +0200
tree 929c5b1a61d10a20f87a68525a9d74930d647351
parent 004437a5a35c36cc128a3a2e4e5b392a8df83102

Fiwed a wrong String comparaison using '=' (DIRSERVER-2398)

interceptors/authz/src/main/java/org/apache/directory/server/core/authz/GroupCache.java

diff --git a/interceptors/authz/src/main/java/org/apache/directory/server/core/authz/GroupCache.java b/interceptors/authz/src/main/java/org/apache/directory/server/core/authz/GroupCache.java
index cdf9a01..46298da 100644
--- a/interceptors/authz/src/main/java/org/apache/directory/server/core/authz/GroupCache.java
+++ b/interceptors/authz/src/main/java/org/apache/directory/server/core/authz/GroupCache.java
@@ -43,6 +43,7 @@
 import org.apache.directory.api.ldap.model.name.Dn;
 import org.apache.directory.api.ldap.model.schema.AttributeType;
 import org.apache.directory.api.ldap.model.schema.SchemaManager;
+import org.apache.directory.api.util.Strings;
 import org.apache.directory.server.constants.ServerDNConstants;
 import org.apache.directory.server.core.api.CoreSession;
 import org.apache.directory.server.core.api.DirectoryService;
@@ -417,7 +418,7 @@
 
         for ( Modification modification : mods )
         {
-            if ( memberAttr.getOid() == modification.getAttribute().getId() )
+            if ( Strings.equals( memberAttr.getOid(), modification.getAttribute().getId() ) )
             {
                 Set<String> memberSet = groups.get( name.getNormName() );