diff options
author | Junio C Hamano <gitster@pobox.com> | 2007-11-08 01:38:12 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2007-11-08 01:43:34 -0800 |
commit | 6738c8194286fa0017f72cb57628dcae9ec07b27 (patch) | |
tree | 6af8805826bddec635fc079c4e28349104a4faa2 /t | |
parent | b50fa2bd061c3bb21f2918849ece43ac9ca2c504 (diff) | |
download | git-6738c8194286fa0017f72cb57628dcae9ec07b27.tar.gz git-6738c8194286fa0017f72cb57628dcae9ec07b27.tar.xz |
send-pack: segfault fix on forced push
When pushing to overwrite a ref that points at a commit we do
not even have, the recent "terse push" patch tried to get a
unique abbreviation for the non-existent (from our point of
view) object, which resulted in strcpy(buf, NULL) and
segfaulted.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
-rwxr-xr-x | t/t5405-send-pack-rewind.sh | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/t/t5405-send-pack-rewind.sh b/t/t5405-send-pack-rewind.sh new file mode 100755 index 000000000..86abc6227 --- /dev/null +++ b/t/t5405-send-pack-rewind.sh @@ -0,0 +1,42 @@ +#!/bin/sh + +test_description='forced push to replace commit we do not have' + +. ./test-lib.sh + +test_expect_success setup ' + + >file1 && git add file1 && test_tick && + git commit -m Initial && + + mkdir another && ( + cd another && + git init && + git fetch .. master:master + ) && + + >file2 && git add file2 && test_tick && + git commit -m Second + +' + +test_expect_success 'non forced push should die not segfault' ' + + ( + cd another && + git push .. master:master + test $? = 1 + ) + +' + +test_expect_success 'forced push should succeed' ' + + ( + cd another && + git push .. +master:master + ) + +' + +test_done |