[jboss-cvs] JBossAS SVN: r58240 - trunk/deployment/src/main/org/jboss/deployment/services

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Thu Nov 9 17:35:19 EST 2006


Author: jason.greene at jboss.com
Date: 2006-11-09 17:35:18 -0500 (Thu, 09 Nov 2006)
New Revision: 58240

Modified:
   trunk/deployment/src/main/org/jboss/deployment/services/DeploymentManagerService.java
Log:
Fix JSR-88 undeployment.
This would lead to leaking classloaders, and eventual failure.


Modified: trunk/deployment/src/main/org/jboss/deployment/services/DeploymentManagerService.java
===================================================================
--- trunk/deployment/src/main/org/jboss/deployment/services/DeploymentManagerService.java	2006-11-09 22:35:02 UTC (rev 58239)
+++ trunk/deployment/src/main/org/jboss/deployment/services/DeploymentManagerService.java	2006-11-09 22:35:18 UTC (rev 58240)
@@ -269,7 +269,8 @@
       if (deployFile.exists() == false)
          throw new IOException("deployURL(" + url + ") has no local archive");
 
-      mainDeployer.removeDeploymentContext(deployURL.toString());
+      VirtualFile root = VFS.getRoot(deployFile.toURI());
+      mainDeployer.removeDeploymentContext(AbstractDeploymentContext.getDeploymentName(root));
       mainDeployer.process();
       moduleID.setRunning(false);
    }




More information about the jboss-cvs-commits mailing list