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:45 +0200 |
commit | 6ff69665eb87db56b0f1fdbaf3113cb66f0c2fdd (patch) | |
tree | 7a72b6e4a12e2fe09e8509294d700d1c694f4175 /test | |
parent | a8293b44f8a8c8a6159a9a06ce1305fabc463eb1 (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)
(cherry picked from commit 91a5c0e40cf272d18b65c9e4c9a0268f244758a8)
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 32dae32d93..09bf2ec17f 100644 --- a/test/cmp_protect_test.c +++ b/test/cmp_protect_test.c @@ -37,15 +37,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) |