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(); } /**