diff options
author | Matt Caswell <matt@openssl.org> | 2015-06-08 09:29:56 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2015-06-10 11:03:20 +0100 |
commit | e43a13c807e42688c72c4f3d001112bf0a110464 (patch) | |
tree | b81219fb8cf576bfcc326f3793660647eaf512c7 /crypto | |
parent | 68886be7e2cd395a759fcd41d2cede461b68843d (diff) |
Fix leak in HMAC error path
In the event of an error in the HMAC function, leaks can occur because the
HMAC_CTX does not get cleaned up.
Thanks to the BoringSSL project for reporting this issue.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/hmac/hmac.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/hmac/hmac.c b/crypto/hmac/hmac.c index 572393e2c7..d50fabb7c1 100644 --- a/crypto/hmac/hmac.c +++ b/crypto/hmac/hmac.c @@ -219,6 +219,7 @@ unsigned char *HMAC(const EVP_MD *evp_md, const void *key, int key_len, HMAC_CTX_cleanup(&c); return md; err: + HMAC_CTX_cleanup(&c); return NULL; } |