checkout: set ignorecase=0 when config lookup fails

When `git_repository__cvar` fails we may end up with a
`ignorecase` value of `-1`. As we subsequently check if
`ignorecase` is non-zero, we may end up reporting that data
should be removed when in fact it should not.

Err on the safer side and set `ignorecase = 0` when
`git_repository__cvar` fails.
This commit is contained in:
Patrick Steinhardt 2016-05-02 15:47:54 +02:00 committed by Carlos Martín Nieto
parent 66633e836f
commit f627e19662

View File

@ -1342,9 +1342,11 @@ fail:
static bool should_remove_existing(checkout_data *data)
{
int ignorecase = 0;
int ignorecase;
git_repository__cvar(&ignorecase, data->repo, GIT_CVAR_IGNORECASE);
if (git_repository__cvar(&ignorecase, data->repo, GIT_CVAR_IGNORECASE) < 0) {
ignorecase = 0;
}
return (ignorecase &&
(data->strategy & GIT_CHECKOUT_DONT_REMOVE_EXISTING) == 0);