Skip to content

Comments

Clarify BN_mod_exp docs (1.1.0/1.0.2)#6138

Closed
mattcaswell wants to merge 2 commits intoopenssl:OpenSSL_1_1_0-stablefrom
mattcaswell:clarify-bn-mod-exp-110
Closed

Clarify BN_mod_exp docs (1.1.0/1.0.2)#6138
mattcaswell wants to merge 2 commits intoopenssl:OpenSSL_1_1_0-stablefrom
mattcaswell:clarify-bn-mod-exp-110

Conversation

@mattcaswell
Copy link
Member

Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes #5082

This is the 1.1.0/1.0.2 version of #6137

Checklist
  • documentation is added or updated
  • tests are added or updated

Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes openssl#5082
@mattcaswell mattcaswell added branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL) 1.1.0 labels Apr 30, 2018
@mattcaswell mattcaswell added this to the 1.1.0 milestone Apr 30, 2018
@richsalz richsalz added the approval: done This pull request has the required number of approvals label May 1, 2018
levitte pushed a commit that referenced this pull request May 1, 2018
Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes #5082

Reviewed-by: Rich Salz <[email protected]>
(Merged from #6138)
@mattcaswell
Copy link
Member Author

Pushed. Thanks.

@mattcaswell mattcaswell closed this May 1, 2018
levitte pushed a commit that referenced this pull request May 1, 2018
Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes #5082

Reviewed-by: Rich Salz <[email protected]>
(Merged from #6138)

(cherry picked from commit 966b3df)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants