diff options
author | Pauli <pauli@openssl.org> | 2023-10-12 08:46:19 +1100 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2023-10-13 17:35:05 +0200 |
commit | 91a5c0e40cf272d18b65c9e4c9a0268f244758a8 (patch) | |
tree | b4688f9c4f5d8b6d81af03b621b2187133c821b6 /test | |
parent | 008ca01e506d85acf0cc06ea8f219a883328344c (diff) |
cmp: add null pointer check in tear_down test function
problem reported by: 2ourc3
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22355)
Diffstat (limited to 'test')
-rw-r--r-- | test/cmp_protect_test.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/test/cmp_protect_test.c b/test/cmp_protect_test.c index 0d2311fc29..89f342458e 100644 --- a/test/cmp_protect_test.c +++ b/test/cmp_protect_test.c @@ -38,15 +38,17 @@ static OSSL_PROVIDER *default_null_provider = NULL, *provider = NULL; static void tear_down(CMP_PROTECT_TEST_FIXTURE *fixture) { - OSSL_CMP_CTX_free(fixture->cmp_ctx); - OSSL_CMP_MSG_free(fixture->msg); - OSSL_CMP_PKISI_free(fixture->si); + if (fixture != NULL) { + OSSL_CMP_CTX_free(fixture->cmp_ctx); + OSSL_CMP_MSG_free(fixture->msg); + OSSL_CMP_PKISI_free(fixture->si); - OPENSSL_free(fixture->mem); - sk_X509_free(fixture->certs); - sk_X509_free(fixture->chain); + OPENSSL_free(fixture->mem); + sk_X509_free(fixture->certs); + sk_X509_free(fixture->chain); - OPENSSL_free(fixture); + OPENSSL_free(fixture); + } } static CMP_PROTECT_TEST_FIXTURE *set_up(const char *const test_case_name) |