You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2010/11/18 14:26:33 UTC

svn commit: r1036448 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java

Author: gtully
Date: Thu Nov 18 13:26:33 2010
New Revision: 1036448

URL: http://svn.apache.org/viewvc?rev=1036448&view=rev
Log:
resolve potential concurrent mod of LinkedHashMap, reads can modify as map is access ordered

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java?rev=1036448&r1=1036447&r2=1036448&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/ConnectionAudit.java Thu Nov 18 13:26:33 2010
@@ -64,7 +64,7 @@ class ConnectionAudit {
         return false;
     }
 
-    protected void rollbackDuplicate(ActiveMQDispatcher dispatcher, Message message) {
+    protected synchronized void rollbackDuplicate(ActiveMQDispatcher dispatcher, Message message) {
         if (checkForDuplicates && message != null) {
             ActiveMQDestination destination = message.getDestination();
             if (destination != null) {