summaryrefslogtreecommitdiffstats
path: root/crypto/dsa/dsa_ossl.c
diff options
context:
space:
mode:
authorMatt Caswell <matt@openssl.org>2016-04-28 16:47:36 +0100
committerMatt Caswell <matt@openssl.org>2016-04-29 09:23:45 +0100
commit10a57adc606b86949007c076336e74970c2f336f (patch)
tree94e7d766760e0ecd71ca12180fc0b6fe7295c091 /crypto/dsa/dsa_ossl.c
parentb375f08145d78479674580042614c9e770b011dd (diff)
Remove some dead code
Commit e1d9f1ab39eea left some dead code behind. This removes it. Reviewed-by: Stephen Henson <steve@openssl.org>
Diffstat (limited to 'crypto/dsa/dsa_ossl.c')
-rw-r--r--crypto/dsa/dsa_ossl.c8
1 files changed, 1 insertions, 7 deletions
diff --git a/crypto/dsa/dsa_ossl.c b/crypto/dsa/dsa_ossl.c
index 92855537b8..2e4b3a70f9 100644
--- a/crypto/dsa/dsa_ossl.c
+++ b/crypto/dsa/dsa_ossl.c
@@ -140,7 +140,6 @@ static DSA_SIG *dsa_do_sign(const unsigned char *dgst, int dlen, DSA *dsa)
BN_CTX *ctx = NULL;
int reason = ERR_R_BN_LIB;
DSA_SIG *ret = NULL;
- int noredo = 0;
int rv = 0;
m = BN_new();
@@ -191,13 +190,8 @@ static DSA_SIG *dsa_do_sign(const unsigned char *dgst, int dlen, DSA *dsa)
* Redo if r or s is zero as required by FIPS 186-3: this is very
* unlikely.
*/
- if (BN_is_zero(r) || BN_is_zero(s)) {
- if (noredo) {
- reason = DSA_R_NEED_NEW_SETUP_VALUES;
- goto err;
- }
+ if (BN_is_zero(r) || BN_is_zero(s))
goto redo;
- }
rv = 1;