activemq-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christopher L. Shannon (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (AMQ-5857) Message content stored twice while sending
Date Fri, 31 Jul 2015 18:51:05 GMT

     [ https://issues.apache.org/jira/browse/AMQ-5857?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Christopher L. Shannon resolved AMQ-5857.
-----------------------------------------
    Resolution: Fixed
    Regression:   (was: Unit Test Broken)

There was a race condition in the storeContext() method of ActiveMQTextMessage where in between
checking the text field for null and writing the content of text to the stream, the text field
was set to null.  A fix has been applied that caches the reference to the text string locally
in the method to prevent having to synchronize.  I also tweaked a couple other methods with
a similar change to prevent the same issue.

CamelVmTransportRoutingTest is now passing again.  I will double check with Jenkins tomorrow
to make sure the test failure has been cleared up.

> Message content stored twice while sending
> ------------------------------------------
>
>                 Key: AMQ-5857
>                 URL: https://issues.apache.org/jira/browse/AMQ-5857
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: JMS client
>    Affects Versions: 5.7.0, 5.8.0, 5.9.0, 5.10.0, 5.11.0
>            Reporter: Benjamin Graf
>            Assignee: Christopher L. Shannon
>             Fix For: 5.12.0
>
>
> The storeContent method in ActiveMQTextMessage does not clean text field after storing
content. Therefor the message temporary exists twice which can lead to OOM problems for large
or many text messages concurrently processing. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message