Jeff King <> writes:

> On Mon, Aug 04, 2014 at 05:45:44PM +0200, Matthieu Moy wrote:
>> Tanay Abhra <> writes:
>> > `git_pretty_formats_config()` continues without checking 
>> > git_config_string's
>> > return value which can lead to a SEGFAULT.
>> Indeed, without the patch:
>> $ git -c log --pretty=my                        
>> error: Missing value for ''                         
>> zsh: segmentation fault  git -c log --pretty=my
> Hmm. Not related to the original patch, but that really looks like a
> bug. Shouldn't "git -c ..." set to the empty string?
> I'd expect "git -c ..." to set it to NULL (i.e., the "implicit
> true" you get from omitting the "=" in the config files themselves).


strbuf_split_buf() does not seem to distinguish between x= and x. No
time to debug this further, sorry.

Matthieu Moy
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