Repository: camel Updated Branches: refs/heads/camel-2.19.x 739961c8d -> 3e4c7820e
CAMEL-11386 Potential NullPointerException if H... ...TTP client not started and stop was performed This adds `null` check of `inflightRequests` to the `stop` method. Project: http://git-wip-us.apache.org/repos/asf/camel/repo Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/3e4c7820 Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/3e4c7820 Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/3e4c7820 Branch: refs/heads/camel-2.19.x Commit: 3e4c7820eadc54f40bea0cd91be3acd4cfebae47 Parents: 739961c Author: Zoran Regvart <zregv...@apache.org> Authored: Thu Jun 8 15:02:54 2017 +0200 Committer: Zoran Regvart <zregv...@apache.org> Committed: Thu Jun 8 15:03:40 2017 +0200 ---------------------------------------------------------------------- .../internal/client/AbstractClientBase.java | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/camel/blob/3e4c7820/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/AbstractClientBase.java ---------------------------------------------------------------------- diff --git a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/AbstractClientBase.java b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/AbstractClientBase.java index 26a86d9..e996de5 100644 --- a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/AbstractClientBase.java +++ b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/AbstractClientBase.java @@ -103,12 +103,14 @@ public abstract class AbstractClientBase implements SalesforceSession.Salesforce @Override public void stop() throws Exception { - inflightRequests.arrive(); - if (!inflightRequests.isTerminated()) { - try { - inflightRequests.awaitAdvanceInterruptibly(0, terminationTimeout, TimeUnit.SECONDS); - } catch (InterruptedException | TimeoutException ignored) { - // exception is ignored + if (inflightRequests != null) { + inflightRequests.arrive(); + if (!inflightRequests.isTerminated()) { + try { + inflightRequests.awaitAdvanceInterruptibly(0, terminationTimeout, TimeUnit.SECONDS); + } catch (InterruptedException | TimeoutException ignored) { + // exception is ignored + } } }
