src: check uv_fs_close() return value

cjihrig · cjihrig · commit bb98f271815f · 2019-03-30T20:50:18.000-04:00
Coverity was complaining about not checking the return value. PR-URL: nodejs#26967 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
diff --git 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);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -494,7 +494,7 @@ inline Maybe<uv_file> OpenDescriptor(const std::string& path) {`
`494`	`494`
`495`	`495`	`inline void CloseDescriptor(uv_file fd) {`
`496`	`496`	`uv_fs_t fs_req;`
`497`		`- uv_fs_close(nullptr, &fs_req, fd, nullptr);`
	`497`	`+ CHECK_EQ(0, uv_fs_close(nullptr, &fs_req, fd, nullptr));`
`498`	`498`	`uv_fs_req_cleanup(&fs_req);`
`499`	`499`	`}`
`500`	`500`