Code

apply: don't segfault on binary files with missing data
authorJeff King <peff@peff.net>
Mon, 18 Oct 2010 18:39:17 +0000 (14:39 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 29 Oct 2010 21:13:33 +0000 (14:13 -0700)
commit24305cd7009d682ba4115103c95844446a2bb305
treedee9960c98dde707d545b022dcb8c5d9d4b4ba50
parent87b50542a08ac6caa083ddc376e674424e37940a
apply: don't segfault on binary files with missing data

Usually when applying a binary diff generated without
--binary, it will be rejected early, as we don't even have
the full sha1 of the pre- and post-images.

However, if the diff is generated with --full-index (but not
--binary), then we will actually try to apply it. If we have
the postimage blob, then we can take a shortcut and never
even look at the binary diff at all (e.g., this can happen
when rebasing changes within a repository).

If we don't have the postimage blob, though, we try to look
at the actual fragments, of which there are none, and get a
segfault. This patch checks explicitly for that case and
complains to the user instead of segfaulting. We need to
keep the check at a low level so that the "shortcut" case
above continues to work.

We also add a test that demonstrates the segfault. While
we're at it, let's also explicitly test the shortcut case.

Reported-by: Rafaël Carré <rafael.carre@gmail.com>
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/apply.c
t/t4103-apply-binary.sh