undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fsck: complain about NTFS ".git" aliases in trees

Browse Source 
Now that the index can block pathnames that can be mistaken
to mean ".git" on NTFS and FAT32, it would be helpful for
fsck to notice such problematic paths. This lets servers
which use receive.fsckObjects block them before the damage
spreads.

Note that the fsck check is always on, even for systems
without core.protectNTFS set. This is technically more
restrictive than we need to be, as a set of users on ext4
could happily use these odd filenames without caring about
NTFS.

However, on balance, it's helpful for all servers to block
these (because the paths can be used for mischief, and
servers which bother to fsck would want to stop the spread
whether they are on NTFS themselves or not), and hardly
anybody will be affected (because the blocked names are
variants of .git or git~1, meaning mischief is almost
certainly what the tree author had in mind).

Ideally these would be controlled by a separate
"fsck.protectNTFS" flag. However, it would be much nicer to
be able to enable/disable _any_ fsck flag individually, and
any scheme we choose should match such a system. Given the
likelihood of anybody using such a path in practice, it is
not unreasonable to wait until such a system materializes.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Johannes Schindelin 2014-12-10 22:28:27 +01:00 committed by Junio C Hamano
parent 2b4c6efc82
commit d08c13b947
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
fsck.c
View File

@ -176,7 +176,8 @@ static int fsck_tree(struct tree *item, int strict, fsck_error error_func)
has_dot = 1; has_dot = 1;
if (!strcmp(name, "..")) if (!strcmp(name, ".."))
has_dotdot = 1; has_dotdot = 1;
if (!strcasecmp(name, ".git") || is_hfs_dotgit(name)) if (!strcasecmp(name, ".git") || is_hfs_dotgit(name) ||
is_ntfs_dotgit(name))
has_dotgit = 1; has_dotgit = 1;
has_zero_pad |= *(char *)desc.buffer == '0'; has_zero_pad |= *(char *)desc.buffer == '0';
update_tree_entry(&desc); update_tree_entry(&desc);

9
t/t1450-fsck.sh
View File

@ -251,10 +251,10 @@ while read name path pretty; do
tree=$(git rev-parse HEAD^{tree}) && tree=$(git rev-parse HEAD^{tree}) &&
value=$(eval "echo \$$type") && value=$(eval "echo \$$type") &&
printf "$mode $type %s\t%s" "$value" "$path" >bad && printf "$mode $type %s\t%s" "$value" "$path" >bad &&
git mktree <bad && bad_tree=$(git mktree <bad) &&
git fsck 2>out && git fsck 2>out &&
cat out && cat out &&
grep "warning.*\\." out grep "warning.*tree $bad_tree" out
)' )'
done <<-\EOF done <<-\EOF
100644 blob 100644 blob
@ -266,6 +266,11 @@ dotdot ..
dotgit .git dotgit .git
dotgit-case .GIT dotgit-case .GIT
dotgit-unicode .gI${u200c}T .gI{u200c}T dotgit-unicode .gI${u200c}T .gI{u200c}T
dotgit-case2 .Git
git-tilde1 git~1
dotgitdot .git.
dot-backslash-case .\\\\.GIT\\\\foobar
dotgit-case-backslash .git\\\\foobar
EOF EOF
test_done test_done