# [PATCH] lock_ref_sha1_basic does not remove empty directories on BSD 2 messages from 2006-10-02 to 2006-10-03. Participants: Dennis Stosberg, Junio C Hamano. Thread: https://gitlist.dev/t/5787 ## Dennis Stosberg, 2006-10-02 17:23 Subject: [PATCH] lock_ref_sha1_basic does not remove empty directories on BSD Message-ID: <20061002172353.G44b12bbc@leonov.stosberg.net> URL: https://gitlist.dev/e/20061002172353.G44b12bbc%40leonov.stosberg.net ``` lock_ref_sha1_basic relies on errno beeing set to EISDIR by the call to read() in resolve_ref() to detect directories. But calling read() on a directory under NetBSD returns EPERM, and even succeeds for local filesystems on FreeBSD. Signed-off-by: Dennis Stosberg --- refs.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/refs.c b/refs.c index aa4c4e0..305c1a9 100644 --- a/refs.c +++ b/refs.c @@ -234,6 +234,12 @@ const char *resolve_ref(const char *ref, } } + /* Is it a directory? */ + if (S_ISDIR(st.st_mode)) { + errno = EISDIR; + return NULL; + } + /* * Anything else, just open it and try to use it as * a ref -- 1.4.2 ``` ## Junio C Hamano, 2006-10-03 04:14 Subject: Re: [PATCH] lock_ref_sha1_basic does not remove empty directories on BSD Message-ID: <7vu02mt4qx.fsf@assigned-by-dhcp.cox.net> URL: https://gitlist.dev/e/7vu02mt4qx.fsf%40assigned-by-dhcp.cox.net In-Reply-To: <20061002172353.G44b12bbc@leonov.stosberg.net> ``` Dennis Stosberg writes: > lock_ref_sha1_basic relies on errno beeing set to EISDIR by the > call to read() in resolve_ref() to detect directories. But calling > read() on a directory under NetBSD returns EPERM, and even succeeds > for local filesystems on FreeBSD. > > Signed-off-by: Dennis Stosberg Thanks. I've always wondered about the code that follows where you patched. It relies on either open() on a directory to fail, or read() from a file descriptor to return something other than what starts with 40-byte hexadecimal (or "ref: blah") to skip directories. You might probably meant the patch primarily to fix the leftover empty directories issue in "next", but it is also the right thing to do for "master" (and even for "maint"), I think. I'll apply it to "master" and then merge it into "next". ```