[jboss-cvs] JBossAS SVN: r73336 - branches/JBPAPP_4_2_0_GA_CP/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 04:17:53 -0400 (Tue, 13 May 2008)
New Revision: 73336

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

Modified: branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-05-13 08:05:40 UTC (rev 73335)
+++ branches/JBPAPP_4_2_0_GA_CP/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-05-13 08:17:53 UTC (rev 73336)
@@ -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;