From 77e416b7115fd6bf8f36176f2d0aaa3369167cff Mon Sep 17 00:00:00 2001 From: Felipe Morales Date: Sun, 30 Aug 2015 18:14:26 -0300 Subject: do_path_expand: Avoid non-readable directories. #3273 Closes https://github.com/neovim/neovim/issues/3164 Closes https://github.com/neovim/neovim/issues/3194 Closes https://github.com/neovim/neovim/issues/3221 Helped-by: @splinterofchaos Helped-by: @oni-link Helped-by: @justinmk --- src/nvim/os/fs.c | 4 ++-- src/nvim/path.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/nvim/os/fs.c b/src/nvim/os/fs.c index 3789da3b17..522e49950c 100644 --- a/src/nvim/os/fs.c +++ b/src/nvim/os/fs.c @@ -421,10 +421,10 @@ bool os_scandir(Directory *dir, const char *path) FUNC_ATTR_NONNULL_ALL { int r = uv_fs_scandir(&fs_loop, &dir->request, path, 0, NULL); - if (r <= 0) { + if (r < 0) { os_closedir(dir); } - return r > 0; + return r >= 0; } /// Increments the directory pointer. diff --git a/src/nvim/path.c b/src/nvim/path.c index f68cb46f33..a3b5bb7490 100644 --- a/src/nvim/path.c +++ b/src/nvim/path.c @@ -612,8 +612,8 @@ static size_t do_path_expand(garray_T *gap, const char_u *path, *s = NUL; Directory dir; - if (os_scandir(&dir, *buf == NUL ? "." : (char *)buf) - || os_isdir(*buf == NUL ? (char_u *)"." : (char_u *)buf)) { + char *dirpath = (*buf == NUL ? "." : (char *)buf); + if (os_file_is_readable(dirpath) && os_scandir(&dir, dirpath)) { // Find all matching entries. char_u *name; scandir_next_with_dots(NULL /* initialize */); -- cgit