fs: refactor to avoid unsafe array iteration

lib/fs.js

@@ -181,7 +181,7 @@ function maybeCallback(cb) {
 function makeCallback(cb) {
   validateCallback(cb);
 
-  return (...args) => cb(...args);
+  return (...args) => ReflectApply(cb, this, args);
 }
 
 // Special case of `makeCallback()` that is specific to async `*stat()` calls as

lib/internal/fs/dir.js

@@ -99,7 +99,7 @@ class Dir {
     }
 
     if (this[kDirBufferedEntries].length > 0) {
-      const [ name, type ] =
+      const { 0: name, 1: type } =
         ArrayPrototypeSplice(this[kDirBufferedEntries], 0, 2);
       if (maybeSync)
         process.nextTick(getDirent, this[kDirPath], name, type, callback);
@@ -142,7 +142,7 @@ class Dir {
     }
 
     if (this[kDirBufferedEntries].length > 0) {
-      const [ name, type ] =
+      const { 0: name, 1: type } =
           ArrayPrototypeSplice(this[kDirBufferedEntries], 0, 2);
       return getDirent(this[kDirPath], name, type);
     }
@@ -182,7 +182,7 @@ class Dir {
     }
 
     if (this[kDirOperationQueue] !== null) {
-      this[kDirOperationQueue].push(() => {
+      ArrayPrototypePush(this[kDirOperationQueue], () => {
         this.close(callback);
       });
       return;

lib/internal/fs/promises.js

@@ -19,6 +19,7 @@ const {
   PromisePrototypeFinally,
   PromisePrototypeThen,
   PromiseResolve,
+  SafeArrayIterator,
   Symbol,
   Uint8Array,
 } = primordials;
@@ -263,7 +264,7 @@ async function fsCall(fn, handle, ...args) {
 
   try {
     handle[kRef]();
-    return await fn(handle, ...args);
+    return await fn(handle, ...new SafeArrayIterator(args));
   } finally {
     handle[kUnref]();
   }

lib/internal/fs/utils.js

@@ -217,7 +217,7 @@ function join(path, name) {
     'path', ['string', 'Buffer'], path);
 }
 
-function getDirents(path, [names, types], callback) {
+function getDirents(path, { 0: names, 1: types }, callback) {
   let i;
   if (typeof callback === 'function') {
     const len = names.length;