speed: range check the argument given to -multi for 1.1.1#16922
Closed
paulidale wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
Closed
speed: range check the argument given to -multi for 1.1.1#16922paulidale wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
paulidale wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
Conversation
Fixes openssl#16899 for 1.1.1 branch.
paulidale
added
approval: review pending
slontis
reviewed
Oct 27, 2021
| case OPT_MULTI: | ||
| #ifndef NO_FORK | ||
| multi = atoi(opt_arg()); | ||
| if (multi >= INT_MAX / (int)sizeof(int)) { |
Member
There was a problem hiding this comment.
perhaps a comment here explaining why you do the divide is a good idea?
Contributor
Author
There was a problem hiding this comment.
There isn't such a comment in the equivalent code in master or 3.0 :)
t8m
approved these changes
Oct 27, 2021
openssl-machine
pushed a commit
that referenced
this pull request
Oct 28, 2021
Fixes #16899 for 1.1.1 branch. Reviewed-by: Tomas Mraz <[email protected]> (Merged from #16922)
Contributor
Author
|
Merged. |
Fixes #16899 for 1.1.1 branch.