diff options
| author | Rich Salz <rsalz@openssl.org> | 2017-09-05 17:21:38 -0400 |
|---|---|---|
| committer | Rich Salz <rsalz@openssl.org> | 2017-09-06 10:01:34 -0400 |
| commit | 7dd13abe0ccdbf184303049087eea9a057b94d4b (patch) | |
| tree | 92824d3ce79de4fb6442a0952638c34287dd290d /crypto/pkcs12 | |
| parent | 90507fbb5c880e2903be2321f6054b3cebebda4c (diff) | |
Add checks for alloc failing.
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/4341)
(cherry picked from commit d3c3dfc5778ab2cca0d25c5959c8b814a334addb)
Diffstat (limited to 'crypto/pkcs12')
| -rw-r--r-- | crypto/pkcs12/p12_mutl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crypto/pkcs12/p12_mutl.c b/crypto/pkcs12/p12_mutl.c index 899a12e134..a9e22026c3 100644 --- a/crypto/pkcs12/p12_mutl.c +++ b/crypto/pkcs12/p12_mutl.c @@ -124,8 +124,8 @@ static int pkcs12_gen_mac(PKCS12 *p12, const char *pass, int passlen, PKCS12err(PKCS12_F_PKCS12_GEN_MAC, PKCS12_R_KEY_GEN_ERROR); return 0; } - hmac = HMAC_CTX_new(); - if (!HMAC_Init_ex(hmac, key, md_size, md_type, NULL) + if ((hmac = HMAC_CTX_new()) == NULL + || !HMAC_Init_ex(hmac, key, md_size, md_type, NULL) || !HMAC_Update(hmac, p12->authsafes->d.data->data, p12->authsafes->d.data->length) || !HMAC_Final(hmac, mac, maclen)) { |