You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-dev@jackrabbit.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2012/07/24 12:32:34 UTC

[jira] [Commented] (OAK-206) RootImpl#getTree relies on accessibility of all intermediate trees

    [ https://issues.apache.org/jira/browse/OAK-206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13421314#comment-13421314 ] 

Michael Dürig commented on OAK-206:
-----------------------------------

bq. for the time being we should fix the current implementation to use an implementation specific access method on TreeImpl.

works for me
                
> RootImpl#getTree relies on accessibility of all intermediate trees
> ------------------------------------------------------------------
>
>                 Key: OAK-206
>                 URL: https://issues.apache.org/jira/browse/OAK-206
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: core
>            Reporter: angela
>
> as explained in OAK-205 the currently implementation of Root#getTree 
> relies on the assumption that all intermediate trees are accessible:
>         TreeImpl child = rootTree;
>         for (String name : elements(path)) {
>             child = child.getChild(name);
>             if (child == null) {
>                 return null;
>             }
>         }
>         return child;
> this might not be the case once we have permissions enforced in oak-core.
> we may either fix that on the API level (see OAK-205) which apart from
> making the oak-api permission sensitive would be quite convenient to use
> in oak-jcr... 
> for the time being we should fix the current implementation to use an
> implementation specific access method on TreeImpl.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira