diff options
| author | Matt Caswell <matt@openssl.org> | 2018-03-29 17:48:28 +0100 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2018-04-05 15:44:24 +0100 |
| commit | 06d3b485db8b6bfd5437c9998d92e882a3cdfa1f (patch) | |
| tree | f51d43ea9820fe3999013a451b1f10a08c586db5 /crypto | |
| parent | fdb8113daedbcc01e3effc5b0a1ed97558d700a5 (diff) | |
| download | openssl-06d3b485db8b6bfd5437c9998d92e882a3cdfa1f.zip openssl-06d3b485db8b6bfd5437c9998d92e882a3cdfa1f.tar.gz openssl-06d3b485db8b6bfd5437c9998d92e882a3cdfa1f.tar.bz2 | |
Don't crash if an unrecognised digest is used with dsa_paramgen_md
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5800)
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/dsa/dsa_pmeth.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/crypto/dsa/dsa_pmeth.c b/crypto/dsa/dsa_pmeth.c index 88744a9..4934276 100644 --- a/crypto/dsa/dsa_pmeth.c +++ b/crypto/dsa/dsa_pmeth.c @@ -189,9 +189,15 @@ static int pkey_dsa_ctrl_str(EVP_PKEY_CTX *ctx, NULL); } if (strcmp(type, "dsa_paramgen_md") == 0) { + const EVP_MD *md = EVP_get_digestbyname(value); + + if (md == NULL) { + DSAerr(DSA_F_PKEY_DSA_CTRL_STR, DSA_R_INVALID_DIGEST_TYPE); + return 0; + } return EVP_PKEY_CTX_ctrl(ctx, EVP_PKEY_DSA, EVP_PKEY_OP_PARAMGEN, EVP_PKEY_CTRL_DSA_PARAMGEN_MD, 0, - (void *)EVP_get_digestbyname(value)); + (void *)md); } return -2; } |