diff options
author | Yuchi <yt8mn@virginia.edu> | 2017-02-05 19:33:47 -0500 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2017-02-14 10:27:08 +0000 |
commit | b37fce59cb7c48776361ac8cb8a808793016eefd (patch) | |
tree | 6eda75be64ab79bdc1e90e3ec73e8ae9a05e7d4d /crypto/ec/ec_ameth.c | |
parent | 955286c9f38c11b8be719d632fa9267eb13467f8 (diff) |
mem leak on error path and error propagation fix
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/2559)
(cherry picked from commit e0670973d5c0b837eb5a9f1670e47107f466fbc7)
Diffstat (limited to 'crypto/ec/ec_ameth.c')
-rw-r--r-- | crypto/ec/ec_ameth.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/crypto/ec/ec_ameth.c b/crypto/ec/ec_ameth.c index 66437e0da5..fa5bd0318c 100644 --- a/crypto/ec/ec_ameth.c +++ b/crypto/ec/ec_ameth.c @@ -254,8 +254,10 @@ static int eckey_priv_encode(PKCS8_PRIV_KEY_INFO *p8, const EVP_PKEY *pkey) } if (!PKCS8_pkey_set0(p8, OBJ_nid2obj(NID_X9_62_id_ecPublicKey), 0, - ptype, pval, ep, eplen)) + ptype, pval, ep, eplen)) { + OPENSSL_free(ep); return 0; + } return 1; } |