diff options
author | inoguchi <> | 2022-01-16 07:11:49 +0000 |
---|---|---|
committer | inoguchi <> | 2022-01-16 07:11:49 +0000 |
commit | b134fe0513c26b66216024dfb60a5a320355ab74 (patch) | |
tree | 36baabb71318914d0257e06c5397547c7999493e /src | |
parent | 11a0f7c47c759a6a24cb1a97b23160532ee09267 (diff) | |
download | openbsd-b134fe0513c26b66216024dfb60a5a320355ab74.tar.gz openbsd-b134fe0513c26b66216024dfb60a5a320355ab74.tar.bz2 openbsd-b134fe0513c26b66216024dfb60a5a320355ab74.zip |
Avoid memory leak in error path with openssl(1) cms
CID 345314 345320
ok tb@
Diffstat (limited to 'src')
-rw-r--r-- | src/usr.bin/openssl/cms.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/usr.bin/openssl/cms.c b/src/usr.bin/openssl/cms.c index 8d066e5a55..5b876941fb 100644 --- a/src/usr.bin/openssl/cms.c +++ b/src/usr.bin/openssl/cms.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: cms.c,v 1.28 2022/01/08 06:05:39 inoguchi Exp $ */ | 1 | /* $OpenBSD: cms.c,v 1.29 2022/01/16 07:11:49 inoguchi Exp $ */ |
2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL | 2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL |
3 | * project. | 3 | * project. |
4 | */ | 4 | */ |
@@ -1395,6 +1395,7 @@ cms_main(int argc, char **argv) | |||
1395 | BIO_printf(bio_err, | 1395 | BIO_printf(bio_err, |
1396 | "Error writing certs to %s\n", | 1396 | "Error writing certs to %s\n", |
1397 | cms_config.certsoutfile); | 1397 | cms_config.certsoutfile); |
1398 | sk_X509_pop_free(allcerts, X509_free); | ||
1398 | ret = 5; | 1399 | ret = 5; |
1399 | goto end; | 1400 | goto end; |
1400 | } | 1401 | } |
@@ -1695,6 +1696,7 @@ cms_main(int argc, char **argv) | |||
1695 | BIO_printf(bio_err, | 1696 | BIO_printf(bio_err, |
1696 | "Error writing signers to %s\n", | 1697 | "Error writing signers to %s\n", |
1697 | cms_config.signerfile); | 1698 | cms_config.signerfile); |
1699 | sk_X509_free(signers); | ||
1698 | ret = 5; | 1700 | ret = 5; |
1699 | goto end; | 1701 | goto end; |
1700 | } | 1702 | } |