nodejs
diff --git a/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+9-3Lines changed: 9 additions & 3 deletions b/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+9-3Lines changed: 9 additions & 3 deletions
diff --git a/Collapse file
‎src/node_crypto.h‎
Copy file name to clipboardExpand all lines: src/node_crypto.h
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎src/node_crypto.h‎
Copy file name to clipboardExpand all lines: src/node_crypto.h
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎src/tls_wrap.cc‎
Copy file name to clipboardExpand all lines: src/tls_wrap.cc
+1-2Lines changed: 1 addition & 2 deletions b/Collapse file
‎src/tls_wrap.cc‎
Copy file name to clipboardExpand all lines: src/tls_wrap.cc
+1-2Lines changed: 1 addition & 2 deletions
@@ -2991,9 +2991,15 @@ void SSLWrap<Base>::CertCbDone(const FunctionCallbackInfo<Value>& args) {
     goto fire_cb;
 
   if (cons->HasInstance(ctx)) {
-    SecureContext* sc;
-    ASSIGN_OR_RETURN_UNWRAP(&sc, ctx.As<Object>());
-    w->sni_context_.Reset(env->isolate(), ctx);
+    SecureContext* sc = Unwrap<SecureContext>(ctx.As<Object>());
+    CHECK_NOT_NULL(sc);
+    // XXX: There is a method w->SetSNIContext(sc), and you might think that
+    // it makes sense to call that here and make setting w->sni_context_ part
+    // of it. In fact, that passes the test suite, although SetSNIContext()
+    // performs a lot more operations.
+    // If anybody is familiar enough with the TLS code to know whether it makes
+    // sense, please do so or document why it doesn't.
+    w->sni_context_ = BaseObjectPtr<SecureContext>(sc);
 
     int rv;
 
 
@@ -310,7 +310,7 @@ class SSLWrap {
   ClientHelloParser hello_parser_;
 
   v8::Global<v8::ArrayBufferView> ocsp_response_;
-  v8::Global<v8::Value> sni_context_;
+  BaseObjectPtr<SecureContext> sni_context_;
 
   friend class SecureContext;
 };
 
@@ -1065,10 +1065,9 @@ int TLSWrap::SelectSNIContextCallback(SSL* s, int* ad, void* arg) {
     return SSL_TLSEXT_ERR_NOACK;
   }
 
-  p->sni_context_.Reset(env->isolate(), ctx);
-
   SecureContext* sc = Unwrap<SecureContext>(ctx.As<Object>());
   CHECK_NOT_NULL(sc);
+  p->sni_context_ = BaseObjectPtr<SecureContext>(sc);
   p->SetSNIContext(sc);
   return SSL_TLSEXT_ERR_OK;
 }
Original file line number	Diff line number	Diff line change
`@@ -1065,10 +1065,9 @@ int TLSWrap::SelectSNIContextCallback(SSL* s, int* ad, void* arg) {`
`1065`	`1065`	`return SSL_TLSEXT_ERR_NOACK;`
`1066`	`1066`	`}`
`1067`	`1067`
`1068`		`- p->sni_context_.Reset(env->isolate(), ctx);`
`1069`		`-`
`1070`	`1068`	`SecureContext* sc = Unwrap<SecureContext>(ctx.As<Object>());`
`1071`	`1069`	`CHECK_NOT_NULL(sc);`
	`1070`	`+ p->sni_context_ = BaseObjectPtr<SecureContext>(sc);`
`1072`	`1071`	`p->SetSNIContext(sc);`
`1073`	`1072`	`return SSL_TLSEXT_ERR_OK;`
`1074`	`1073`	`}`