[trafficserver] branch 7.1.x updated: SSL_CTX_remove_session() and external cache

[zw...@apache.org]

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

zwoop pushed a commit to branch 7.1.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/7.1.x by this push:
       new  66b29f4   SSL_CTX_remove_session() and external cache
66b29f4 is described below

commit 66b29f4da1155684ef1cf02f22d1a33e6e7a7023
Author: Jack Bates <ja...@nottheoilrig.com>
AuthorDate: Sat Jan 28 15:25:31 2017 -0700

    SSL_CTX_remove_session() and external cache
    
    The remove session callback is triggered when a session is removed,
    so at best, calling SSL_CTX_remove_session() again is redundant.
    In OpenSSL 1.1 it's recursive.
    
    Fixes #1386
    
    (cherry picked from commit cf33b5bfa2cbb362cff2f13711e718f5316c8589)
---
 iocore/net/SSLUtils.cc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/iocore/net/SSLUtils.cc b/iocore/net/SSLUtils.cc
index 15f41b7..01268ad 100644
--- a/iocore/net/SSLUtils.cc
+++ b/iocore/net/SSLUtils.cc
@@ -281,8 +281,6 @@ ssl_new_cached_session(SSL *ssl, SSL_SESSION *sess)
 static void
 ssl_rm_cached_session(SSL_CTX *ctx, SSL_SESSION *sess)
 {
-  SSL_CTX_remove_session(ctx, sess);
-
   unsigned int len        = 0;
   const unsigned char *id = SSL_SESSION_get_id(sess, &len);
   SSLSessionID sid(id, len);

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].