[jboss-cvs] JBossAS SVN: r73331 - branches/JBPAPP_4_3_0_GA_CP01_JBPAPP-825/cluster/src/main/org/jboss/invocation/unified/interfaces.

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

Author: galder.zamarreno at jboss.com
Date: 2008-05-13 03:03:16 -0400 (Tue, 13 May 2008)
New Revision: 73331

Modified:
   branches/JBPAPP_4_3_0_GA_CP01_JBPAPP-825/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java
Log:
[JBPAPP-825] Fixed unified invoker proxy NPE and thread safety issue by synchronising getClient method.

Modified: branches/JBPAPP_4_3_0_GA_CP01_JBPAPP-825/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java
===================================================================
--- branches/JBPAPP_4_3_0_GA_CP01_JBPAPP-825/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-05-13 07:01:46 UTC (rev 73330)
+++ branches/JBPAPP_4_3_0_GA_CP01_JBPAPP-825/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-05-13 07:03:16 UTC (rev 73331)
@@ -166,7 +166,7 @@
     * @return
     * @throws MalformedURLException
     */
-   protected Client getClient(Invocation invocationBasedRouting) throws MalformedURLException
+   protected synchronized Client getClient(Invocation invocationBasedRouting) throws MalformedURLException
    {
       Object target = loadBalancePolicy.chooseTarget(familyClusterInfo, invocationBasedRouting);
       InvokerLocator targetLocator = (InvokerLocator) target;