[jboss-cvs] JBossAS SVN: r61254 - branches/Branch_4_2/server/src/main/org/jboss/ejb.

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

Author: bstansberry at jboss.com
Date: 2007-03-10 19:15:56 -0500 (Sat, 10 Mar 2007)
New Revision: 61254

Modified:
   branches/Branch_4_2/server/src/main/org/jboss/ejb/Container.java
Log:
Destroyed Container doesn't hold ref to application classes via InvocationStatistics

Modified: branches/Branch_4_2/server/src/main/org/jboss/ejb/Container.java
===================================================================
--- branches/Branch_4_2/server/src/main/org/jboss/ejb/Container.java	2007-03-10 23:54:43 UTC (rev 61253)
+++ branches/Branch_4_2/server/src/main/org/jboss/ejb/Container.java	2007-03-11 00:15:56 UTC (rev 61254)
@@ -859,7 +859,12 @@
       localHomeInterface = null;
       localInterface = null;
       methodPermissionsCache.clear();
-      invokeStats.resetStats();
+      // InvocationStatistics holds refs to Methods from
+      // application classes, so to avoid a classloader
+      // leak, lets not just resetStats() but also replace
+      // the object
+      invokeStats.resetStats(); // in case someone else has a ref
+      invokeStats = new InvocationStatistics();
       marshalledInvocationMapping.clear();
    }