Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Clarify BN_mod_exp docs (1.1.0/1.0.2) #6138

Closed
wants to merge 2 commits into from
Closed

Clarify BN_mod_exp docs (1.1.0/1.0.2) #6138

wants to merge 2 commits into from

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

@mattcaswell
Clarify BN_mod_exp docs
2073acd 
Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes openssl#5082
@mattcaswell mattcaswell added branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch 1.1.0 labels Apr 30, 2018
@mattcaswell mattcaswell added this to the 1.1.0 milestone Apr 30, 2018
@mattcaswell mattcaswell mentioned this pull request May 1, 2018
Closed
@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
@mattcaswell
Clarify BN_mod_exp docs
966b3df 
Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes #5082

Reviewed-by: Rich Salz <rsalz@openssl.org>
(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
@mattcaswell
Clarify BN_mod_exp docs
a96022b 
Specifically this is not supported with an even modulus and
BN_FLG_CONSTTIME.

Fixes #5082

Reviewed-by: Rich Salz <rsalz@openssl.org>
(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
Reviewers

@richsalz richsalz richsalz approved these changes

Assignees
No one assigned
Labels
approval: done This pull request has the required number of approvals branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mattcaswell @richsalz