You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2018/08/29 23:13:09 UTC
directory-fortress-core git commit: FC-239 - verify the role exists
Repository: directory-fortress-core
Updated Branches:
refs/heads/master fc1f89dc6 -> 0beb2db48
FC-239 - verify the role exists
Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/0beb2db4
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/0beb2db4
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/0beb2db4
Branch: refs/heads/master
Commit: 0beb2db48e56c78d5e38062ab6b84f7c3239bcf3
Parents: fc1f89d
Author: Shawn McKinney <sm...@apache.org>
Authored: Tue Aug 28 22:39:29 2018 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Tue Aug 28 22:39:29 2018 -0500
----------------------------------------------------------------------
.../java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0beb2db4/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java b/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
index 89a0a8a..d3ded33 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
@@ -405,6 +405,8 @@ public final class AdminMgrImpl extends Manageable implements AdminMgr, Serializ
String propKey = GlobalIds.CONSTRAINT_KEY_PREFIX + role.getName().toLowerCase();
String propValue = roleConstraint.getKey();
VUtil.assertNotNull( propValue, GlobalErrIds.ROLE_CONSTRAINT_KEY_NULL, CLS_NM + methodName );
+ // Verify the role exists:
+ roleP.read( role );
Properties props = new Properties();
props.setProperty( propKey, propValue );
// Retrieve parameters from the config node stored in target LDAP DIT: