vm: remove unnecessary access checks

No reason to install access checks if they're always going to return true.
nodejs · Jun 1, 2015 · d119272 · d119272
1 parent d3a01fd
commit d119272
Showing 1 changed file with 0 additions and 18 deletions.
diff --git a/src/node_contextify.cc b/src/node_contextify.cc
@@ -208,8 +208,6 @@ class ContextifyContext {
                                              GlobalPropertyDeleterCallback,
                                              GlobalPropertyEnumeratorCallback,
                                              CreateDataWrapper(env));
-    object_template->SetAccessCheckCallbacks(GlobalPropertyNamedAccessCheck,
-                                             GlobalPropertyIndexedAccessCheck);
 
     Local<Context> ctx = Context::New(env->isolate(), nullptr, object_template);
     if (!ctx.IsEmpty())
@@ -343,22 +341,6 @@ class ContextifyContext {
   }
 
 
-  static bool GlobalPropertyNamedAccessCheck(Local<Object> host,
-                                             Local<Value> key,
-                                             AccessType type,
-                                             Local<Value> data) {
-    return true;
-  }
-
-
-  static bool GlobalPropertyIndexedAccessCheck(Local<Object> host,
-                                               uint32_t key,
-                                               AccessType type,
-                                               Local<Value> data) {
-    return true;
-  }
-
-
   static void GlobalPropertyGetterCallback(
       Local<String> property,
       const PropertyCallbackInfo<Value>& args) {