tree 23ec2124c704a5edfb23a51c6392fd1ce4e3438a
parent da2630c190542fff25a2d925610c8e8a7f75e2f7
author Brian Smith <brian@briansmith.org> 1469948032 -1000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1470169498 +0000

Use BN_nnmod instead of BN_mod in BN_mod_exp_mont_consttime.

|BN_mod_exp_mont| uses |BN_nnmod| so it seems like
|BN_mod_exp_mont_consttime| should too. Further, I created
these test vectors by doing the math by hand, and the tests
passed for |BN_mod_exp_mont| but failed for
|BN_mod_exp_mont_consttime| without this change.

Change-Id: I7cffa1375e94dd8eaee87ada78285cd67fff1bac
Reviewed-on: https://boringssl-review.googlesource.com/9032
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: Adam Langley <agl@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>