diff options
| author | cjihrig <cjihrig@gmail.com> | 2019-03-28 16:41:37 -0400 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2019-03-30 20:50:18 -0400 |
| commit | bb98f271815f6f1357c3c7178a3367d439c44029 (patch) | |
| tree | 4c18cef9e7be2c0a1a8764869737f7352620f403 /src | |
| parent | 9fbf0c60b583dae3d34598352c3c7614118cd035 (diff) | |
| download | node-new-bb98f271815f6f1357c3c7178a3367d439c44029.tar.gz | |
src: check uv_fs_close() return value
Coverity was complaining about not checking the return value.
PR-URL: https://github.com/nodejs/node/pull/26967
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'src')
| -rw-r--r-- | src/module_wrap.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/module_wrap.cc b/src/module_wrap.cc index 622deed45f..b11a4fe898 100644 --- a/src/module_wrap.cc +++ b/src/module_wrap.cc @@ -494,7 +494,7 @@ inline Maybe<uv_file> OpenDescriptor(const std::string& path) { inline void CloseDescriptor(uv_file fd) { uv_fs_t fs_req; - uv_fs_close(nullptr, &fs_req, fd, nullptr); + CHECK_EQ(0, uv_fs_close(nullptr, &fs_req, fd, nullptr)); uv_fs_req_cleanup(&fs_req); } |