jackrabbit-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From un...@apache.org
Subject svn commit: r1350207 - /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java
Date Thu, 14 Jun 2012 12:13:35 GMT
Author: unico
Date: Thu Jun 14 12:13:34 2012
New Revision: 1350207

URL: http://svn.apache.org/viewvc?rev=1350207&view=rev
Log:
JCR-3265 guard against possible npe's due to double checking: if the bundle is no longer there
we don't need to continue to check it

Modified:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java

Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java?rev=1350207&r1=1350206&r2=1350207&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java
(original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/ConsistencyCheckerImpl.java
Thu Jun 14 12:13:34 2012
@@ -309,6 +309,8 @@ public class ConsistencyCheckerImpl {
                             log.error(message);
                             missingChildren.add(entry);
                         }
+                    } else {
+                        return;
                     }
                 } else {
                     NodeId cp = childBundle.getParentId();
@@ -333,6 +335,8 @@ public class ConsistencyCheckerImpl {
                                     log.error(message);
                                 }
                             }
+                        } else {
+                            return;
                         }
                     }
                 }
@@ -376,6 +380,8 @@ public class ConsistencyCheckerImpl {
                                 modifications.add(bundle);
                             }
                         }
+                    } else {
+                        return;
                     }
                 } else {
                     boolean found = false;
@@ -410,6 +416,8 @@ public class ConsistencyCheckerImpl {
                                         + "'");
                                 modifications.add(parentBundle);
                             }
+                        } else {
+                            return;
                         }
                     }
 



Mime
View raw message