diff options
author | Rohan McLure <rohanmclure@linux.ibm.com> | 2023-08-15 15:20:20 +1000 |
---|---|---|
committer | Hugo Landau <hlandau@openssl.org> | 2023-09-01 10:06:54 +0100 |
commit | 670e73d9084465384b11ef24802ca4a313e1d2f4 (patch) | |
tree | 062fdf3665f69a98eda0b41faa4375e83ddbbcf6 /crypto | |
parent | 60421893a286bb9eb7fb7c2454b84af9778ffca4 (diff) | |
download | openssl-670e73d9084465384b11ef24802ca4a313e1d2f4.tar.gz |
ecc: Remove extraneous parentheses in secp384r1
Substitutions in the felem_reduce() method feature unecessary
parentheses, remove them.
Signed-off-by: Rohan McLure <rohan.mclure@linux.ibm.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21749)
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/ec/ecp_nistp384.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/crypto/ec/ecp_nistp384.c b/crypto/ec/ecp_nistp384.c index 14f9530d07..ff68f9cc7a 100644 --- a/crypto/ec/ecp_nistp384.c +++ b/crypto/ec/ecp_nistp384.c @@ -540,7 +540,7 @@ static void felem_reduce(felem out, const widefelem in) acc[7] += in[12] >> 8; acc[6] += (in[12] & 0xff) << 48; acc[6] -= in[12] >> 16; - acc[5] -= ((in[12] & 0xffff) << 40); + acc[5] -= (in[12] & 0xffff) << 40; acc[6] += in[12] >> 48; acc[5] += (in[12] & 0xffffffffffff) << 8; @@ -549,7 +549,7 @@ static void felem_reduce(felem out, const widefelem in) acc[6] += in[11] >> 8; acc[5] += (in[11] & 0xff) << 48; acc[5] -= in[11] >> 16; - acc[4] -= ((in[11] & 0xffff) << 40); + acc[4] -= (in[11] & 0xffff) << 40; acc[5] += in[11] >> 48; acc[4] += (in[11] & 0xffffffffffff) << 8; @@ -558,7 +558,7 @@ static void felem_reduce(felem out, const widefelem in) acc[5] += in[10] >> 8; acc[4] += (in[10] & 0xff) << 48; acc[4] -= in[10] >> 16; - acc[3] -= ((in[10] & 0xffff) << 40); + acc[3] -= (in[10] & 0xffff) << 40; acc[4] += in[10] >> 48; acc[3] += (in[10] & 0xffffffffffff) << 8; @@ -567,7 +567,7 @@ static void felem_reduce(felem out, const widefelem in) acc[4] += in[9] >> 8; acc[3] += (in[9] & 0xff) << 48; acc[3] -= in[9] >> 16; - acc[2] -= ((in[9] & 0xffff) << 40); + acc[2] -= (in[9] & 0xffff) << 40; acc[3] += in[9] >> 48; acc[2] += (in[9] & 0xffffffffffff) << 8; @@ -582,7 +582,7 @@ static void felem_reduce(felem out, const widefelem in) acc[3] += acc[8] >> 8; acc[2] += (acc[8] & 0xff) << 48; acc[2] -= acc[8] >> 16; - acc[1] -= ((acc[8] & 0xffff) << 40); + acc[1] -= (acc[8] & 0xffff) << 40; acc[2] += acc[8] >> 48; acc[1] += (acc[8] & 0xffffffffffff) << 8; @@ -591,7 +591,7 @@ static void felem_reduce(felem out, const widefelem in) acc[2] += acc[7] >> 8; acc[1] += (acc[7] & 0xff) << 48; acc[1] -= acc[7] >> 16; - acc[0] -= ((acc[7] & 0xffff) << 40); + acc[0] -= (acc[7] & 0xffff) << 40; acc[1] += acc[7] >> 48; acc[0] += (acc[7] & 0xffffffffffff) << 8; |