[jboss-cvs] JBossAS SVN: r112448 - in branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc: local and 1 other directories.

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

Author: jbertram at redhat.com
Date: 2011-11-11 14:51:39 -0500 (Fri, 11 Nov 2011)
New Revision: 112448

Modified:
   branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/BaseWrapperManagedConnectionFactory.java
   branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java
   branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnectionFactory.java
Log:
JBPAPP-7468

Modified: branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/BaseWrapperManagedConnectionFactory.java
===================================================================
--- branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/BaseWrapperManagedConnectionFactory.java	2011-11-11 15:19:18 UTC (rev 112447)
+++ branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/BaseWrapperManagedConnectionFactory.java	2011-11-11 19:51:39 UTC (rev 112448)
@@ -491,7 +491,7 @@
     * <p>In fact, we have a problem here. Theoretically, there is a possible
     * name collision between config properties and "user"/"password".
     */
-   protected Properties getConnectionProperties(Subject subject, ConnectionRequestInfo cri)
+   protected synchronized Properties getConnectionProperties(Subject subject, ConnectionRequestInfo cri)
       throws ResourceException
    {
       if (cri != null && cri.getClass() != WrappedConnectionRequestInfo.class)

Modified: branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java
===================================================================
--- branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java	2011-11-11 15:19:18 UTC (rev 112447)
+++ branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java	2011-11-11 19:51:39 UTC (rev 112448)
@@ -165,7 +165,7 @@
       }
    }
 
-   public ManagedConnection createManagedConnection(Subject subject, ConnectionRequestInfo cri)
+   public synchronized ManagedConnection createManagedConnection(Subject subject, ConnectionRequestInfo cri)
          throws javax.resource.ResourceException
    {
       Properties props = getConnectionProperties(subject, cri);

Modified: branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnectionFactory.java
===================================================================
--- branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnectionFactory.java	2011-11-11 15:19:18 UTC (rev 112447)
+++ branches/JBPAPP_5_1_0_Final_JBPAPP-7468/connector/src/main/org/jboss/resource/adapter/jdbc/xa/XAManagedConnectionFactory.java	2011-11-11 19:51:39 UTC (rev 112448)
@@ -401,7 +401,7 @@
       this.isSameRMOverrideValue = isSameRMOverrideValue;
    }
 
-   public ManagedConnection createManagedConnection(Subject subject, ConnectionRequestInfo cri)
+   public synchronized ManagedConnection createManagedConnection(Subject subject, ConnectionRequestInfo cri)
    		throws javax.resource.ResourceException
    {	   
 	   if(xadsSelector == null)