diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2018-02-21 15:24:18 +0100 |
|---|---|---|
| committer | Ben Noordhuis <info@bnoordhuis.nl> | 2018-02-21 15:24:18 +0100 |
| commit | d4024815b7d55005c3ec25c74cf647f7eba1d5cc (patch) | |
| tree | b86653a53494704be52a30b175886bf9ed054cd4 /src/tls_wrap.cc | |
| parent | 992703f2b50f6b3483e9b930737d177b9e01256d (diff) | |
| download | android-node-v8-d4024815b7d55005c3ec25c74cf647f7eba1d5cc.tar.gz android-node-v8-d4024815b7d55005c3ec25c74cf647f7eba1d5cc.tar.bz2 android-node-v8-d4024815b7d55005c3ec25c74cf647f7eba1d5cc.zip | |
src: remove unnecessary Reset() calls
The previous commit made persistent handles auto-reset on destruction.
This commit removes the Reset() calls that are now no longer necessary.
PR-URL: https://github.com/nodejs/node/pull/18656
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Diffstat (limited to 'src/tls_wrap.cc')
| -rw-r--r-- | src/tls_wrap.cc | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc index a889279f6c..7e0962ee9e 100644 --- a/src/tls_wrap.cc +++ b/src/tls_wrap.cc @@ -86,12 +86,7 @@ TLSWrap::TLSWrap(Environment* env, TLSWrap::~TLSWrap() { enc_in_ = nullptr; enc_out_ = nullptr; - sc_ = nullptr; - -#ifdef SSL_CTRL_SET_TLSEXT_SERVERNAME_CB - sni_context_.Reset(); -#endif // SSL_CTRL_SET_TLSEXT_SERVERNAME_CB } @@ -835,7 +830,6 @@ int TLSWrap::SelectSNIContextCallback(SSL* s, int* ad, void* arg) { return SSL_TLSEXT_ERR_NOACK; } - p->sni_context_.Reset(); p->sni_context_.Reset(env->isolate(), ctx); SecureContext* sc = Unwrap<SecureContext>(ctx.As<Object>()); |