This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push: new 67909e0 Add comment to explain the cleanup 67909e0 is described below commit 67909e03e2a8c00a5fe6b8214f26305a67548fd1 Author: remm <r...@apache.org> AuthorDate: Fri Nov 8 09:38:14 2019 +0100 Add comment to explain the cleanup --- java/org/apache/tomcat/util/net/AprEndpoint.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/java/org/apache/tomcat/util/net/AprEndpoint.java b/java/org/apache/tomcat/util/net/AprEndpoint.java index 0315075..1d40f29 100644 --- a/java/org/apache/tomcat/util/net/AprEndpoint.java +++ b/java/org/apache/tomcat/util/net/AprEndpoint.java @@ -695,6 +695,8 @@ public class AprEndpoint extends AbstractEndpoint<Long,Long> implements SNICallB // the pool and its queue are full log.error(sm.getString("endpoint.process.fail"), t); } + // Note: doing connections.remove is not needed since closeSocket will + // be called, which then calls close on the wrapper return false; } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org