coverity 1462545 Dereference after null check

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/11651)

diff --git a/crypto/evp/p_lib.c b/crypto/evp/p_lib.c
index 84fb91412f1a42a9f23f8dd98d2552b01ac56c22..053c9cb557e89ae38375b405a8c0f826dc7e3c9b 100644 (file)
--- a/crypto/evp/p_lib.c
+++ b/crypto/evp/p_lib.c
@@ -230,9 +230,11 @@ static int evp_pkey_cmp_any(const EVP_PKEY *a, const EVP_PKEY *b,
      * us to compare types using legacy NIDs.
      */
     if ((a->type != EVP_PKEY_NONE
      * us to compare types using legacy NIDs.
      */
     if ((a->type != EVP_PKEY_NONE

-         && !EVP_KEYMGMT_is_a(b->keymgmt, OBJ_nid2sn(a->type)))
+         && (b->keymgmt == NULL
+             || !EVP_KEYMGMT_is_a(b->keymgmt, OBJ_nid2sn(a->type))))

         || (b->type != EVP_PKEY_NONE
         || (b->type != EVP_PKEY_NONE

-            && !EVP_KEYMGMT_is_a(a->keymgmt, OBJ_nid2sn(b->type))))
+            && (a->keymgmt == NULL
+                || !EVP_KEYMGMT_is_a(a->keymgmt, OBJ_nid2sn(b->type)))))

         return -1;               /* not the same key type */
 
     /*
         return -1;               /* not the same key type */
 
     /*