Repository: camel
Updated Branches:
  refs/heads/camel-2.18.x efc97083e -> 9bb08f6c3


CAMEL-11607: Fix npe in MBeanInfoAssembler when debug is enabled.

If debug is enabled and cache is not initialised an npe is thrown
when stopping the component.

Conflicts:
        
camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/9bb08f6c
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/9bb08f6c
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/9bb08f6c

Branch: refs/heads/camel-2.18.x
Commit: 9bb08f6c35f2d9ae305aa11397c95580870c992c
Parents: efc9708
Author: sarelp <sa...@yannitech.com>
Authored: Thu Jul 27 15:06:36 2017 +1000
Committer: Andrea Cosentino <anco...@gmail.com>
Committed: Thu Jul 27 15:57:17 2017 +0200

----------------------------------------------------------------------
 .../java/org/apache/camel/management/MBeanInfoAssembler.java | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/9bb08f6c/camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java
----------------------------------------------------------------------
diff --git 
a/camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java 
b/camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java
index 09cbf89..de9c9f8 100644
--- 
a/camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java
+++ 
b/camel-core/src/main/java/org/apache/camel/management/MBeanInfoAssembler.java
@@ -73,10 +73,12 @@ public class MBeanInfoAssembler implements Service {
 
     @Override
     public void stop() throws Exception {
-        if (LOG.isDebugEnabled()) {
-            LOG.debug("Clearing cache[size={}, hits={}, misses={}, 
evicted={}]", new Object[]{cache.size(), cache.getHits(), cache.getMisses(), 
cache.getEvicted()});
+        if (cache != null) {
+            if (LOG.isDebugEnabled()) {
+                LOG.debug("Clearing cache[size={}, hits={}, misses={}, 
evicted={}]", new Object[]{cache.size(), cache.getHits(), cache.getMisses(), 
cache.getEvicted()});
+            }
+            cache.clear();
         }
-        cache.clear();
     }
 
     /**

Reply via email to