[jboss-cvs] JBossAS SVN: r69145 - branches/JBPAPP_4_2_0_GA_CP/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:50:46 -0500 (Sat, 19 Jan 2008)
New Revision: 69145

Modified:
   branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java
Log:
[JBPAPP-490] Avoid NPE when event arrives before we create handler

Modified: branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java	2008-01-20 02:41:35 UTC (rev 69144)
+++ branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/ha/framework/server/DistributedReplicantManagerImpl.java	2008-01-20 02:50:46 UTC (rev 69145)
@@ -104,6 +104,8 @@
       this.mbeanserver = server;
       this.log = Logger.getLogger(DistributedReplicantManagerImpl.class.getName() + "." + partition.getPartitionName());
       this.trace = log.isTraceEnabled();
+      // JBAS-5068 Create the handler early so we don't risk NPEs
+      this.asynchHandler = new AsynchEventHandler(this, "AsynchKeyChangeHandler");
    }
 
    // Public --------------------------------------------------------
@@ -131,9 +133,7 @@
    public void start() 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!