Fix copyFile wrapper when retry hits EMFILE again

PR-URL: #209 Credit: @mbargiel Close: #209 Reviewed-by: @isaacs
isaacs · Jul 20, 2021 · 16f8da2 · 16f8da2
1 parent 89dc133
commit 16f8da2
Showing 1 changed file with 13 additions and 9 deletions.
diff --git a/graceful-fs.js b/graceful-fs.js
@@ -178,15 +178,19 @@ function patch (fs) {
       cb = flags
       flags = 0
     }
-    return fs$copyFile(src, dest, flags, function (err) {
-      if (err && (err.code === 'EMFILE' || err.code === 'ENFILE'))
-        enqueue([fs$copyFile, [src, dest, flags, cb]])
-      else {
-        if (typeof cb === 'function')
-          cb.apply(this, arguments)
-        retry()
-      }
-    })
+    return go$copyFile(src, dest, flags, cb)
+
+    function go$copyFile (src, dest, flags, cb) {
+      return fs$copyFile(src, dest, flags, function (err) {
+        if (err && (err.code === 'EMFILE' || err.code === 'ENFILE'))
+          enqueue([go$copyFile, [src, dest, flags, cb]])
+        else {
+          if (typeof cb === 'function')
+            cb.apply(this, arguments)
+          retry()
+        }
+      })
+    }
   }
 
   var fs$readdir = fs.readdir