Fixed bug #291340: Race condition in onException() notifications.

git-svn-id: svn+ssh://dev.eclipse.org/svnroot/rt/org.eclipse.jetty/jetty/trunk@966 7e9141cc-0065-0410-87d8-b60c137991c4
This commit is contained in:
Simone Bordet 2009-10-05 13:16:24 +00:00
parent f9f81cda61
commit 5d022a627b
3 changed files with 4 additions and 3 deletions

View File

@ -9,6 +9,7 @@ jetty-7.0.1-SNAPSHOT
+ 289960 start.jar assumes command line args are configs
+ JETTY-1114 unsynchronised WebAppClassloader.getResource(String)
+ Fixed XSS issue in CookieDump demo servlet.
+ 291340 Race condition in onException() notifications
jetty-7.0.0
+ 289958 StatisticsServlet incorrectly adds StatisticsHandler

View File

@ -262,8 +262,8 @@ public class HttpConnection implements Connection
// but we don't report it as it is normal cancelling operation
if (_exchange.getStatus() != HttpExchange.STATUS_CANCELLING)
{
_exchange.getEventListener().onException(e);
_exchange.setStatus(HttpExchange.STATUS_EXCEPTED);
_exchange.getEventListener().onException(e);
}
}
}

View File

@ -253,8 +253,8 @@ public class HttpDestination
else if (_queue.size() > 0)
{
HttpExchange ex = _queue.removeFirst();
ex.getEventListener().onConnectionFailed(throwable);
ex.setStatus(HttpExchange.STATUS_EXCEPTED);
ex.getEventListener().onConnectionFailed(throwable);
}
}
@ -279,8 +279,8 @@ public class HttpDestination
if (_queue.size() > 0)
{
HttpExchange ex = _queue.removeFirst();
ex.getEventListener().onException(throwable);
ex.setStatus(HttpExchange.STATUS_EXCEPTED);
ex.getEventListener().onException(throwable);
}
}
}