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

does not build with libressl #50

Closed
asac opened this issue Jan 22, 2017 · 2 comments
Closed

does not build with libressl #50

asac opened this issue Jan 22, 2017 · 2 comments

Comments

@asac
Copy link

asac commented Jan 22, 2017

libressl dropped FIPS_mode_set.

For me the following would work at least. Don't think it would hurt anyone with pristine openssl either, so maybe consider this:

diff --git a/src/tlsio_openssl.c b/src/tlsio_openssl.c
index 5cd805b..a9db3c1 100644
--- a/src/tlsio_openssl.c
+++ b/src/tlsio_openssl.c
@@ -930,7 +930,9 @@ void tlsio_openssl_deinit(void)
 {
     openssl_dynamic_locks_uninstall();
     openssl_static_locks_uninstall();
+#ifndef LIBRESSL_VERSION_NUMBER
     FIPS_mode_set(0);
+#endif
     CRYPTO_set_locking_callback(NULL);
     CRYPTO_set_id_callback(NULL);
     ERR_free_strings();
@anporumb
Copy link
Collaborator

Hello @asac ,

We have a fix checked in master branch and will close the issue once we publish packages.

Best Regards,
Andrei Porumb

@anporumb
Copy link
Collaborator

Published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants