[trafficserver] branch master updated: TLS Session Resumption: fix timed out session (#8667)

[du...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

duke8253 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new c82122e  TLS Session Resumption: fix timed out session (#8667)
c82122e is described below

commit c82122ebad10801efe60ef9d2fd454b8f358094c
Author: Mo Chen <un...@gmail.com>
AuthorDate: Mon Feb 14 14:11:06 2022 -0600

    TLS Session Resumption: fix timed out session (#8667)
    
    Reverts the behavior of getOriginSession() for a timed out session to the previous behavior, where it returns a null session pointer.
---
 iocore/net/TLSSessionResumptionSupport.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/iocore/net/TLSSessionResumptionSupport.cc b/iocore/net/TLSSessionResumptionSupport.cc
index af22b2e..36cfbb5 100644
--- a/iocore/net/TLSSessionResumptionSupport.cc
+++ b/iocore/net/TLSSessionResumptionSupport.cc
@@ -184,6 +184,7 @@ TLSSessionResumptionSupport::getOriginSession(SSL *ssl, const std::string &looku
     if (is_ssl_session_timed_out(shared_sess.get())) {
       SSL_INCREMENT_DYN_STAT(ssl_origin_session_cache_miss);
       origin_sess_cache->remove_session(lookup_key);
+      shared_sess.reset();
     } else {
       SSL_INCREMENT_DYN_STAT(ssl_origin_session_cache_hit);
       this->_setSSLSessionCacheHit(true);