From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Thu, 2 Dec 2010 16:51:08 +0000 (+0000)
Subject: Correct removal of LifeCycleListenters from Containers via JMX.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=4f0fdc3e96fa98e6858784be1bbf428863720a02;p=tomcat7.0

Correct removal of LifeCycleListenters from Containers via JMX.
Found with FindBugs

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1041448 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/mbeans/ContainerMBean.java b/java/org/apache/catalina/mbeans/ContainerMBean.java
index 110d51449..73a880fc9 100644
--- a/java/org/apache/catalina/mbeans/ContainerMBean.java
+++ b/java/org/apache/catalina/mbeans/ContainerMBean.java
@@ -263,7 +263,7 @@ public class ContainerMBean extends BaseModelMBean {
         
         LifecycleListener[] listeners = container.findLifecycleListeners();
         for(LifecycleListener listener: listeners){
-            if(listener.getClass().equals(type)){
+            if(listener.getClass().getName().equals(type)){
                 container.removeLifecycleListener(listener);
             }
         }
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 78fdc065c..3e25305da 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -83,6 +83,10 @@
       <fix>
         Correct handling of versioned web applications in deployer. (markt)
       </fix>
+      <fix>
+        Correct removal of <code>LifeCycleListenter</code>s from
+        <code>Container</code>s via JMX. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Coyote">