From 5707ef5fe321402967c7c79cc581ffda73eba0ea Mon Sep 17 00:00:00 2001 From: Trevor Norris Date: Tue, 24 Nov 2015 10:52:28 -0700 Subject: [PATCH] node: fix leaking Context handle The call to node::Environment::GetCurrent(Isolate*) makes the call to v8::Isolate::GetCurrentContext(). Doing so creates a new handle that bubbled to the v8::SealHandleScope(). PR-URL: https://github.com/nodejs/node/pull/3945 Reviewed-By: Fedor Indutny Reviewed-By: James Snell --- src/node.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/node.cc b/src/node.cc index 200f67bdcbdf73..a2698f12bccbe0 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3163,6 +3163,7 @@ static void EnableDebug(Environment* env) { // Called from the main thread. static void DispatchDebugMessagesAsyncCallback(uv_async_t* handle) { + HandleScope scope(node_isolate); if (debugger_running == false) { fprintf(stderr, "Starting debugger agent.\n");