fs: add missing call to uv_fs_req_cleanup

Justin-Nietzel · RafaelGSS · commit 52e4967f4509 · 2025-05-02T17:06:19.000-03:00
Always call uv_fs_req_cleanup after calling uv_fs_open instead of just when uv_fs_open returns a negative result. I referenced ReadFileSync from node:js2c when making this change. https://github.com/bnoordhuis made the same suggestion based on the PR #49691. Fixes: #57800 PR-URL: #57811 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/src/node_file.cc b/src/node_file.cc
@@ -2711,10 +2711,10 @@ static void ReadFileUtf8(const FunctionCallbackInfo<Value>& args) {
     FS_SYNC_TRACE_END(open);
     if (req.result < 0) {
       uv_fs_req_cleanup(&req);
-      // req will be cleaned up by scope leave.
       return env->ThrowUVException(
           static_cast<int>(req.result), "open", nullptr, path.out());
     }
+    uv_fs_req_cleanup(&req);
   }
 
   auto defer_close = OnScopeLeave([file, is_fd, &req]() {

Original file line number	Diff line number	Diff line change
`@@ -2711,10 +2711,10 @@ static void ReadFileUtf8(const FunctionCallbackInfo<Value>& args) {`
`2711`	`2711`	`FS_SYNC_TRACE_END(open);`
`2712`	`2712`	`if (req.result < 0) {`
`2713`	`2713`	`uv_fs_req_cleanup(&req);`
`2714`		`- // req will be cleaned up by scope leave.`
`2715`	`2714`	`return env->ThrowUVException(`
`2716`	`2715`	`static_cast<int>(req.result), "open", nullptr, path.out());`
`2717`	`2716`	`}`
	`2717`	`+ uv_fs_req_cleanup(&req);`
`2718`	`2718`	`}`
`2719`	`2719`
`2720`	`2720`	`auto defer_close = OnScopeLeave([file, is_fd, &req]() {`