Re: [PATCH 23/44] setup: set the_repository's hash algo when checking format
- From
brian m. carlson <sandals@crustytoothpaste.net>
- Date
- May 16, 2020, 19:29 UTC
- Message-ID
- <20200516192955.GE6362@camp.crustytoothpaste.net>
- In-Reply-To
- <CAN0heSrMrXE2g4LaZ4A71CxGteVfE2RfsT6WyGzmfFSHFKjATQ@mail.gmail.com>
On 2020-05-16 at 11:03:56, Martin Ågren wrote:
Show 12 quoted lines
> On Wed, 13 May 2020 at 02:56, brian m. carlson > <sandals@crustytoothpaste.net> wrote: > > > > When we're checking the repository's format, set the hash algorithm at > > the same time. This ensures that we perform a suitable initialization > > early enough to avoid confusing any parts of the code. If we defer > > until later, we can end up with portions of the code which are confused > > about the hash algorithm, resulting in segfaults. > > This doesn't make a difference as long as you just use SHA-1, right? > That is, this isn't a bug in the first half of this series nor in > v2.27-rc0 as long as you stick to SHA-1?
Correct, because the default is SHA-1 if no algorithm is specified. I'll update the commit message to reflect that this affects only SHA-256.
-- brian m. carlson: Houston, Texas, US OpenPGP: https://keybase.io/bk2204