diff options
| author | Jeff King <peff@peff.net> | 2018-05-15 09:56:50 -0400 |
|---|---|---|
| committer | Jeff King <peff@peff.net> | 2018-05-21 23:50:11 -0400 |
| commit | 641084b618ddbe099f0992161988c3e479ae848b (patch) | |
| tree | 2a7e0bcf72c6477e605093c6f0ba2f972e3670d8 | |
| parent | verify_path: drop clever fallthrough (diff) | |
| download | git-641084b618ddbe099f0992161988c3e479ae848b.tar.gz git-641084b618ddbe099f0992161988c3e479ae848b.zip | |
verify_dotfile: mention case-insensitivity in comment
We're more restrictive than we need to be in matching ".GIT"
on case-sensitive filesystems; let's make a note that this
is intentional.
Signed-off-by: Jeff King <peff@peff.net>
| -rw-r--r-- | read-cache.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/read-cache.c b/read-cache.c index 5c5dfc629d..333e0c5429 100644 --- a/read-cache.c +++ b/read-cache.c @@ -810,7 +810,10 @@ static int verify_dotfile(const char *rest) switch (*rest) { /* - * ".git" followed by NUL or slash is bad. + * ".git" followed by NUL or slash is bad. Note that we match + * case-insensitively here, even if ignore_case is not set. + * This outlaws ".GIT" everywhere out of an abundance of caution, + * since there's really no good reason to allow it. */ case 'g': case 'G': |