]> git.ipfire.org Git - thirdparty/git.git/commit - gpg-interface.c
gpg-interface: handle bool user.signingkey
authorJeff King <peff@peff.net>
Fri, 13 Apr 2018 21:18:30 +0000 (15:18 -0600)
committerJunio C Hamano <gitster@pobox.com>
Mon, 16 Apr 2018 05:15:02 +0000 (14:15 +0900)
commit1b0eeec3f359888f8a638de8af253f621a5b836e
treee6fca8ffe4624b808ced2ea3add8c7e4f07d7ef4
parentcf98a52ba4d8176a3ec73c72d296275999ecb52d
gpg-interface: handle bool user.signingkey

The config handler for user.signingkey does not check for a
boolean value, and thus:

  git -c user.signingkey tag

will segfault. We could fix this and even shorten the code
by using git_config_string(). But our set_signing_key()
helper is used by other code outside of gpg-interface.c, so
we must keep it (and we may as well use it, because unlike
git_config_string() it does not leak when we overwrite an
old value).

Ironically, the handler for gpg.program just below _could_
use git_config_string() but doesn't. But since we're going
to touch that in a future patch, we'll leave it alone for
now. We will add some whitespace and returns in preparation
for adding more config keys, though.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Ben Toews <mastahyeti@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
gpg-interface.c