bind hooks to null

nodejs#35524 did this (not sure whether it's necessary)
JakobJingleheimer · Apr 18, 2021 · d921030 · d921030
1 parent e7093f7
commit d921030
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/internal/modules/esm/loader.js b/lib/internal/modules/esm/loader.js
@@ -145,18 +145,20 @@ class ESMLoader {
 
       if (globalPreloader) ArrayPrototypePush(
         this.#globalPreloaders,
-        globalPreloader,
+        FunctionPrototypeBind(globalPreloader, null), // [1]
       );
       if (resolver) ArrayPrototypePush(
         this.#resolvers,
-        resolver,
+        FunctionPrototypeBind(resolver, null), // [1]
       );
       if (loader) ArrayPrototypePush(
         this.#loaders,
-        loader,
+        FunctionPrototypeBind(loader, null), // [1]
       );
     }
 
+    // [1] ensure hook function is not bound to ESMLoader instance
+
     this.preload();
   }