diff options
| author | Johannes Schindelin <Johannes.Schindelin@gmx.de> | 2007-06-05 03:36:49 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2007-06-04 23:07:19 -0700 |
| commit | 9f30855d0ff5206e85e45f0307be9d18ffda41d3 (patch) | |
| tree | e30410dbde5863a683dcaea3d70689dd141f5afa | |
| parent | 6bfce93e04d13ecb42008a3cf214cc892f480f0c (diff) | |
| download | git-9f30855d0ff5206e85e45f0307be9d18ffda41d3.tar.gz git-9f30855d0ff5206e85e45f0307be9d18ffda41d3.tar.xz | |
merge-recursive: refuse to merge binary files
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | merge-recursive.c | 6 | ||||
| -rwxr-xr-x | t/t6024-recursive-merge.sh | 14 |
2 files changed, 20 insertions, 0 deletions
diff --git a/merge-recursive.c b/merge-recursive.c index 8f72b2c07..4a82b741a 100644 --- a/merge-recursive.c +++ b/merge-recursive.c @@ -680,6 +680,12 @@ static int ll_xdl_merge(const struct ll_merge_driver *drv_unused, { xpparam_t xpp; + if (buffer_is_binary(orig->ptr, orig->size) || + buffer_is_binary(src1->ptr, src1->size) || + buffer_is_binary(src2->ptr, src2->size)) + return error("Cannot merge binary files: %s vs. %s\n", + name1, name2); + memset(&xpp, 0, sizeof(xpp)); return xdl_merge(orig, src1, name1, diff --git a/t/t6024-recursive-merge.sh b/t/t6024-recursive-merge.sh index a39855613..058db9cc5 100755 --- a/t/t6024-recursive-merge.sh +++ b/t/t6024-recursive-merge.sh @@ -81,4 +81,18 @@ EOF test_expect_success "virtual trees were processed" "git diff expect out" +git reset --hard +test_expect_success 'refuse to merge binary files' ' + printf "\0" > binary-file && + git add binary-file && + git commit -m binary && + git checkout G && + printf "\0\0" > binary-file && + git add binary-file && + git commit -m binary2 && + ! git merge F > merge.out 2> merge.err && + grep "Cannot merge binary files: HEAD:binary-file vs. F:binary-file" \ + merge.err +' + test_done |