You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2006/12/13 14:31:23 UTC

svn commit: r486645 - /incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java

Author: gnodet
Date: Wed Dec 13 05:31:22 2006
New Revision: 486645

URL: http://svn.apache.org/viewvc?view=rev&rev=486645
Log:
Avoid NPE when component shuts down and the stats mbean is not registered

Modified:
    incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java

Modified: incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java?view=diff&rev=486645&r1=486644&r2=486645
==============================================================================
--- incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java (original)
+++ incubator/servicemix/trunk/core/servicemix-core/src/main/java/org/apache/servicemix/jbi/monitoring/StatisticsService.java Wed Dec 13 05:31:22 2006
@@ -215,12 +215,15 @@
         ComponentMBeanImpl component = event.getComponent();
         String key = component.getName();
         ComponentStats stats = (ComponentStats) componentStats.remove(key);
+        if (stats == null) {
+            return;
+        }
         // Register MBean
         ManagementContext context= container.getManagementContext();
         try {
             context.unregisterMBean(context.createObjectName(context.createObjectNameProps(stats, true)));
         } catch (Exception e) {
-            log.info("Unable to unregister component statistics MBean: " + e.getMessage());
+            log.info("Unable to unregister component statistics MBean: " + e);
             if (log.isDebugEnabled()) {
                 log.debug("Unable to unregister component statistics MBean", e);
             }