From a510c8d1ecb2b5ac02bd64d202b4fbe2a5fbdac2 Mon Sep 17 00:00:00 2001 From: Robert Godfrey Date: Fri, 15 Aug 2014 10:14:24 +0000 Subject: QPID-6002 : [Java Broker] NullPointerException when writing message discard logging message git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1618139 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/qpid/server/protocol/v0_8/AMQChannel.java | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/qpid/java/broker-plugins/amqp-0-8-protocol/src/main/java/org/apache/qpid/server/protocol/v0_8/AMQChannel.java b/qpid/java/broker-plugins/amqp-0-8-protocol/src/main/java/org/apache/qpid/server/protocol/v0_8/AMQChannel.java index 7877812d84..49bc26149e 100644 --- a/qpid/java/broker-plugins/amqp-0-8-protocol/src/main/java/org/apache/qpid/server/protocol/v0_8/AMQChannel.java +++ b/qpid/java/broker-plugins/amqp-0-8-protocol/src/main/java/org/apache/qpid/server/protocol/v0_8/AMQChannel.java @@ -474,13 +474,12 @@ public class AMQChannel> } else { - getVirtualHost().getEventLogger().message(ExchangeMessages.DISCARDMSG(_currentMessage.getExchangeName().asString(), - _currentMessage.getMessagePublishInfo().getRoutingKey() - == null - ? null - : _currentMessage.getMessagePublishInfo() - .getRoutingKey() - .toString())); + AMQShortString exchangeName = _currentMessage.getExchangeName(); + AMQShortString routingKey = _currentMessage.getMessagePublishInfo().getRoutingKey(); + + getVirtualHost().getEventLogger().message( + ExchangeMessages.DISCARDMSG(exchangeName == null ? null : exchangeName.asString(), + routingKey == null ? null : routingKey.asString())); } } -- cgit v1.2.1