You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2013/10/31 19:56:55 UTC
[3/4] git commit: CAMEL-6921: Fixed netty producer to not cause NPE
later in messageReceived when destoying channel from pool
CAMEL-6921: Fixed netty producer to not cause NPE later in messageReceived when destoying channel from pool
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/f50a6182
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/f50a6182
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/f50a6182
Branch: refs/heads/camel-2.11.x
Commit: f50a6182906e337ef72c7fd19afd826339c898b2
Parents: 73a66bc
Author: Claus Ibsen <da...@apache.org>
Authored: Thu Oct 31 19:55:58 2013 +0100
Committer: Claus Ibsen <da...@apache.org>
Committed: Thu Oct 31 19:57:16 2013 +0100
----------------------------------------------------------------------
.../main/java/org/apache/camel/component/netty/NettyProducer.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/f50a6182/components/camel-netty/src/main/java/org/apache/camel/component/netty/NettyProducer.java
----------------------------------------------------------------------
diff --git a/components/camel-netty/src/main/java/org/apache/camel/component/netty/NettyProducer.java b/components/camel-netty/src/main/java/org/apache/camel/component/netty/NettyProducer.java
index d3012e3..a6c5a7e 100644
--- a/components/camel-netty/src/main/java/org/apache/camel/component/netty/NettyProducer.java
+++ b/components/camel-netty/src/main/java/org/apache/camel/component/netty/NettyProducer.java
@@ -479,8 +479,7 @@ public class NettyProducer extends DefaultAsyncProducer {
@Override
public void destroyObject(Channel channel) throws Exception {
LOG.trace("Destroying channel: {}", channel);
- NettyHelper.close(channel);
- ALL_CHANNELS.remove(channel);
+ // noop
}
@Override