[jboss-cvs] JBossAS SVN: r71748 - branches/Branch_4_2/cluster/src/main/org/jboss/invocation/unified/interfaces.

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

Author: galder.zamarreno at jboss.com
Date: 2008-04-06 11:15:58 -0400 (Sun, 06 Apr 2008)
New Revision: 71748

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

Modified: branches/Branch_4_2/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java
===================================================================
--- branches/Branch_4_2/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-04-05 11:31:38 UTC (rev 71747)
+++ branches/Branch_4_2/cluster/src/main/org/jboss/invocation/unified/interfaces/UnifiedInvokerHAProxy.java	2008-04-06 15:15:58 UTC (rev 71748)
@@ -169,7 +169,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;