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);


Reply via email to