jackrabbit-oak-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ju...@apache.org
Subject svn commit: r1538037 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/SecureNodeBuilder.java
Date Fri, 01 Nov 2013 19:10:10 GMT
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



Mime
View raw message