src: fix SetClientCertEngine() nullptr dereference

Introduced in commit 6ee985f ("tls: implement clientCertEngine option") which was merged November 11. PR-URL: #16965 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Dec 11, 2017 · 7ac760b · 7ac760b
1 parent f6ec5fa
commit 7ac760b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -1348,7 +1348,8 @@ void SecureContext::SetClientCertEngine(
   CHECK_EQ(args.Length(), 1);
   CHECK(args[0]->IsString());
 
-  SecureContext* sc = Unwrap<SecureContext>(args.This());
+  SecureContext* sc;
+  ASSIGN_OR_RETURN_UNWRAP(&sc, args.Holder());
 
   MarkPopErrorOnReturn mark_pop_error_on_return;