diff options
author | David Benjamin <davidben@google.com> | 2018-07-17 13:20:28 -0400 |
---|---|---|
committer | David Benjamin <davidben@google.com> | 2018-07-24 11:48:48 -0400 |
commit | 61ac9fc5c44718bf61ab68328333cc158230d090 (patch) | |
tree | 6c236e8f20840c8e3a5cda8f9f64d704524956ad /crypto/bn | |
parent | 8937a4ed8ac3fd64be61e9ce7a16bccccf3d2273 (diff) |
Remove zero special-case in BN_mod_exp_mont.
A number intended to treat the base as secret should not be branching on
whether it is zero. Test-wise, this is covered by existing tests in bnmod.txt.
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6733)
Diffstat (limited to 'crypto/bn')
-rw-r--r-- | crypto/bn/bn_exp.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/crypto/bn/bn_exp.c b/crypto/bn/bn_exp.c index 4e24c94ad3..2c92d7eac9 100644 --- a/crypto/bn/bn_exp.c +++ b/crypto/bn/bn_exp.c @@ -356,11 +356,6 @@ int BN_mod_exp_mont(BIGNUM *rr, const BIGNUM *a, const BIGNUM *p, aa = val[0]; } else aa = a; - if (BN_is_zero(aa)) { - BN_zero(rr); - ret = 1; - goto err; - } if (!bn_to_mont_fixed_top(val[0], aa, mont, ctx)) goto err; /* 1 */ |