Commit 7ac760b
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 <cjihrig@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent f6ec5fa commit 7ac760b
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1348,7 +1348,8 @@ void SecureContext::SetClientCertEngine( | |||
| 1348 | 1348 | CHECK_EQ(args.Length(), 1); | |
| 1349 | 1349 | CHECK(args[0]->IsString()); | |
| 1350 | 1350 | ||
| 1351 | - SecureContext* sc = Unwrap<SecureContext>(args.This()); | ||
| 1351 | + SecureContext* sc; | ||
| 1352 | + ASSIGN_OR_RETURN_UNWRAP(&sc, args.Holder()); | ||
| 1352 | 1353 | ||
| 1353 | 1354 | MarkPopErrorOnReturn mark_pop_error_on_return; | |
| 1354 | 1355 | ||
0 commit comments