fs: return errno and take fs_req_wrap in SyncCall

PR-URL: nodejs#17914
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

src/node_file.cc

@@ -375,11 +375,10 @@ inline FSReqWrap* AsyncCall(Environment* env,
 // the error number and the syscall in the context instead of
 // creating an error in the C++ land.
 template <typename Func, typename... Args>
-inline void SyncCall(Environment* env, Local<Value> ctx,
+inline int SyncCall(Environment* env, Local<Value> ctx, fs_req_wrap* req_wrap,
     const char* syscall, Func fn, Args... args) {
-  fs_req_wrap req_wrap;
   env->PrintSyncTrace();
-  int err = fn(env->event_loop(), &req_wrap.req, args..., nullptr);
+  int err = fn(env->event_loop(), &(req_wrap->req), args..., nullptr);
   if (err < 0) {
     Local<Context> context = env->context();
     Local<Object> ctx_obj = ctx->ToObject(context).ToLocalChecked();
@@ -391,6 +390,7 @@ inline void SyncCall(Environment* env, Local<Value> ctx,
              env->syscall_string(),
              OneByteString(isolate, syscall)).FromJust();
   }
+  return err;
 }
 
 #define SYNC_DEST_CALL(func, path, dest, ...)                                 \
@@ -426,7 +426,8 @@ void Access(const FunctionCallbackInfo<Value>& args) {
     AsyncCall(env, args, "access", UTF8, AfterNoArgs,
               uv_fs_access, *path, mode);
   } else {  // access(path, mode, undefined, ctx)
-    SyncCall(env, args[3], "access", uv_fs_access, *path, mode);
+    fs_req_wrap req_wrap;
+    SyncCall(env, args[3], &req_wrap, "access", uv_fs_access, *path, mode);
   }
 }
 
@@ -446,7 +447,8 @@ void Close(const FunctionCallbackInfo<Value>& args) {
     AsyncCall(env, args, "close", UTF8, AfterNoArgs,
               uv_fs_close, fd);
   } else {  // close(fd, undefined, ctx)
-    SyncCall(env, args[2], "close", uv_fs_close, fd);
+    fs_req_wrap req_wrap;
+    SyncCall(env, args[2], &req_wrap, "close", uv_fs_close, fd);
   }
 }