Author: fmeschbe
Date: Wed Feb 10 10:36:51 2010
New Revision: 908440

URL: http://svn.apache.org/viewvc?rev=908440&view=rev
Log:
SLING-1363 Must start the session configurer tracker before starting the 
repository because starting the repository tries to login which in turn 
accesses the tracker thus causing an NPE.

Modified:
    
sling/trunk/bundles/jcr/base/src/main/java/org/apache/sling/jcr/base/AbstractSlingRepository.java

Modified: 
sling/trunk/bundles/jcr/base/src/main/java/org/apache/sling/jcr/base/AbstractSlingRepository.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/base/src/main/java/org/apache/sling/jcr/base/AbstractSlingRepository.java?rev=908440&r1=908439&r2=908440&view=diff
==============================================================================
--- 
sling/trunk/bundles/jcr/base/src/main/java/org/apache/sling/jcr/base/AbstractSlingRepository.java
 (original)
+++ 
sling/trunk/bundles/jcr/base/src/main/java/org/apache/sling/jcr/base/AbstractSlingRepository.java
 Wed Feb 10 10:36:51 2010
@@ -572,6 +572,9 @@
 
         componentContext.getBundleContext().addBundleListener(this);
 
+        this.sessionConfigurerTracker = new 
ServiceTracker(componentContext.getBundleContext(), 
SessionConfigurer.SERVICE_NAME, null);
+        this.sessionConfigurerTracker.open();
+
         // immediately try to start the repository while activating
         // this component instance
         try {
@@ -586,9 +589,6 @@
                 "activate: Unexpected problem starting repository", t);
         }
 
-        this.sessionConfigurerTracker = new 
ServiceTracker(componentContext.getBundleContext(), 
SessionConfigurer.SERVICE_NAME, null);
-        this.sessionConfigurerTracker.open();
-
         // launch the background repository checker now
         startRepositoryPinger();
     }


Reply via email to