Skip to content

Commit 260d777

Browse files
refackrvagg
authored andcommitted
src: fix resource leak in node::fs::FileHandle
PR-URL: #23793 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
1 parent c0a9a83 commit 260d777

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/node_file.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -406,7 +406,7 @@ class FileHandle : public AsyncWrap, public StreamBase {
406406
ref_.Reset(env->isolate(), ref);
407407
}
408408

409-
~CloseReq() {
409+
~CloseReq() override {
410410
uv_fs_req_cleanup(req());
411411
promise_.Reset();
412412
ref_.Reset();

0 commit comments

Comments
 (0)