From d111712f6a7ae0ce37062d75fa3fa72e277e7455 Mon Sep 17 00:00:00 2001 From: Pauli Date: Mon, 9 Sep 2019 08:04:45 +1000 Subject: [PATCH] Coverity 1453633: Error handling issues (CHECKED_RETURN) Reviewed-by: Bernd Edlinger (Merged from https://github.com/openssl/openssl/pull/9805) --- providers/common/kdfs/pbkdf2.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/providers/common/kdfs/pbkdf2.c b/providers/common/kdfs/pbkdf2.c index ce395576dd..c8480125b2 100644 --- a/providers/common/kdfs/pbkdf2.c +++ b/providers/common/kdfs/pbkdf2.c @@ -107,7 +107,9 @@ static void kdf_pbkdf2_init(KDF_PBKDF2 *ctx) params[0] = OSSL_PARAM_construct_utf8_string(OSSL_KDF_PARAM_DIGEST, SN_sha1, 0); - ossl_prov_digest_load_from_params(&ctx->digest, params, provctx); + if (!ossl_prov_digest_load_from_params(&ctx->digest, params, provctx)) + /* This is an error, but there is no way to indicate such directly */ + ossl_prov_digest_reset(&ctx->digest); ctx->iter = PKCS5_DEFAULT_ITER; ctx->lower_bound_checks = KDF_PBKDF2_DEFAULT_CHECKS; } -- 2.25.1