Stefan Beller <> writes:

> The condition as it is written in that line was most likely intended to
> check for the pointer passed to free(), rather than checking for the
> 'repo_abbrev', which is already checked against being non null at the
> beginning of the function.
> -                     if (repo_abbrev)
> +                     if (*repo_abbrev)
>                               free(*repo_abbrev);

But now the test is useless, because free(NULL) is defined to be a

Thomas Rast
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to