Author: markt
Date: Tue May 31 17:37:25 2016
New Revision: 1746306

URL: http://svn.apache.org/viewvc?rev=1746306&view=rev
Log:
Simplify.
If getErrorState().isError() is true then response.getErrorException() will 
always be null.

Modified:
    tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java?rev=1746306&r1=1746305&r2=1746306&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Tue May 31 
17:37:25 2016
@@ -1117,10 +1117,8 @@ public class Http11Processor extends Abs
                     // set the status to 500 and set the errorException.
                     // If we fail here, then the response is likely already
                     // committed, so we can't try and set headers.
-                    if(keepAlive && !getErrorState().isError() && (
-                            response.getErrorException() != null ||
-                                    (!isAsync() &&
-                                    
statusDropsConnection(response.getStatus())))) {
+                    if(keepAlive && !getErrorState().isError() && !isAsync() &&
+                            statusDropsConnection(response.getStatus())) {
                         setErrorState(ErrorState.CLOSE_CLEAN, null);
                     }
                 } catch (InterruptedIOException e) {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to