diff options
| author |  John Keeping <john@keeping.me.uk> | 2014-02-16 16:06:02 +0000 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2014-02-18 15:51:29 -0800 |
| commit | aa012e906542dbc533dbc5bafe1c4e0a47bdc59e (patch) | |
| tree | 6c68c14293f461733d3e632827ef94d31d600aa5 /notes-utils.c | |
| parent | Git 1.9.0 (diff) | |
| download | tgif-aa012e906542dbc533dbc5bafe1c4e0a47bdc59e.tar.xz | |
notes-utils: handle boolean notes.rewritemode correctly
If we carry on after outputting config_error_nonbool then we're
guaranteed to dereference a null pointer.
Signed-off-by: John Keeping <john@keeping.me.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'notes-utils.c')
| -rw-r--r-- | notes-utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/notes-utils.c b/notes-utils.c index 2975dcd581..4aa7023903 100644 --- a/notes-utils.c +++ b/notes-utils.c @@ -75,7 +75,7 @@ static int notes_rewrite_config(const char *k, const char *v, void *cb) return 0; } else if (!c->mode_from_env && !strcmp(k, "notes.rewritemode")) { if (!v) - config_error_nonbool(k); + return config_error_nonbool(k); c->combine = parse_combine_notes_fn(v); if (!c->combine) { error(_("Bad notes.rewriteMode value: '%s'"), v); |