about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJohn Keeping <john@keeping.me.uk>2015-10-08 23:23:57 +0100
committerJason A. Donenfeld <Jason@zx2c4.com>2015-10-09 10:54:30 +0200
commit687cdf69689edcaff8e9035af93baf1a9e71cf32 (patch)
tree1bacbdc963a38cd0befc9ddf7e4ed8d98d974fa8
parentcgit.c: remove useless null check (diff)
downloadcgit-magenta-687cdf69689edcaff8e9035af93baf1a9e71cf32.tar.xz
scan-tree: remove useless strdup()
parse_configfile() takes a "const char *" and doesn't hold any
references to it after it returns; there is no reason to pass it a
duplicate.

Coverity-id: 13941
Signed-off-by: John Keeping <john@keeping.me.uk>
-rw-r--r--scan-tree.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/scan-tree.c b/scan-tree.c

index 8e3cf52..b5a10ff 100644 --- a/scan-tree.c +++ b/scan-tree.c
@@ -174,7 +174,7 @@ static void add_repo(const char *base, struct strbuf *path, repo_config_fn fn) strbuf_addstr(path, "cgitrc"); if (!stat(path->buf, &st)) - parse_configfile(xstrdup(path->buf), &repo_config); + parse_configfile(path->buf, &repo_config); strbuf_release(&rel); }