From bcb018e70b596811146a17d95532498bbe3ef13a Mon Sep 17 00:00:00 2001 From: Richard Levitte Date: Wed, 29 Apr 2020 06:55:40 +0200 Subject: [PATCH] EVP: Only use the engine when one is defined, in pkey_mac_ctrl() Fixes #11671 Reviewed-by: Shane Lontis (Merged from https://github.com/openssl/openssl/pull/11674) --- crypto/evp/pkey_mac.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/crypto/evp/pkey_mac.c b/crypto/evp/pkey_mac.c index 56231e3938..3503aac6d3 100644 --- a/crypto/evp/pkey_mac.c +++ b/crypto/evp/pkey_mac.c @@ -308,11 +308,14 @@ static int pkey_mac_ctrl(EVP_PKEY_CTX *ctx, int type, int p1, void *p2) OSSL_PARAM params[3]; size_t params_n = 0; char *ciphname = (char *)OBJ_nid2sn(EVP_CIPHER_nid(p2)); + #ifndef OPENSSL_NO_ENGINE - char *engineid = (char *)ENGINE_get_id(ctx->engine); + if (ctx->engine != NULL) { + char *engid = (char *)ENGINE_get_id(ctx->engine); - params[params_n++] = - OSSL_PARAM_construct_utf8_string("engine", engineid, 0); + params[params_n++] = + OSSL_PARAM_construct_utf8_string("engine", engid, 0); + } #endif params[params_n++] = OSSL_PARAM_construct_utf8_string(OSSL_MAC_PARAM_CIPHER, @@ -458,13 +461,14 @@ static int pkey_mac_ctrl(EVP_PKEY_CTX *ctx, int type, int p1, void *p2) size_t params_n = 0; char *mdname = (char *)OBJ_nid2sn(EVP_MD_nid(hctx->raw_data.md)); + #ifndef OPENSSL_NO_ENGINE - char *engineid = ctx->engine == NULL - ? NULL : (char *)ENGINE_get_id(ctx->engine); + if (ctx->engine != NULL) { + char *engid = (char *)ENGINE_get_id(ctx->engine); - if (engineid != NULL) params[params_n++] = - OSSL_PARAM_construct_utf8_string("engine", engineid, 0); + OSSL_PARAM_construct_utf8_string("engine", engid, 0); + } #endif params[params_n++] = OSSL_PARAM_construct_utf8_string(OSSL_MAC_PARAM_DIGEST, -- 2.25.1