From 3457a550c64ab8009c7cd0175675ac140cab33c2 Mon Sep 17 00:00:00 2001 From: Bernd Edlinger Date: Sun, 3 Dec 2023 11:24:18 +0100 Subject: [PATCH] Fix a possible memleak in cms_main Reviewed-by: Shane Lontis Reviewed-by: Tomas Mraz (Merged from https://github.com/openssl/openssl/pull/22918) --- apps/cms.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/apps/cms.c b/apps/cms.c index a16318f37c..f93c98ac92 100644 --- a/apps/cms.c +++ b/apps/cms.c @@ -628,7 +628,8 @@ int cms_main(int argc, char **argv) "recipient certificate file"); if (cert == NULL) goto end; - sk_X509_push(encerts, cert); + if (!sk_X509_push(encerts, cert)) + goto end; cert = NULL; } else { recipfile = opt_arg(); @@ -837,7 +838,8 @@ int cms_main(int argc, char **argv) "recipient certificate file"); if (cert == NULL) goto end; - sk_X509_push(encerts, cert); + if (!sk_X509_push(encerts, cert)) + goto end; cert = NULL; } }