[jboss-cvs] JBossAS SVN: r61287 - trunk/server/src/main/org/jboss/ejb.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Mon Mar 12 17:02:29 EDT 2007


Author: bstansberry at jboss.com
Date: 2007-03-12 17:02:29 -0400 (Mon, 12 Mar 2007)
New Revision: 61287

Modified:
   trunk/server/src/main/org/jboss/ejb/Container.java
Log:
Don't leak classloader via InvocationStatistics

Modified: trunk/server/src/main/org/jboss/ejb/Container.java
===================================================================
--- trunk/server/src/main/org/jboss/ejb/Container.java	2007-03-12 17:05:27 UTC (rev 61286)
+++ trunk/server/src/main/org/jboss/ejb/Container.java	2007-03-12 21:02:29 UTC (rev 61287)
@@ -881,7 +881,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();
    }
 




More information about the jboss-cvs-commits mailing list