Skip to content

Commit

Permalink
Fix setting TLS groups with BoringSSL (#11840)
Browse files Browse the repository at this point in the history
* Fix setting TLS groups with BoringSSL

* Get rid of #ifdef for old openssl
  • Loading branch information
masaori335 authored Nov 17, 2024
1 parent 675b9f4 commit a860656
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions src/iocore/net/SSLUtils.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1530,20 +1530,15 @@ SSLMultiCertConfigLoader::_set_cipher_suites([[maybe_unused]] SSL_CTX *ctx)
}

bool
SSLMultiCertConfigLoader::_set_curves([[maybe_unused]] SSL_CTX *ctx)
SSLMultiCertConfigLoader::_set_curves(SSL_CTX *ctx)
{
#if defined(SSL_CTX_set1_groups_list) || defined(SSL_CTX_set1_curves_list)
if (this->_params->server_groups_list != nullptr) {
#ifdef SSL_CTX_set1_groups_list
if (!SSL_CTX_set1_groups_list(ctx, this->_params->server_groups_list)) {
#else
if (!SSL_CTX_set1_curves_list(ctx, this->_params->server_groups_list)) {
#endif
SSLError("invalid groups list for server in %s", ts::filename::RECORDS);
return false;
}
}
#endif

return true;
}

Expand Down

0 comments on commit a860656

Please sign in to comment.