Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some fixes for curl_global_sslset() when built with a single SSL backend #3346

Closed
wants to merge 3 commits into from

Conversation

dscho
Copy link
Contributor

@dscho dscho commented Dec 6, 2018

While working on #3345, I realized that there were a couple of issues with the curl_global_sslset() when built with a single SSL backend, resulting in an incorrect error messages in one case, and in another case an error when we should have succeeded.

This PR fixes both issues.

lib/vtls/vtls.c Outdated Show resolved Hide resolved
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
It is allowed to call that function with id set to -1, specifying the
backend by the name instead. We should imitate what is done further down
in that function to allow for that.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
When compiled with a single SSL backend, Curl_ssl is never set to
Curl_ssl_multi, but to that single backend from the beginning.

When the caller tries to set the SSL backend to a different backend than
the one that was compiled in, in this scenario we should not say that it
is too late to set the SSL backend, but that it is an unknown backend to
begin with.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@bagder
Copy link
Member

bagder commented Dec 7, 2018

Thanks!

@bagder bagder closed this in 2456152 Dec 7, 2018
@dscho dscho deleted the ssl-backend-fixes branch December 7, 2018 11:20
@dscho
Copy link
Contributor Author

dscho commented Dec 7, 2018

@bagder thank you so much! I would also like to take a moment to thank you for being an excellent open source maintainer. You know, I strive to imitate you in your welcoming, gentle and inviting manners?

@bagder
Copy link
Member

bagder commented Dec 7, 2018

Thank you. It is certainly good to know that my efforts pay off. Getting praise from equals is the highest form of flattery I can imagine! ✨

@lock lock bot locked as resolved and limited conversation to collaborators Mar 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants