diff options
author | Bernd Edlinger <bernd.edlinger@hotmail.de> | 2023-12-03 11:24:18 +0100 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2023-12-12 19:45:50 +0100 |
commit | dfe320772d2750b40f9fafe30639fa72f4c823eb (patch) | |
tree | 6602a45643e6a1949ed90687c77a0129df9f5d77 /apps | |
parent | ed5f9ce63e98da2e7fddd55040c8e9e03f3af975 (diff) |
Fix a possible memleak in cms_main
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22918)
(cherry picked from commit 3457a550c64ab8009c7cd0175675ac140cab33c2)
Diffstat (limited to 'apps')
-rw-r--r-- | apps/cms.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/apps/cms.c b/apps/cms.c index 12095b9641..3994cb0fcd 100644 --- a/apps/cms.c +++ b/apps/cms.c @@ -620,7 +620,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(); @@ -831,7 +832,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; } } |