Author: markt
Date: Tue Nov 26 10:53:16 2013
New Revision: 1545606
URL: http://svn.apache.org/r1545606
Log:
Test for finished request before doing any processing
Modified:
tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
Modified: tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java?rev=1545606&r1=1545605&r2=1545606&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
(original)
+++ tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java Tue Nov
26 10:53:16 2013
@@ -539,7 +539,7 @@ public class CoyoteAdapter implements Ad
if (asyncConImpl != null) {
async = true;
ReadListener readListener = req.getReadListener();
- if (readListener != null) {
+ if (readListener != null && request.isFinished()) {
// Possible the all data may have been read during
service()
// method so this needs to be checked here
ClassLoader oldCL =
@@ -548,9 +548,7 @@ public class CoyoteAdapter implements Ad
request.getContext().getLoader().getClassLoader();
try {
Thread.currentThread().setContextClassLoader(newCL);
- if (request.isFinished()) {
- req.getReadListener().onAllDataRead();
- }
+ req.getReadListener().onAllDataRead();
} finally {
Thread.currentThread().setContextClassLoader(oldCL);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
