]> git.ipfire.org Git - thirdparty/git.git/commit
unpack-trees: fix sparse directory recursion check
authorVictoria Dye <vdye@github.com>
Thu, 1 Sep 2022 21:02:33 +0000 (21:02 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 2 Sep 2022 16:43:09 +0000 (09:43 -0700)
commit037f8ea6d961a52deaa8df5150049850a53d61ac
treebd9665165c48c92cd903b0e22d11fbfadd93b950
parentb15207b8cf1a1930fe5eb076c08c6ddc92d9282d
unpack-trees: fix sparse directory recursion check

Ensure 'is_sparse_directory_entry()' receives a valid 'name_entry *' if one
exists in the list of tree(s) being unpacked in 'unpack_callback()'.

Currently, 'is_sparse_directory_entry()' is called with the first
'name_entry' in the 'names' list of entries on 'unpack_callback()'. However,
this entry may be empty even when other elements of 'names' are not (such as
when switching from an orphan branch back to a "normal" branch). As a
result, 'is_sparse_directory_entry()' could incorrectly indicate that a
sparse directory is *not* actually sparse because the name of the index
entry does not match the (empty) 'name_entry' path.

Fix the issue by using the existing 'name_entry *p' value in
'unpack_callback()', which points to the first non-empty entry in 'names'.
Because 'p' is 'const', also update 'is_sparse_directory_entry()'s
'name_entry *' argument to be 'const'.

Finally, add a regression test case.

Reported-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Victoria Dye <vdye@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t1092-sparse-checkout-compatibility.sh
unpack-trees.c