Author: markt Date: Thu Jan 5 15:08:22 2017 New Revision: 1777472 URL: http://svn.apache.org/viewvc?rev=1777472&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60409 When unable to complete sendfile request, ensure the Processor will be added to the cache only once.
Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java?rev=1777472&r1=1777471&r2=1777472&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java Thu Jan 5 15:08:22 2017 @@ -1872,11 +1872,15 @@ public class NioEndpoint extends Abstrac } }catch ( IOException x ) { if ( log.isDebugEnabled() ) log.debug("Unable to complete sendfile request:", x); - cancelledKey(sk,SocketStatus.ERROR,false); + if (!calledByProcessor) { + cancelledKey(sk,SocketStatus.ERROR,false); + } return SendfileState.ERROR; }catch ( Throwable t ) { log.error("",t); - cancelledKey(sk, SocketStatus.ERROR, false); + if (!calledByProcessor) { + cancelledKey(sk, SocketStatus.ERROR, false); + } return SendfileState.ERROR; } } Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1777472&r1=1777471&r2=1777472&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Thu Jan 5 15:08:22 2017 @@ -50,6 +50,10 @@ <fix> <bug>57799</bug>: Remove useless sendfile check for NIO SSL. (remm) </fix> + <fix> + <bug>60409</bug>: When unable to complete sendfile request, ensure the + Processor will be added to the cache only once. (markt/violetagg) + </fix> </changelog> </subsection> <subsection name="Jasper"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org