diff options
author | Angel Baez <51308340+abaez004@users.noreply.github.com> | 2024-02-07 10:34:48 -0500 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2024-02-09 10:34:16 +0100 |
commit | 76cecff5e9bedb2bafc60062283f99722697082a (patch) | |
tree | 09c880f9237bcf1dada3e82282e01bfc13f3db6d /crypto/ec/curve448 | |
parent | 9c7ad6989125e325f5496b37f236cf573372c6c7 (diff) |
Rearrange terms in gf_mul to prevent segfault
CLA: trivial
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/23512)
Diffstat (limited to 'crypto/ec/curve448')
-rw-r--r-- | crypto/ec/curve448/arch_64/f_impl64.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/crypto/ec/curve448/arch_64/f_impl64.c b/crypto/ec/curve448/arch_64/f_impl64.c index c944005da5..c8f7671788 100644 --- a/crypto/ec/curve448/arch_64/f_impl64.c +++ b/crypto/ec/curve448/arch_64/f_impl64.c @@ -45,9 +45,9 @@ void ossl_gf_mul(gf_s * RESTRICT cs, const gf as, const gf bs) accum0 += widemul(a[j + 4], b[i - j + 4]); } for (; j < 4; j++) { - accum2 += widemul(a[j], b[i - j + 8]); - accum1 += widemul(aa[j], bbb[i - j + 4]); - accum0 += widemul(a[j + 4], bb[i - j + 4]); + accum2 += widemul(a[j], b[i + 8 - j]); + accum1 += widemul(aa[j], bbb[i + 4 - j]); + accum0 += widemul(a[j + 4], bb[i + 4 - j]); } accum1 -= accum2; |