diff options
author | Dr. Stephen Henson <steve@openssl.org> | 2010-02-16 14:30:19 +0000 |
---|---|---|
committer | Dr. Stephen Henson <steve@openssl.org> | 2010-02-16 14:30:19 +0000 |
commit | 03fd7f27dbad5cb8d1fa5852de89b8d8c2565430 (patch) | |
tree | 1b1d4a87ca280c841487956f9117f204ba507462 /engines | |
parent | 45d6a15ae97fce06d50a5b33d403a44c39d89ff8 (diff) |
Submitted by: Dmitry Ivanov <vonami@gmail.com>
Don't leave dangling pointers in GOST engine if calls fail.
Diffstat (limited to 'engines')
-rw-r--r-- | engines/ccgost/gost2001.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/engines/ccgost/gost2001.c b/engines/ccgost/gost2001.c index e3354549e1..dacd82d2b9 100644 --- a/engines/ccgost/gost2001.c +++ b/engines/ccgost/gost2001.c @@ -136,12 +136,14 @@ DSA_SIG *gost2001_do_sign(const unsigned char *dgst,int dlen, EC_KEY *eckey) { GOSTerr(GOST_F_GOST2001_DO_SIGN,GOST_R_RANDOM_NUMBER_GENERATOR_FAILED); DSA_SIG_free(newsig); + newsig = NULL; goto err; } if (!EC_POINT_mul(group,C,k,NULL,NULL,ctx)) { GOSTerr(GOST_F_GOST2001_DO_SIGN,ERR_R_EC_LIB); DSA_SIG_free(newsig); + newsig = NULL; goto err; } if (!X) X=BN_CTX_get(ctx); @@ -149,6 +151,7 @@ DSA_SIG *gost2001_do_sign(const unsigned char *dgst,int dlen, EC_KEY *eckey) { GOSTerr(GOST_F_GOST2001_DO_SIGN,ERR_R_EC_LIB); DSA_SIG_free(newsig); + newsig = NULL; goto err; } if (!r) r=BN_CTX_get(ctx); |