check the return value of OPENSSL_strdup to prevent potential memory access error

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/17110)

diff --git a/apps/lib/engine_loader.c b/apps/lib/engine_loader.c
index 7ea05943f380d08b7758592af961334de81f84e2..aa1bad26a4b8a54821eed24f9b7fb533c2b9c083 100644 (file)
--- a/apps/lib/engine_loader.c
+++ b/apps/lib/engine_loader.c
@@ -89,7 +89,7 @@ static OSSL_STORE_LOADER_CTX *engine_open(const OSSL_STORE_LOADER *loader,
         keyid = OPENSSL_strdup(q + 1);
     }
 
-    if (e != NULL)
+    if (e != NULL && keyid != NULL)
         ctx = OSSL_STORE_LOADER_CTX_new(e, keyid);
 
     if (ctx == NULL) {