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

Fix OpenSSL 3.0 deprecation error for SHA256 functions #344

Merged
merged 1 commit into from
May 19, 2022

Conversation

StarWitch
Copy link
Member

This is a small fix for compiling with OpenSSL 3.0. As of 3.0, the SHA256_* functions are deprecated in favour of EVP_* functions, which then triggers an error when -Werror is enabled.

Functionality is the same, and I streamlined things slightly. There's also a single-line change for eliminating an unused variable error.

@StarWitch StarWitch requested review from laochailan and Akaricchi May 9, 2022 19:51
@StarWitch StarWitch self-assigned this May 9, 2022
@StarWitch StarWitch force-pushed the fix_openssl_deprecation branch 2 times, most recently from 528e1b0 to 6114c95 Compare May 11, 2022 16:15
@Akaricchi
Copy link
Member

LGTM except the stage2 commit doesn't belong here. I suggest pushing that to master first and then rebasing this PR (can squish all the commits as well while you're at it)

PR review changes

add libssl to test suite

fix segfault
@StarWitch StarWitch force-pushed the fix_openssl_deprecation branch from 1d7d9f7 to 118fcfb Compare May 19, 2022 19:03
@StarWitch StarWitch merged commit b8a2188 into master May 19, 2022
@StarWitch StarWitch deleted the fix_openssl_deprecation branch May 19, 2022 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants