diff options
author | Jeff King <peff@peff.net> | 2014-01-27 20:37:30 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-01-28 11:59:49 -0800 |
commit | 67beb600563cf28186f44450e528df1ec4d524fd (patch) | |
tree | 1365fcd9d56c7d156756e4417b1af25f2ffaf768 /t | |
parent | expand_user_path: do not look at NULL path (diff) | |
download | tgif-67beb600563cf28186f44450e528df1ec4d524fd.tar.xz |
handle_path_include: don't look at NULL value
When we see config like:
[include]
path
the expand_user_path helper notices that the config value is
empty, but we then dereference NULL while printing the error
message (glibc will helpfully print "(null)" for us here,
but we cannot rely on that).
$ git -c include.path rev-parse
error: Could not expand include path '(null)'
fatal: unable to parse command-line config
Instead of tweaking our message, let's actually use
config_error_nonbool to match other config variables that
expect a value:
$ git -c include.path rev-parse
error: Missing value for 'include.path'
fatal: unable to parse command-line config
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
0 files changed, 0 insertions, 0 deletions