src: remove duplicate HMAC_Init calls

bnoordhuis · bnoordhuis · commit d378b8fbe891 · 2016-07-11T23:56:34.000+02:00
PR-URL: nodejs#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>
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -3330,13 +3330,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;

Original file line number	Diff line number	Diff line change
`@@ -3330,13 +3330,10 @@ void Hmac::HmacInit(const char* hash_type, const char* key, int key_len) {`
`3330`	`3330`	`return env()->ThrowError("Unknown message digest");`
`3331`	`3331`	`}`
`3332`	`3332`	`HMAC_CTX_init(&ctx_);`
`3333`		`- int result = 0;`
`3334`	`3333`	`if (key_len == 0) {`
`3335`		`- result = HMAC_Init(&ctx_, "", 0, md);`
`3336`		`- } else {`
`3337`		`- result = HMAC_Init(&ctx_, key, key_len, md);`
	`3334`	`+ key = "";`
`3338`	`3335`	`}`
`3339`		`- if (!result) {`
	`3336`	`+ if (!HMAC_Init(&ctx_, key, key_len, md)) {`
`3340`	`3337`	`return ThrowCryptoError(env(), ERR_get_error());`
`3341`	`3338`	`}`
`3342`	`3339`	`initialised_ = true;`