Skip to content

Commit

Permalink
src: remove duplicate HMAC_Init calls
Browse files Browse the repository at this point in the history
PR-URL: #7374
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information
bnoordhuis authored and Myles Borins committed Jul 14, 2016
1 parent 04d968f commit bf8ac55
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/node_crypto.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3326,13 +3326,10 @@ void Hmac::HmacInit(const char* hash_type, const char* key, int key_len) {
return env()->ThrowError("Unknown message digest");
}
HMAC_CTX_init(&ctx_);
int result = 0;
if (key_len == 0) {
result = HMAC_Init(&ctx_, "", 0, md);
} else {
result = HMAC_Init(&ctx_, key, key_len, md);
key = "";
}
if (!result) {
if (!HMAC_Init(&ctx_, key, key_len, md)) {
return ThrowCryptoError(env(), ERR_get_error());
}
initialised_ = true;
Expand Down

0 comments on commit bf8ac55

Please sign in to comment.