[jboss-cvs] JBossAS SVN: r61054 - trunk/connector/src/main/org/jboss/resource/connectionmanager.

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

Author: weston.price at jboss.com
Date: 2007-03-02 11:32:27 -0500 (Fri, 02 Mar 2007)
New Revision: 61054

Modified:
   trunk/connector/src/main/org/jboss/resource/connectionmanager/InternalManagedConnectionPool.java
Log:
[JBAS-4170]Fix incorrect resource usage for background validation.

Modified: trunk/connector/src/main/org/jboss/resource/connectionmanager/InternalManagedConnectionPool.java
===================================================================
--- trunk/connector/src/main/org/jboss/resource/connectionmanager/InternalManagedConnectionPool.java	2007-03-02 15:33:30 UTC (rev 61053)
+++ trunk/connector/src/main/org/jboss/resource/connectionmanager/InternalManagedConnectionPool.java	2007-03-02 16:32:27 UTC (rev 61054)
@@ -692,7 +692,6 @@
 
                         if (cl.getState() != ConnectionListener.DESTROY)
                         {
-                           log.warn("");
                            doDestroy(cl);
                            destroyed = true;
 
@@ -708,11 +707,13 @@
                }
                finally
                {
-                  synchronized (cls)
+                  if(!destroyed)
                   {
-
-                     returnForFrequencyCheck(cl);
-
+                     synchronized (cls)
+                     {
+                        returnForFrequencyCheck(cl);
+                     }
+                     
                   }
 
                }