mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
strbuf: support long paths w/o read rights in strbuf_getcwd() on FreeBSD
FreeBSD implements getcwd(3) as a syscall, but falls back to a version
based on readdir(3) if it fails for some reason. The latter requires
permissions to read and execute path components, while the former does
not. That means that if our buffer is too small and we're missing
rights we could get EACCES, but we may succeed with a bigger buffer.
Keep retrying if getcwd(3) indicates lack of permissions until our
buffer can fit PATH_MAX bytes, as that's the maximum supported by the
syscall on FreeBSD anyway. This way we do what we can to be able to
benefit from the syscall, but we also won't loop forever if there is a
real permission issue.
This fixes a regression introduced with 7333ed17
(setup: convert
setup_git_directory_gently_1 et al. to strbuf, 2014-07-28) for paths
longer than 127 bytes with components that miss read or execute
permissions (e.g. 0711 on /home for privacy reasons); we used a fixed
PATH_MAX-sized buffer before.
Reported-by: Zenobiusz Kunegunda <zenobiusz.kunegunda@interia.pl>
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
8f9aeb0d36
commit
a54e938e5b
11
strbuf.c
11
strbuf.c
|
@ -449,6 +449,17 @@ int strbuf_getcwd(struct strbuf *sb)
|
|||
strbuf_setlen(sb, strlen(sb->buf));
|
||||
return 0;
|
||||
}
|
||||
|
||||
/*
|
||||
* If getcwd(3) is implemented as a syscall that falls
|
||||
* back to a regular lookup using readdir(3) etc. then
|
||||
* we may be able to avoid EACCES by providing enough
|
||||
* space to the syscall as it's not necessarily bound
|
||||
* to the same restrictions as the fallback.
|
||||
*/
|
||||
if (errno == EACCES && guessed_len < PATH_MAX)
|
||||
continue;
|
||||
|
||||
if (errno != ERANGE)
|
||||
break;
|
||||
}
|
||||
|
|
|
@ -315,6 +315,20 @@ test_expect_success 'init with separate gitdir' '
|
|||
test_path_is_dir realgitdir/refs
|
||||
'
|
||||
|
||||
test_expect_success 'init in long base path' '
|
||||
# exceed initial buffer size of strbuf_getcwd()
|
||||
component=123456789abcdef &&
|
||||
test_when_finished "chmod 0700 $component; rm -rf $component" &&
|
||||
p31=$component/$component &&
|
||||
p127=$p31/$p31/$p31/$p31 &&
|
||||
mkdir -p $p127 &&
|
||||
chmod 0111 $component &&
|
||||
(
|
||||
cd $p127 &&
|
||||
git init newdir
|
||||
)
|
||||
'
|
||||
|
||||
test_expect_success 're-init on .git file' '
|
||||
( cd newdir && git init )
|
||||
'
|
||||
|
|
Loading…
Reference in a new issue