Fix fs.scandir (introduced by #122) (#124)

The check should be after the allocation, where it is. It was misplaced and also duplicated the existing one. libtuv-DCO-1.0-Signed-off-by: Robert Sipka [email protected]
Samsung · Aug 24, 2018 · 0bfc04d · 0bfc04d
1 parent 46f0242
commit 0bfc04d
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/src/unix/fs.c b/src/unix/fs.c
@@ -407,13 +407,7 @@ static ssize_t uv__fs_scandir(uv_fs_t* req) {
       cnt++;
   }
 
-  if (cnt > 0 && dents == NULL) {
-      closedir(dir);
-      cnt = -1;
-      goto error;
-  }
-
-  /* Allcoate memory for the directory entries. */
+  /* Allocate memory for the directory entries. */
   dents = (uv__dirent_t**) malloc(sizeof (uv__dirent_t*) * cnt);
 
   if (cnt > 0 && dents == NULL) {