You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by al...@apache.org on 2012/06/06 14:58:28 UTC

svn commit: r1346866 - /jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java

Author: alexparvulescu
Date: Wed Jun  6 12:58:27 2012
New Revision: 1346866

URL: http://svn.apache.org/viewvc?rev=1346866&view=rev
Log:
JCR-3050 NullPointerException on removing a node acquired from search result(2.4 backport)

Modified:
    jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java

Modified: jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java
URL: http://svn.apache.org/viewvc/jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java?rev=1346866&r1=1346865&r2=1346866&view=diff
==============================================================================
--- jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java (original)
+++ jackrabbit/branches/2.4/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/state/ItemState.java Wed Jun  6 12:58:27 2012
@@ -207,7 +207,12 @@ public abstract class ItemState {
      * @throws RepositoryException
      */
     public NodeState getParent() throws ItemNotFoundException, RepositoryException {
-        return getHierarchyEntry().getParent().getNodeState();
+        // safeguard against root node's null parent
+        NodeEntry parent = getHierarchyEntry().getParent();
+        if (parent != null) {
+            return getHierarchyEntry().getParent().getNodeState();
+        }
+        return null;
     }
 
     /**