Repository: activemq-artemis
Updated Branches:
  refs/heads/master 8b8eff053 -> d3b11bc43


ARTEMIS-1754 LargeServerMessageImpl.toString() may leak files


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/38ba5d79
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/38ba5d79
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/38ba5d79

Branch: refs/heads/master
Commit: 38ba5d797a18d99447f42f7790446c0009dc44d7
Parents: 8b8eff0
Author: Howard Gao <[email protected]>
Authored: Tue Mar 20 21:54:41 2018 +0800
Committer: Clebert Suconic <[email protected]>
Committed: Tue Mar 20 15:40:44 2018 -0400

----------------------------------------------------------------------
 .../core/persistence/impl/journal/LargeServerMessageImpl.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/38ba5d79/artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/LargeServerMessageImpl.java
----------------------------------------------------------------------
diff --git 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/LargeServerMessageImpl.java
 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/LargeServerMessageImpl.java
index 9a2e285..297328d 100644
--- 
a/artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/LargeServerMessageImpl.java
+++ 
b/artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/LargeServerMessageImpl.java
@@ -355,7 +355,7 @@ public final class LargeServerMessageImpl extends 
CoreMessage implements LargeSe
    @Override
    public String toString() {
       try {
-         return "LargeServerMessage[messageID=" + messageID + ",durable=" + 
isDurable() + ",userID=" + getUserID() + ",priority=" + this.getPriority() + ", 
timestamp=" + toDate(getTimestamp()) + ",expiration=" + toDate(getExpiration()) 
+ ", durable=" + durable + ", address=" + getAddress() + ",size=" + 
getPersistentSize() + ",properties=" + (properties != null ? 
properties.toString() : "") + "]@" + System.identityHashCode(this);
+         return "LargeServerMessage[messageID=" + messageID + ",durable=" + 
isDurable() + ",userID=" + getUserID() + ",priority=" + this.getPriority() + ", 
timestamp=" + toDate(getTimestamp()) + ",expiration=" + toDate(getExpiration()) 
+ ", durable=" + durable + ", address=" + getAddress() + ", properties=" + 
(properties != null ? properties.toString() : "") + "]@" + 
System.identityHashCode(this);
       } catch (Exception e) {
          e.printStackTrace();
          return "LargeServerMessage[messageID=" + messageID + "]";

Reply via email to