From: Michael Haggerty Date: Sat, 18 Jan 2014 22:48:56 GMT Subject: [PATCH v3 12/17] remove_dir_recurse(): tighten condition for removing unreadable dir Message-ID: <1390085341-2553-13-git-send-email-mhagger@alum.mit.edu> In-Reply-To: <1390085341-2553-1-git-send-email-mhagger@alum.mit.edu> If opendir() fails on the top-level directory, it makes sense to try to delete it anyway--but only if the failure was due to EACCES. Signed-off-by: Michael Haggerty --- dir.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dir.c b/dir.c index d10a63f..785a83e 100644 --- a/dir.c +++ b/dir.c @@ -1511,8 +1511,11 @@ static int remove_dir_recurse(struct strbuf *path, int flag, int *kept_up) flag &= ~REMOVE_DIR_KEEP_TOPLEVEL; dir = opendir(path->buf); if (!dir) { - /* an empty dir could be removed even if it is unreadble */ - if (!keep_toplevel) + if (errno == EACCES && !keep_toplevel) + /* + * An empty dir could be removable even if it + * is unreadable: + */ return rmdir(path->buf); else return -1; -- 1.8.5.2