fs: fix not found close creation context

Author: anonrig

lib/internal/fs/read/context.js

@@ -15,7 +15,7 @@ const {
 
 const { Buffer } = require('buffer');
 
-const { FSReqCallback, close, read } = internalBinding('fs');
+const binding = internalBinding('fs');
 
 const {
   AbortError,
@@ -102,11 +102,11 @@ class ReadFileContext {
       length = MathMin(kReadFileBufferLength, this.size - this.pos);
     }
 
-    const req = new FSReqCallback();
+    const req = new binding.FSReqCallback();
     req.oncomplete = readFileAfterRead;
     req.context = this;
 
-    read(this.fd, buffer, offset, length, -1, req);
+    binding.read(this.fd, buffer, offset, length, -1, req);
   }
 
   close(err) {
@@ -117,12 +117,12 @@ class ReadFileContext {
       return;
     }
 
-    const req = new FSReqCallback();
+    const req = new binding.FSReqCallback();
     req.oncomplete = readFileAfterClose;
     req.context = this;
     this.err = err;
 
-    close(this.fd, req);
+    binding.close(this.fd, req);
   }
 }
 

test/parallel/test-fs-close-fast-api.js

@@ -0,0 +1,27 @@
+'use strict';
+
+require('../common');
+
+const assert = require('assert');
+const fs = require('fs');
+const path = require('path');
+
+// This test runs `fs.readFile` which calls ReadFileContext
+// that triggers binding.close() when read operation is done.
+// The goal of this test is to not crash
+let val;
+
+// For loop is required to trigger fast API.
+for (let i = 0; i < 100_000; i++) {
+  try {
+    val = fs.readFile(path.join(__dirname, './test-fs-close.js'), (a, b) => {
+      assert.strictEqual(a, null);
+      assert.ok(b.length > 0);
+    });
+  } catch {
+    // do nothing
+  }
+
+}
+
+console.log(val);