diff options
| author | Brian Smith <brian@briansmith.org> | 2016-03-16 02:10:11 +0300 |
|---|---|---|
| committer | David Benjamin <davidben@google.com> | 2016-03-20 20:18:35 +0300 |
| commit | fdc955cf14c79fb5399652f3a166e7fc4dec0767 (patch) | |
| tree | f2b13323270dce95e4ce80697be7ed26d8a85a15 /crypto/ec | |
| parent | df1201e6eefc920fd70d693db4bfb5f76f177b95 (diff) | |
Fix parameter type of p256-64.c's |select_point|.
Make it match how it is done in p224-64.c. Note in particular that
|size| may be 17, so presumably |pre_comp[16]| is accessed, which one
would not expect when it was declared |precomp[16][3]|.
Change-Id: I54c1555f9e20ccaacbd4cd75a7154b483b4197b7
Reviewed-on: https://boringssl-review.googlesource.com/7467
Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'crypto/ec')
| -rw-r--r-- | crypto/ec/p256-64.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/crypto/ec/p256-64.c b/crypto/ec/p256-64.c index 24778c46..e3b7c480 100644 --- a/crypto/ec/p256-64.c +++ b/crypto/ec/p256-64.c @@ -1404,7 +1404,8 @@ static const smallfelem g_pre_comp[2][16][3] = { /* select_point selects the |idx|th point from a precomputation table and * copies it to out. */ static void select_point(const u64 idx, unsigned int size, - const smallfelem pre_comp[16][3], smallfelem out[3]) { + const smallfelem pre_comp[/*size*/][3], + smallfelem out[3]) { unsigned i, j; u64 *outlimbs = &out[0][0]; memset(outlimbs, 0, 3 * sizeof(smallfelem)); |