Fixing Coverity warnings introduced in 24de499877ca.

CIDs: 1465870, 1465871, 1465872.

diff --git a/src/njs_fs.c b/src/njs_fs.c
index 51a3349ddd6ee1bedebc101c12f6a7e0e1297fb4..648e5461cf8bab7ca84cde421e85a87cff32c600 100644 (file)
--- a/src/njs_fs.c
+++ b/src/njs_fs.c
@@ -1292,6 +1292,12 @@ njs_ftw(char *path, njs_file_tree_walk_cb_t cb, int fd_limit,
         return 0;
     }
 
+    for (h = parent; h != NULL; h = h->chain) {
+        if (h->dev == st.st_dev && h->ino == st.st_ino) {
+            return 0;
+        }
+    }
+
     len = njs_strlen(path);
     base = len && (path[len - 1] == '/') ? len - 1 : len;
 
@@ -1305,25 +1311,24 @@ njs_ftw(char *path, njs_file_tree_walk_cb_t cb, int fd_limit,
     if (type == NJS_FTW_D || type == NJS_FTW_DP) {
         dfd = open(path, O_RDONLY);
         err = errno;
-        if (dfd < 0 && err == EACCES) {
-            type = NJS_FTW_DNR;
-        }
+        if (dfd < 0) {
+            if (err == EACCES) {
+                type = NJS_FTW_DNR;
+            }
 
-        if (fd_limit == 0) {
-            close(dfd);
+        } else if (fd_limit == 0) {
+            (void) close(dfd);
         }
     }
 
     if (!(flags & NJS_FTW_DEPTH)) {
         ret = cb(path, &st, type);
         if (njs_slow_path(ret != 0)) {
-            return ret;
-        }
-    }
+            if (dfd >= 0) {
+                (void) close(dfd);
+            }
 
-    for (h = parent; h != NULL; h = h->chain) {
-        if (h->dev == st.st_dev && h->ino == st.st_ino) {
-            return 0;
+            return ret;
         }
     }
 
@@ -1335,7 +1340,7 @@ njs_ftw(char *path, njs_file_tree_walk_cb_t cb, int fd_limit,
 
         d = fdopendir(dfd);
         if (njs_slow_path(d == NULL)) {
-            close(dfd);
+            (void) close(dfd);
             return -1;
         }