diff options
author | Nils Larsch <nils@openssl.org> | 2005-05-19 22:10:40 +0000 |
---|---|---|
committer | Nils Larsch <nils@openssl.org> | 2005-05-19 22:10:40 +0000 |
commit | 7f246621b5b8802a12e3bf06ae6c0473d74daefb (patch) | |
tree | 06f7f4341fec4ea549f6894c888976eb3bb62289 | |
parent | b67d9889151bc6f09c3fb5f671c4c676dab095a4 (diff) | |
download | openssl-7f246621b5b8802a12e3bf06ae6c0473d74daefb.zip openssl-7f246621b5b8802a12e3bf06ae6c0473d74daefb.tar.gz openssl-7f246621b5b8802a12e3bf06ae6c0473d74daefb.tar.bz2 |
fix potential memory leak
Submitted by: Goetz Babin-Ebell
-rw-r--r-- | crypto/pkcs7/pk7_smime.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/crypto/pkcs7/pk7_smime.c b/crypto/pkcs7/pk7_smime.c index d37be48..b6146d7 100644 --- a/crypto/pkcs7/pk7_smime.c +++ b/crypto/pkcs7/pk7_smime.c @@ -88,6 +88,7 @@ PKCS7 *PKCS7_sign(X509 *signcert, EVP_PKEY *pkey, STACK_OF(X509) *certs, if (!(si = PKCS7_add_signature(p7,signcert,pkey,EVP_sha1()))) { PKCS7err(PKCS7_F_PKCS7_SIGN,PKCS7_R_PKCS7_ADD_SIGNATURE_ERROR); + PKCS7_free(p7); return NULL; } @@ -105,6 +106,7 @@ PKCS7 *PKCS7_sign(X509 *signcert, EVP_PKEY *pkey, STACK_OF(X509) *certs, { if(!(smcap = sk_X509_ALGOR_new_null())) { PKCS7err(PKCS7_F_PKCS7_SIGN,ERR_R_MALLOC_FAILURE); + PKCS7_free(p7); return NULL; } #ifndef OPENSSL_NO_DES @@ -130,6 +132,7 @@ PKCS7 *PKCS7_sign(X509 *signcert, EVP_PKEY *pkey, STACK_OF(X509) *certs, if (!(p7bio = PKCS7_dataInit(p7, NULL))) { PKCS7err(PKCS7_F_PKCS7_SIGN,ERR_R_MALLOC_FAILURE); + PKCS7_free(p7); return NULL; } @@ -139,10 +142,12 @@ PKCS7 *PKCS7_sign(X509 *signcert, EVP_PKEY *pkey, STACK_OF(X509) *certs, if (!PKCS7_dataFinal(p7,p7bio)) { PKCS7err(PKCS7_F_PKCS7_SIGN,PKCS7_R_PKCS7_DATASIGN); + PKCS7_free(p7); + BIO_free_all(p7bio); return NULL; } - BIO_free_all(p7bio); + BIO_free_all(p7bio); return p7; } |