GitHub user gaohoward opened a pull request:
https://github.com/apache/activemq-artemis/pull/1959
ARTEMIS-1754 LargeServerMessageImpl.toString() may leak files
This overridden method calls getPersistentSize() which may lead
to open the large file to get the size. Calling it alone will
cause the file being opened without closing it.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gaohoward/activemq-artemis g_1754
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/1959.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1959
----
commit 40fde75d6483b76330292957df7e77313973963a
Author: Howard Gao <howard.gao@...>
Date: 2018-03-20T11:01:16Z
ARTEMIS-1754 LargeServerMessageImpl.toString() may leak files
This overridden method calls getPersistentSize() which may lead
to open the large file to get the size. Calling it alone will
cause the file being opened without closing it.
----
---
