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 <[email protected]>
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
