You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2013/11/01 20:10:10 UTC

svn commit: r1538037 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java

Author: jukka
Date: Fri Nov  1 19:10:10 2013
New Revision: 1538037

URL: http://svn.apache.org/r1538037
Log:
OAK-1141: SecureNodeBuilder.getChildNode() should not optimize for SecurityContext.canReadAll()

Drop the canReadAll() check in getChildNode()

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java?rev=1538037&r1=1538036&r2=1538037&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java Fri Nov  1 19:10:10 2013
@@ -301,12 +301,7 @@ class SecureNodeBuilder implements NodeB
 
     @Override
     public NodeBuilder getChildNode(@Nonnull String name) {
-        NodeBuilder child = builder.getChildNode(checkNotNull(name));
-        if (child.exists() && !getSecurityContext().canReadAll()) {
-            return new SecureNodeBuilder(this, name);
-        } else {
-            return child;
-        }
+        return new SecureNodeBuilder(this, checkNotNull(name));
     }
 
     @Override