03267e8656
The read_cache() in prepare_to_commit() would end up clobbering the pointer we had for a previously populated "the_index.cache_tree" in the very common case of "git commit" stressed by e.g. the tests being changed here. We'd populate "the_index.cache_tree" by calling "update_main_cache_tree" in prepare_index(), but would not end up with a "fully prepared" index. What constitutes an existing index is clearly overly fuzzy, here we'll check "active_nr" (aka "the_index.cache_nr"), but our "the_index.cache_tree" might have been malloc()'d already. Thus the code added in11c8a74a64(commit: write cache-tree data when writing index anyway, 2011-12-06) would end up allocating the "cache_tree", and would interact here with code added in7168624c35(Do not generate full commit log message if it is not going to be used, 2007-11-28). The result was a very common memory leak. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Taylor Blau <me@ttaylorr.com>
66 lines
1.4 KiB
Bash
Executable File
66 lines
1.4 KiB
Bash
Executable File
#!/bin/sh
|
|
|
|
test_description='help.autocorrect finding a match'
|
|
|
|
TEST_PASSES_SANITIZE_LEAK=true
|
|
. ./test-lib.sh
|
|
|
|
test_expect_success 'setup' '
|
|
# An alias
|
|
git config alias.lgf "log --format=%s --first-parent" &&
|
|
|
|
# A random user-defined command
|
|
write_script git-distimdistim <<-EOF &&
|
|
echo distimdistim was called
|
|
EOF
|
|
|
|
PATH="$PATH:." &&
|
|
export PATH &&
|
|
|
|
git commit --allow-empty -m "a single log entry" &&
|
|
|
|
# Sanity check
|
|
git lgf >actual &&
|
|
echo "a single log entry" >expect &&
|
|
test_cmp expect actual &&
|
|
|
|
git distimdistim >actual &&
|
|
echo "distimdistim was called" >expect &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success 'autocorrect showing candidates' '
|
|
git config help.autocorrect 0 &&
|
|
|
|
test_must_fail git lfg 2>actual &&
|
|
grep "^ lgf" actual &&
|
|
|
|
test_must_fail git distimdist 2>actual &&
|
|
grep "^ distimdistim" actual
|
|
'
|
|
|
|
for immediate in -1 immediate
|
|
do
|
|
test_expect_success 'autocorrect running commands' '
|
|
git config help.autocorrect $immediate &&
|
|
|
|
git lfg >actual &&
|
|
echo "a single log entry" >expect &&
|
|
test_cmp expect actual &&
|
|
|
|
git distimdist >actual &&
|
|
echo "distimdistim was called" >expect &&
|
|
test_cmp expect actual
|
|
'
|
|
done
|
|
|
|
test_expect_success 'autocorrect can be declined altogether' '
|
|
git config help.autocorrect never &&
|
|
|
|
test_must_fail git lfg 2>actual &&
|
|
grep "is not a git command" actual &&
|
|
test_line_count = 1 actual
|
|
'
|
|
|
|
test_done
|