This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 11.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/11.0.x by this push: new fd65e3c59e Return null SSL session on zero length array fd65e3c59e is described below commit fd65e3c59eaa2f2c9f507ba6ea67918d7a6e4bb2 Author: remm <r...@apache.org> AuthorDate: Mon Oct 7 11:13:27 2024 +0200 Return null SSL session on zero length array --- java/org/apache/tomcat/util/net/jsse/JSSESupport.java | 2 +- webapps/docs/changelog.xml | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/java/org/apache/tomcat/util/net/jsse/JSSESupport.java b/java/org/apache/tomcat/util/net/jsse/JSSESupport.java index 059cf7750c..d4438f66f4 100644 --- a/java/org/apache/tomcat/util/net/jsse/JSSESupport.java +++ b/java/org/apache/tomcat/util/net/jsse/JSSESupport.java @@ -170,7 +170,7 @@ public class JSSESupport implements SSLSupport, SSLSessionManager { } // Expose ssl_session (getId) byte [] ssl_session = session.getId(); - if ( ssl_session == null) { + if (ssl_session == null || ssl_session.length == 0) { return null; } StringBuilder buf=new StringBuilder(); diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml index 0c68760f90..c353d0e7e5 100644 --- a/webapps/docs/changelog.xml +++ b/webapps/docs/changelog.xml @@ -125,6 +125,14 @@ </scode> </changelog> </subsection> + <subsection name="Coyote"> + <changelog> + <fix> + Return null SSL session id on zero length byte array returned from the + SSL implementation. (remm) + </fix> + </changelog> + </subsection> </section> <section name="Tomcat 11.0.0 (markt)" rtext="release in progress"> <subsection name="Catalina"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org