mserver != null will never be true if the mbean was registered. Drop the check since...

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

diff --git a/java/org/apache/coyote/AbstractProtocolHandler.java b/java/org/apache/coyote/AbstractProtocolHandler.java
index a851e38..14bed60 100644 (file)
--- a/java/org/apache/coyote/AbstractProtocolHandler.java
+++ b/java/org/apache/coyote/AbstractProtocolHandler.java
@@ -418,10 +418,8 @@ public abstract class AbstractProtocolHandler implements ProtocolHandler,
                     getName()), e);
         }
         
-        // If object was pre-registered (mserver != null) what ever registered
-        // the ProtocolHandler should de-register it
-        if (oname != null && mserver == null) {
-                Registry.getRegistry(null, null).unregisterComponent(oname);
+        if (oname != null) {
+            Registry.getRegistry(null, null).unregisterComponent(oname);
         }
 
         if (tpOname != null)