diff options
| author | Frederik Wedel-Heinen <frederik.wedel-heinen@dencrypt.dk> | 2024-12-21 15:15:11 +0100 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2025-01-08 11:11:00 +0100 |
| commit | c626fda8a66a203d9f1435c34fcd3f7bda89d068 (patch) | |
| tree | fe5219f4edf21176b559b96230cc68e9da0986e9 /test/v3nametest.c | |
| parent | Pass functions with correct signatures to the evp_generic_fetch_xxx methods (diff) | |
| download | openssl-c626fda8a66a203d9f1435c34fcd3f7bda89d068.tar.xz openssl-c626fda8a66a203d9f1435c34fcd3f7bda89d068.zip | |
Check returns of various sk_*_push functions
Check returns of sk_POLICY_MAPPING_push, sk_GENERAL_NAME_push,
sk_ACCESS_DESCRIPTION_push, sk_X509_push, sk_X509_NAME_push,
sk_OPENSSL_CSTRING_push, sk_SCT_push, sk_DIST_POINT_push,
sk_OSSL_CMP_CRLSTATUS_push, sk_ASN1_UTF8STRING_push and
sk_ASN1_OBJECT_push and handle appropriately.
Reviewed-by: Saša Nedvědický <sashan@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/26240)
Diffstat (limited to '')
| -rw-r--r-- | test/v3nametest.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/v3nametest.c b/test/v3nametest.c index 3609eba045..73767abf46 100644 --- a/test/v3nametest.c +++ b/test/v3nametest.c @@ -157,7 +157,8 @@ static int set_altname(X509 *crt, ...) default: abort(); } - sk_GENERAL_NAME_push(gens, gen); + if (!sk_GENERAL_NAME_push(gens, gen)) + goto out; gen = NULL; } if (!X509_add1_ext_i2d(crt, NID_subject_alt_name, gens, 0, 0)) |