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 the code used to detect aarch64 capabilities when we don't have getauxval() #19833
Fix the code used to detect aarch64 capabilities when we don't have getauxval() #19833
Conversation
…etauxval() In addition to a missing prototype there was also a missing closing brace '}'. Fixes openssl#19825.
I don't think the Run-checker fail at |
@@ -354,6 +355,7 @@ void OPENSSL_cpuid_setup(void) | |||
if (sigsetjmp(ill_jmp, 1) == 0) { | |||
_armv8_sm3_probe(); | |||
OPENSSL_armcap_P |= ARMV8_SM3; | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
!!! How did this ever work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it was clearly never even tested :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(which I did do before this PR)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it was clearly never even tested :(
Which is why we fortunately have alpha and beta pre-releases. As we cannot test on all environments that might be possibly supported by OpenSSL code base. 😁
The message to anyone who is using OpenSSL on platform that is not on the primary or secondary list - test the prereleases if you want the final release to support your platform.
Is this master/3.1 only? Or does it apply to 3.0 too? |
Fortunately this isn't in 3.0 |
This pull request is ready to merge |
Merged to master and 3.1 branches. Thank you for your contribution. |
…etauxval() In addition to a missing prototype there was also a missing closing brace '}'. Fixes openssl#19825. Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#19833)
Fixes #19825.