diff options
| author | Theo Niessink <theo@taletn.com> | 2011-06-08 14:04:41 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-06-08 16:34:38 -0700 |
| commit | e0f530ff8afbd252170f57e70a8609a83a7cabe1 (patch) | |
| tree | c6fa772930e8c6dfe359165995e7e83a20024b35 | |
| parent | 3bdf09c7f555d553b4fee00c00c760b546812d4f (diff) | |
| download | git-e0f530ff8afbd252170f57e70a8609a83a7cabe1.tar.gz git-e0f530ff8afbd252170f57e70a8609a83a7cabe1.tar.xz | |
verify_dotfile(): do not assume '/' is the path seperator
verify_dotfile() currently assumes that the path seperator is '/', but on
Windows it can also be '\\', so use is_dir_sep() instead.
Signed-off-by: Theo Niessink <theo@taletn.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | read-cache.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/read-cache.c b/read-cache.c index 3593291f7..e7c968421 100644 --- a/read-cache.c +++ b/read-cache.c @@ -747,11 +747,12 @@ static int verify_dotfile(const char *rest) * has already been discarded, we now test * the rest. */ - switch (*rest) { + /* "." is not allowed */ - case '\0': case '/': + if (*rest == '\0' || is_dir_sep(*rest)) return 0; + switch (*rest) { /* * ".git" followed by NUL or slash is bad. This * shares the path end test with the ".." case. @@ -764,7 +765,7 @@ static int verify_dotfile(const char *rest) rest += 2; /* fallthrough */ case '.': - if (rest[1] == '\0' || rest[1] == '/') + if (rest[1] == '\0' || is_dir_sep(rest[1])) return 0; } return 1; |