[trafficserver] branch master updated: Fixes another memory leak related to OCSP

[rrm]

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

rrm 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 09e3ba2  Fixes another memory leak related to OCSP
09e3ba2 is described below

commit 09e3ba2c1e202d1eca854aaece2c77fa30220e90
Author: Randall Meyer <randallme...@yahoo.com>
AuthorDate: Wed Nov 7 14:18:42 2018 -0800

    Fixes another memory leak related to OCSP
    
    also fixes a thinko I made in a previous commit (d83798a5)
---
 iocore/net/OCSPStapling.cc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/iocore/net/OCSPStapling.cc b/iocore/net/OCSPStapling.cc
index 90133e4..ee0534c 100644
--- a/iocore/net/OCSPStapling.cc
+++ b/iocore/net/OCSPStapling.cc
@@ -60,6 +60,9 @@ certinfo_free(void * /*parent*/, void *ptr, CRYPTO_EX_DATA * 
/*ad*/, int /*idx*/
   if (cinf->certname) {
     ats_free(cinf->certname);
   }
+  if (cinf->cid) {
+    OCSP_CERTID_free(cinf->cid);
+  }
   ink_mutex_destroy(&cinf->stapling_mutex);
   OPENSSL_free(cinf);
 }
@@ -193,7 +196,7 @@ ssl_stapling_init_cert(SSL_CTX *ctx, X509 *cert, const char 
*certname)
   return true;
 
 err:
-  if (cinf->uri) {
+  if (cinf->cid) {
     OCSP_CERTID_free(cinf->cid);
   }