You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by st...@apache.org on 2005/04/11 13:23:53 UTC

svn commit: r160858 - incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java

Author: stefan
Date: Mon Apr 11 04:23:51 2005
New Revision: 160858

URL: http://svn.apache.org/viewcvs?view=rev&rev=160858
Log:
calling Node.getName() on disposed instance could throw NPE

Modified:
    incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java

Modified: incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java
URL: http://svn.apache.org/viewcvs/incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java?view=diff&r1=160857&r2=160858
==============================================================================
--- incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java (original)
+++ incubator/jackrabbit/trunk/src/java/org/apache/jackrabbit/core/NodeImpl.java Mon Apr 11 04:23:51 2005
@@ -1617,12 +1617,14 @@
      * {@inheritDoc}
      */
     public String getName() throws RepositoryException {
+        // check state of this instance
+        sanityCheck();
+
         if (state.getParentUUID() == null) {
             // this is the root node
             return "";
         }
 
-        //QName name = getPrimaryPath().getNameElement().getName();
         QName name = session.getHierarchyManager().getName(id);
         try {
             return name.toJCRName(session.getNamespaceResolver());