Skip to content

Commit 3a48f00

Browse files
trevnorrisMyles Borins
authored and
Myles Borins
committed
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: #3945 Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: James Snell <[email protected]>
1 parent a79baf0 commit 3a48f00

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/node.cc

+1
Original file line numberDiff line numberDiff line change
@@ -3177,6 +3177,7 @@ static void EnableDebug(Environment* env) {
31773177

31783178
// Called from the main thread.
31793179
static void DispatchDebugMessagesAsyncCallback(uv_async_t* handle) {
3180+
HandleScope scope(node_isolate);
31803181
if (debugger_running == false) {
31813182
fprintf(stderr, "Starting debugger agent.\n");
31823183

0 commit comments

Comments
 (0)