Return-Path: X-Original-To: apmail-jackrabbit-dev-archive@www.apache.org Delivered-To: apmail-jackrabbit-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 662B3D95D for ; Thu, 16 May 2013 17:33:17 +0000 (UTC) Received: (qmail 79055 invoked by uid 500); 16 May 2013 17:33:17 -0000 Delivered-To: apmail-jackrabbit-dev-archive@jackrabbit.apache.org Received: (qmail 78920 invoked by uid 500); 16 May 2013 17:33:17 -0000 Mailing-List: contact dev-help@jackrabbit.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@jackrabbit.apache.org Delivered-To: mailing list dev@jackrabbit.apache.org Received: (qmail 78912 invoked by uid 99); 16 May 2013 17:33:17 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 16 May 2013 17:33:17 +0000 Date: Thu, 16 May 2013 17:33:17 +0000 (UTC) From: "Francesco Mari (JIRA)" To: dev@jackrabbit.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (JCR-3382) ItemManager.getNode does not do a permission check when the item data is in the item manager cache MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/JCR-3382?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Francesco Mari updated JCR-3382: -------------------------------- Attachment: performance.patch Added performance tests. > ItemManager.getNode does not do a permission check when the item data is in the item manager cache > -------------------------------------------------------------------------------------------------- > > Key: JCR-3382 > URL: https://issues.apache.org/jira/browse/JCR-3382 > Project: Jackrabbit Content Repository > Issue Type: Bug > Affects Versions: 2.6 > Reporter: Unico Hommes > Assignee: Unico Hommes > Fix For: 2.6.1, 2.7 > > Attachments: JCR-3382.patch, performance.patch > > > Read access should be checked irrespective of whether the item data is in the cache or not. Something might have changed between first reading the node and reading the node again that impacts read access. > We are running into the situation where node.hasNode() returns false for a node that is in the item manager cache and for which access was revoked but node.getNode() returns the node anyway. So node.hasNode is implemented in the right way, but node.getNode is not. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira