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
speed: Always reset the outlen when calling EVP_PKEY_derive #18777
Conversation
f0bda39
to
f41af5b
Compare
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.
Just a nit in a comment :)
apps/speed.c
Outdated
int count; | ||
|
||
for (count = 0; COND(ffdh_c[testnum][0]); count++) | ||
for (count = 0; COND(ffdh_c[testnum][0]); count++) { | ||
/* outlen can be overwritten with too small value (no padding used) */ |
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.
too small a value.
Approved still if the comment is updated |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged to master and 3.0. Thank you. |
Fixes openssl#18768 Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Shane Lontis <shane.lontis@oracle.com> Reviewed-by: Hugo Landau <hlandau@openssl.org> (Merged from openssl#18777)
Fixes #18768