diff options
author | Dr. Stephen Henson <steve@openssl.org> | 2016-03-07 17:31:00 +0000 |
---|---|---|
committer | Dr. Stephen Henson <steve@openssl.org> | 2016-03-07 22:15:04 +0000 |
commit | e1d9f1ab39eeab0c3c2b9415e0aaaa8c05858f77 (patch) | |
tree | ac2225dbe271f7b6250653cdddd21382c781b163 /crypto/dsa | |
parent | f7c4584b15320d94863daba2996eee21dde23732 (diff) | |
download | openssl-e1d9f1ab39eeab0c3c2b9415e0aaaa8c05858f77.tar.gz |
Remove kinv/r fields from DSA structure.
The kinv/r fields in the DSA structure are not used by OpenSSL internally
and should not be used in general.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Diffstat (limited to 'crypto/dsa')
-rw-r--r-- | crypto/dsa/dsa_lib.c | 2 | ||||
-rw-r--r-- | crypto/dsa/dsa_ossl.c | 12 |
2 files changed, 2 insertions, 12 deletions
diff --git a/crypto/dsa/dsa_lib.c b/crypto/dsa/dsa_lib.c index f89297313a..f7795b27bd 100644 --- a/crypto/dsa/dsa_lib.c +++ b/crypto/dsa/dsa_lib.c @@ -180,8 +180,6 @@ void DSA_free(DSA *r) BN_clear_free(r->g); BN_clear_free(r->pub_key); BN_clear_free(r->priv_key); - BN_clear_free(r->kinv); - BN_clear_free(r->r); OPENSSL_free(r); } diff --git a/crypto/dsa/dsa_ossl.c b/crypto/dsa/dsa_ossl.c index cbc1e41bb9..0874e89141 100644 --- a/crypto/dsa/dsa_ossl.c +++ b/crypto/dsa/dsa_ossl.c @@ -158,16 +158,8 @@ static DSA_SIG *dsa_do_sign(const unsigned char *dgst, int dlen, DSA *dsa) if (ctx == NULL) goto err; redo: - if ((dsa->kinv == NULL) || (dsa->r == NULL)) { - if (!dsa_sign_setup(dsa, ctx, &kinv, &r, dgst, dlen)) - goto err; - } else { - kinv = dsa->kinv; - dsa->kinv = NULL; - r = dsa->r; - dsa->r = NULL; - noredo = 1; - } + if (!dsa_sign_setup(dsa, ctx, &kinv, &r, dgst, dlen)) + goto err; if (dlen > BN_num_bytes(dsa->q)) /* |