diff options
| author | Johannes Schindelin <johannes.schindelin@gmx.de> | 2017-12-22 12:50:50 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-12-22 13:34:28 -0800 |
| commit | ed1e52822ea4a8493855d9b8be1049d561946137 (patch) | |
| tree | 0c2cfc83ccc0791528f84ca5392e769f5adf9de3 /sequencer.c | |
| parent | 28d6daed4f119940ace31e523b3b272d3d153d04 (diff) | |
| download | git-ed1e52822ea4a8493855d9b8be1049d561946137.tar.gz | |
sequencer: assign only free()able strings to gpg_sign
The gpg_sign member of the replay_opts structure is of type `char *`,
meaning that the sequencer deems the string to which gpg_sign points to
be under its custody, i.e. it needs to be free()d by the sequencer.
Therefore, let's only assign malloc()ed buffers to it.
Reported-by: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'sequencer.c')
| -rw-r--r-- | sequencer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sequencer.c b/sequencer.c index 7051b20b76..1b2599668f 100644 --- a/sequencer.c +++ b/sequencer.c @@ -160,7 +160,7 @@ static int git_sequencer_config(const char *k, const char *v, void *cb) } if (!strcmp(k, "commit.gpgsign")) { - opts->gpg_sign = git_config_bool(k, v) ? "" : NULL; + opts->gpg_sign = git_config_bool(k, v) ? xstrdup("") : NULL; return 0; } |