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

bneradt 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 d357538a61 SSLConfig: free temporary ALPN string (#11051)
d357538a61 is described below

commit d357538a61306d2e46d3ea532460171e808f460c
Author: Brian Neradt <brian.ner...@gmail.com>
AuthorDate: Wed Feb 7 13:55:41 2024 -0600

    SSLConfig: free temporary ALPN string (#11051)
    
    We allocate a temporary string for the user's client-side ALPN
    configuration, but don't free it after using it. This frees the string
    after we're done with it.
---
 src/iocore/net/SSLConfig.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/iocore/net/SSLConfig.cc b/src/iocore/net/SSLConfig.cc
index 15dbcbfa81..df1f2299e2 100644
--- a/src/iocore/net/SSLConfig.cc
+++ b/src/iocore/net/SSLConfig.cc
@@ -366,6 +366,7 @@ SSLConfigParams::initialize()
   if (clientALPNProtocols) {
     this->alpn_protocols_array_size = MAX_ALPN_STRING;
     convert_alpn_to_wire_format(clientALPNProtocols, 
this->alpn_protocols_array, this->alpn_protocols_array_size);
+    ats_free(clientALPNProtocols);
   }
 
 #ifdef SSL_OP_CIPHER_SERVER_PREFERENCE

Reply via email to