src: explicitly allow JS in ReadHostObject

hashseed · refack · commit e4df1f79aba6 · 2019-01-10T10:03:59.000-05:00
PR-URL: nodejs#23423 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Myles Borins <myles.borins@gmail.com>
diff --git a/src/node_serdes.cc b/src/node_serdes.cc
@@ -296,6 +296,7 @@ MaybeLocal<Object> DeserializerContext::ReadHostObject(Isolate* isolate) {
     return ValueDeserializer::Delegate::ReadHostObject(isolate);
   }
 
+  Isolate::AllowJavascriptExecutionScope allow_js(isolate);
   MaybeLocal<Value> ret =
       read_host_object.As<Function>()->Call(env()->context(),
                                             object(),

Original file line number	Diff line number	Diff line change
`@@ -296,6 +296,7 @@ MaybeLocal<Object> DeserializerContext::ReadHostObject(Isolate* isolate) {`
`296`	`296`	`return ValueDeserializer::Delegate::ReadHostObject(isolate);`
`297`	`297`	`}`
`298`	`298`
	`299`	`+ Isolate::AllowJavascriptExecutionScope allow_js(isolate);`
`299`	`300`	`MaybeLocal<Value> ret =`
`300`	`301`	`read_host_object.As<Function>()->Call(env()->context(),`
`301`	`302`	`object(),`