Skip to content

curl-openssl: fix SRP check for OpenSSL 3.0#8394

Closed
MarcelRaad wants to merge 1 commit into
curl:masterfrom
MarcelRaad:fix_opensslv3_srp_check
Closed

curl-openssl: fix SRP check for OpenSSL 3.0#8394
MarcelRaad wants to merge 1 commit into
curl:masterfrom
MarcelRaad:fix_opensslv3_srp_check

Conversation

@MarcelRaad

Copy link
Copy Markdown
Member

When OpenSSL 3.0 is built with --api=3.0 and no-deprecated, the SRP
functions exist in the library, but are disabled for user code. Check
if they are actually usable instead of only if they exist. Also check
for the functions actually required for TLS-SRP.

TLS-SRP support is still enabled if OpenSSL is configured with just
--api=3.0 or with --api=1.1.1 no-deprecated.

(I'm not very familiar with autoconf and m4 - please let me know if there's a better way to do that.)

When OpenSSL 3.0 is built with `--api=3.0` and `no-deprecated`, the SRP
functions exist in the library, but are disabled for user code. Check
if they are actually usable instead of only if they exist. Also check
for the functions actually required for TLS-SRP.

TLS-SRP support is still enabled if OpenSSL is configured with just
`--api=3.0` or with `--api=1.1.1 no-deprecated`.

Closes

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me!

@MarcelRaad MarcelRaad closed this in fe9440f Feb 7, 2022
@MarcelRaad MarcelRaad deleted the fix_opensslv3_srp_check branch February 7, 2022 14:22
@MarcelRaad

Copy link
Copy Markdown
Member Author

Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants