]> git.ipfire.org Git - thirdparty/git.git/commit
bundle: default to SHA1 when reading bundle headers
authorPatrick Steinhardt <ps@pks.im>
Tue, 13 Aug 2024 09:18:15 +0000 (11:18 +0200)
committerJunio C Hamano <gitster@pobox.com>
Tue, 13 Aug 2024 17:26:44 +0000 (10:26 -0700)
commit96a9a3e42e85874ba5edfcf86d91f7d8c05d5f94
tree795875497f2397ee651c441d3eab34a549f6c062
parent7298bcc573f8e022854b1811a9a7413281da05ea
bundle: default to SHA1 when reading bundle headers

We hit a segfault when trying to open a bundle via `git bundle
list-heads` when running outside of a repository. This is caused by
c8aed5e8da (repository: stop setting SHA1 as the default object hash,
2024-05-07), which stopped setting the default object hash so that
`the_hash_algo` is a `NULL` pointer when running outside of any repo.

This is only a symptom of a deeper issue though. Bundles default to the
SHA1 object format unless they advertise an "@object-format=" header.
Consequently, it has been wrong in the first place to use the object
format used by the current repository when parsing bundles. The
consequence is that trying to open a bundle that uses a different object
hash than the current repository will fail:

    $ git bundle list-heads sha1.bundle
    error: unrecognized header: ee4b540943284700a32591ad09f7e15bdeb2a10c HEAD (45)

Fix the bug by defaulting to the SHA1 object hash. We already handle the
"@object-format=" header as expected, so we don't need to adapt this
part.

Helped-by: brian m. carlson <sandals@crustytoothpaste.net>
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
bundle.c
t/t6020-bundle-misc.sh