Author: norman
Date: Thu Apr  7 07:48:38 2011
New Revision: 1089763

URL: http://svn.apache.org/viewvc?rev=1089763&view=rev
Log:
Copy the TreeSet to fix possible java.util.ConcurrentModificationException. See 
IMAP-278

Modified:
    
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/base/MailboxEventAnalyser.java

Modified: 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/base/MailboxEventAnalyser.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/base/MailboxEventAnalyser.java?rev=1089763&r1=1089762&r2=1089763&view=diff
==============================================================================
--- 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/base/MailboxEventAnalyser.java
 (original)
+++ 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/base/MailboxEventAnalyser.java
 Thu Apr  7 07:48:38 2011
@@ -194,7 +194,9 @@ public class MailboxEventAnalyser extend
     
     public Collection<Long> flagUpdateUids() {
         synchronized (flagUpdateUids) {
-            return Collections.unmodifiableSet(flagUpdateUids);
+            // copy the TreeSet to fix possible 
java.util.ConcurrentModificationException
+            // See IMAP-278
+            return Collections.unmodifiableSet(new 
TreeSet<Long>(flagUpdateUids));
         }
     }
 
@@ -205,7 +207,9 @@ public class MailboxEventAnalyser extend
      */
     public Collection<Long> expungedUids() {
         synchronized (expungedUids) {
-            return Collections.unmodifiableSet(expungedUids);
+            // copy the TreeSet to fix possible 
java.util.ConcurrentModificationException
+            // See IMAP-278
+            return Collections.unmodifiableSet(new 
TreeSet<Long>(expungedUids));
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to