crypto: fix return Local Handle w/o scope.Close()

A new String was being created and returned, but was not sent through
the scope.Close(), which caused it to be cleaned up before being
returned.

diff --git a/src/node_crypto.cc b/src/node_crypto.cc
index 4a02812..667f1e3 100644 (file)
--- a/src/node_crypto.cc
+++ b/src/node_crypto.cc
@@ -1962,7 +1962,8 @@ Handle<Value> Connection::GetNegotiatedProto(const Arguments& args) {
       return False();
     }
 
-    return String::New((const char*) npn_proto, npn_proto_len);
+    return scope.Close(String::New(reinterpret_cast<const char*>(npn_proto),
+                                   npn_proto_len));
   } else {
     return ss->selectedNPNProto_;
   }