Author: angela
Date: Wed Jun 5 12:42:00 2019
New Revision: 1860656
URL: http://svn.apache.org/viewvc?rev=1860656&view=rev
Log:
OAK-8386 : AccessControlValidator.checkValidAccessControlEntry: redundant test
for tree existence
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/accesscontrol/AccessControlValidator.java
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/accesscontrol/AccessControlValidator.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/accesscontrol/AccessControlValidator.java?rev=1860656&r1=1860655&r2=1860656&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/accesscontrol/AccessControlValidator.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/accesscontrol/AccessControlValidator.java
Wed Jun 5 12:42:00 2019
@@ -228,7 +228,7 @@ class AccessControlValidator extends Def
private void checkValidAccessControlEntry(@NotNull Tree aceNode) throws
CommitFailedException {
Tree parent = aceNode.getParent();
- if (!parent.exists() ||
!NT_REP_ACL.equals(TreeUtil.getPrimaryTypeName(parent))) {
+ if (!NT_REP_ACL.equals(TreeUtil.getPrimaryTypeName(parent))) {
throw accessViolation(7, "Isolated access control entry at " +
aceNode.getPath());
}
checkValidPrincipal(aceNode);