You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2011/09/30 22:50:41 UTC

svn commit: r1177826 - /james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java

Author: norman
Date: Fri Sep 30 20:50:41 2011
New Revision: 1177826

URL: http://svn.apache.org/viewvc?rev=1177826&view=rev
Log:
Fix possible NPE in some cases.

Modified:
    james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java

Modified: james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java
URL: http://svn.apache.org/viewvc/james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java?rev=1177826&r1=1177825&r2=1177826&view=diff
==============================================================================
--- james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java (original)
+++ james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store/AbstractDelegatingMailboxListener.java Fri Sep 30 20:50:41 2011
@@ -163,8 +163,10 @@ public abstract class AbstractDelegating
     public void removeGlobalListener(MailboxListener listener, MailboxSession session) throws MailboxException {
         List<MailboxListener> gListeners = getGlobalListeners();
 
-        synchronized (gListeners) {
-            gListeners.remove(listener);
+        if (gListeners != null) {
+            synchronized (gListeners) {
+                gListeners.remove(listener);
+            }
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org