Merge pull request #2506 from libgit2/rb/ignore-pipes-etc

Don't report status on named pipes

1 files changed, 12 insertions, 11 deletions

diff --git a/src/path.c b/src/path.c
index 4837b01f9..77f8d8858 100644
--- a/src/path.c
+++ b/src/path.c
@@ -1110,28 +1110,29 @@ int git_path_dirload_with_stat(
 
 		if ((error = git_buf_joinpath(&full, full.ptr, ps->path)) < 0 ||
 			(error = git_path_lstat(full.ptr, &ps->st)) < 0) {
+
 			if (error == GIT_ENOTFOUND) {
-				giterr_clear();
-				error = 0;
+				/* file was removed between readdir and lstat */
 				git_vector_remove(contents, i--);
-				continue;
-			}
-			/* Treat the file as unreadable if we get any other error */
-			if (error != 0) {
-				giterr_clear();
-				error = 0;
+			} else {
+				/* Treat the file as unreadable if we get any other error */
 				memset(&ps->st, 0, sizeof(ps->st));
 				ps->st.st_mode = GIT_FILEMODE_UNREADABLE;
-				continue;
 			}
-			
-			break;
+
+			giterr_clear();
+			error = 0;
+			continue;
 		}
 
 		if (S_ISDIR(ps->st.st_mode)) {
 			ps->path[ps->path_len++] = '/';
 			ps->path[ps->path_len] = '\0';
 		}
+		else if (!S_ISREG(ps->st.st_mode) && !S_ISLNK(ps->st.st_mode)) {
+			/* skip everything but dirs, plain files, and symlinks */
+			git_vector_remove(contents, i--);
+		}
 	}
 
 	/* sort now that directory suffix is added */