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 a warning about missing prototype on arm #6028
Fix a warning about missing prototype on arm #6028
Conversation
@@ -74,6 +74,8 @@ FILE *openssl_fopen(const char *filename, const char *mode); | |||
void *openssl_fopen(const char *filename, const char *mode); | |||
# endif | |||
|
|||
unsigned long OPENSSL_rdtsc(void); |
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.
Wasn't it decided in #4757 after a lengthy discussion that this should consistently be a uint32_t? (see db26a403b4de69d7846656362e70f5083428)
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.
correction: d807db2
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.
Maybe as another PR. This PR is just trying the fix the --strict-warnings build.
There are likely more issues, for instance sparcv9cap.c
defines "unsigned long OPENSSL_rdtsc(void)" without including "internal/cryptlib.h" first.
Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from #6028)
Merged to 1.1.0 |
(Merged from openssl#6028) (cherry picked from commit 5fc89c1)
(Merged from openssl#6028) (cherry picked from commit 5fc89c1)
(Merged from openssl#6028) (cherry picked from commit 5fc89c1)
No description provided.