[PATCH v2 1/3] setup: free old value before setting `work_tree`
- From
- Martin Ågren <martin.agren@gmail.com>
- Date
- Jan 14, 2019, 18:34 UTC
- Message-ID
- <8585c0bfdc071f17e75346b801a16580d360c18a.1547488709.git.martin.agren@gmail.com>
- In-Reply-To
- <cover.1547488709.git.martin.agren@gmail.com>
Before assigning to `data->work_tree` in `read_worktree_config()`, free any value we might already have picked up, so that we do not leak it.
Signed-off-by: Martin Ågren <martin.agren@gmail.com> --- setup.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/setup.c b/setup.c index 1be5037f12..bb633942bb 100644 --- a/setup.c +++ b/setup.c @@ -411,6 +411,7 @@ static int read_worktree_config(const char *var, const char *value, void *vdata) } else if (strcmp(var, "core.worktree") == 0) { if (!value) return config_error_nonbool(var); + free(data->work_tree); data->work_tree = xstrdup(value); } return 0;
-- 2.20.1