Skip to content

Commit 162089a

Browse files
jogmenhorman
authored andcommitted
Address coverity issue 1655295
Fixes: openssl/project#1267 Signed-off-by: Norbert Pocs <norbertp@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Tim Hudson <tjh@openssl.org> Reviewed-by: Paul Dale <ppzgs1@gmail.com> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from openssl#27989)
1 parent f13abf3 commit 162089a

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

providers/implementations/keymgmt/ml_kem_kmgmt.c.in

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,8 @@ void *ml_kem_load(const void *reference, size_t reference_sz)
542542
}
543543

544544
err:
545-
OPENSSL_secure_clear_free(encoded_dk, key->vinfo->prvkey_bytes);
545+
if (key != NULL && key->vinfo != NULL)
546+
OPENSSL_secure_clear_free(encoded_dk, key->vinfo->prvkey_bytes);
546547
ossl_ml_kem_key_free(key);
547548
return NULL;
548549
}

0 commit comments

Comments
 (0)