diff options
| author | Dr. Stephen Henson <steve@openssl.org> | 2014-06-27 03:21:10 +0100 |
|---|---|---|
| committer | Dr. Stephen Henson <steve@openssl.org> | 2014-06-27 14:35:07 +0100 |
| commit | d2aea038297e0c64ca66e6844cbb37377365885e (patch) | |
| tree | 13e11f5b4a91e0dc5316bb7c6fb7203c313be84b /apps/crl2p7.c | |
| parent | 3b3b69ab25fbaba80a843dedec5ae9733fefca43 (diff) | |
Memory leak and NULL dereference fixes.
PR#3403
Diffstat (limited to 'apps/crl2p7.c')
| -rw-r--r-- | apps/crl2p7.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/apps/crl2p7.c b/apps/crl2p7.c index bbc83774db..42c6886b83 100644 --- a/apps/crl2p7.c +++ b/apps/crl2p7.c @@ -141,7 +141,13 @@ int MAIN(int argc, char **argv) { if (--argc < 1) goto bad; if(!certflst) certflst = sk_OPENSSL_STRING_new_null(); - sk_OPENSSL_STRING_push(certflst,*(++argv)); + if (!certflst) + goto end; + if (!sk_OPENSSL_STRING_push(certflst,*(++argv))) + { + sk_OPENSSL_STRING_free(certflst); + goto end; + } } else { |