[jboss-cvs] JBossAS SVN: r69144 - trunk/cluster/src/main/org/jboss/ha/framework/server.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2008-01-19 21:41:35 -0500 (Sat, 19 Jan 2008)
New Revision: 69144

Modified:
   trunk/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java
Log:
[JBAS-5068] Avoid NPE when event arrives before we create handler

Modified: trunk/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java
===================================================================
--- trunk/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java	2008-01-20 02:37:29 UTC (rev 69143)
+++ trunk/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java	2008-01-20 02:41:35 UTC (rev 69144)
@@ -92,6 +92,8 @@
    {
       super();
       this.log = Logger.getLogger (this.getClass ());
+      // JBAS-5068 Create the handler early so we don't risk NPEs
+      this.asynchHandler = new AsynchEventHandler(this, "AsynchKeyChangeHandler");
    }
 
    // Public --------------------------------------------------------
@@ -112,9 +114,7 @@
    public void startService() throws Exception
    {
       this.nodeName = this.partition.getNodeName();
-   
-      // Create the asynch listener handler thread
-      asynchHandler = new AsynchEventHandler(this, "AsynchKeyChangeHandler");
+      
       asynchHandler.start();
 
       partitionNameKnown.release (); // partition name is now known!