src: fix SetClientCertEngine() nullptr dereference

bnoordhuis · bnoordhuis · commit 6c76140a7a61 · 2017-11-15T12:15:37.000+01:00
Introduced in commit 6ee985f ("tls: implement clientCertEngine option") which was merged November 11. PR-URL: nodejs#16965 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -1341,7 +1341,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;
 
