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/12/21 09:02:02 UTC
svn commit: r1221639 -
/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java
Author: norman
Date: Wed Dec 21 08:02:02 2011
New Revision: 1221639
URL: http://svn.apache.org/viewvc?rev=1221639&view=rev
Log:
Fix NPE which was thrown when no ResponseListener is registered and setResponse(..) is called. See PROTOCOLS-61
Modified:
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java
Modified: james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java
URL: http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java?rev=1221639&r1=1221638&r2=1221639&view=diff
==============================================================================
--- james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java (original)
+++ james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureResponseImpl.java Wed Dec 21 08:02:02 2011
@@ -125,7 +125,7 @@ public class FutureResponseImpl implemen
synchronized (this) {
if (!isReady()) {
this.response = response;
- fire = true;
+ fire = listeners != null;
if (waiters > 0) {
notifyAll();
@@ -142,6 +142,7 @@ public class FutureResponseImpl implemen
}
}
listeners = null;
+
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org