| commit | 5349ddb7479a2bc1f6044be4bc6cb81a778d3943 | [log] [tgz] |
|---|---|---|
| author | David Benjamin <davidben@google.com> | Mon Dec 31 07:28:49 2018 +0000 |
| committer | CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> | Mon Jan 14 20:04:39 2019 +0000 |
| tree | 1fec1ff77af3f4ef3282a959498ec666c999d1eb | |
| parent | 3cbb0299a28a8bd0136257251a78b91a96c5eec8 [diff] |
Fix RSAZ's OPENSSL_cleanse. https://boringssl-review.googlesource.com/28584 switched RSAZ's buffer to being externally-allocated, which means the OPENSSL_cleanse needs to be tweaked to match. Change-Id: I0a7307ac86aa10933d10d380ef652c355fed3ee9 Reviewed-on: https://boringssl-review.googlesource.com/c/34191 Reviewed-by: Adam Langley <agl@google.com> Commit-Queue: David Benjamin <davidben@google.com>
diff --git a/crypto/fipsmodule/bn/rsaz_exp.c b/crypto/fipsmodule/bn/rsaz_exp.c index 64dfff8..0e3221c 100644 --- a/crypto/fipsmodule/bn/rsaz_exp.c +++ b/crypto/fipsmodule/bn/rsaz_exp.c
@@ -233,7 +233,7 @@ rsaz_1024_red2norm_avx2(result_norm, result); - OPENSSL_cleanse(storage, sizeof(storage)); + OPENSSL_cleanse(storage, MOD_EXP_CTIME_STORAGE_LEN * sizeof(BN_ULONG)); } #endif // OPENSSL_X86_64