fs: close file if fstat() fails in readFile()

Currently, if fstat() fails in readFile(), the callback is invoked without closing the file. This commit closes the file before calling back. Closes #7697
nodejs · May 29, 2014 · b93a51e · b93a51e
1 parent 10b6156
commit b93a51e
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/fs.js b/lib/fs.js
@@ -118,7 +118,12 @@ fs.readFile = function(path, encoding_) {
     fd = fd_;
 
     fs.fstat(fd, function(er, st) {
-      if (er) return callback(er);
+      if (er) {
+        return fs.close(fd, function() {
+          callback(er);
+        });
+      }
+
       size = st.size;
       if (size === 0) {
         // the kernel lies about many files.