From 9f54582361e6ad94aeceb529e3cb80dda810b841 Mon Sep 17 00:00:00 2001 From: Masakazu Kitajo Date: Thu, 17 Aug 2023 12:09:10 -0600 Subject: [PATCH] Coverity 1508984: Dereference null return value --- tests/tools/plugins/async_engine.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/tests/tools/plugins/async_engine.c b/tests/tools/plugins/async_engine.c index b596b418050..4fb349ab37f 100644 --- a/tests/tools/plugins/async_engine.c +++ b/tests/tools/plugins/async_engine.c @@ -73,10 +73,14 @@ EVP_PKEY * async_load_privkey(ENGINE *e, const char *s_key_id, UI_METHOD *ui_method, void *callback_data) { fprintf(stderr, "Loading key %s\n", s_key_id); - FILE *f = fopen(s_key_id, "r"); - EVP_PKEY *key = PEM_read_PrivateKey(f, NULL, NULL, NULL); - fclose(f); - return key; + FILE *f = fopen(s_key_id, "r"); + if (f) { + EVP_PKEY *key = PEM_read_PrivateKey(f, NULL, NULL, NULL); + fclose(f); + return key; + } else { + return NULL; + } } static int