diff options
author | John Keeping | 2015-10-08 23:23:57 +0100 |
---|---|---|
committer | Jason A. Donenfeld | 2015-10-09 10:54:30 +0200 |
commit | 687cdf69689edcaff8e9035af93baf1a9e71cf32 (patch) | |
tree | 1bacbdc963a38cd0befc9ddf7e4ed8d98d974fa8 | |
parent | 94182d6031df0d956a94ecd7ece233e345468961 (diff) | |
download | cgit-687cdf69689edcaff8e9035af93baf1a9e71cf32.tar.gz cgit-687cdf69689edcaff8e9035af93baf1a9e71cf32.tar.bz2 cgit-687cdf69689edcaff8e9035af93baf1a9e71cf32.zip |
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.c | 2 |
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) | |||
174 | 174 | ||
175 | strbuf_addstr(path, "cgitrc"); | 175 | strbuf_addstr(path, "cgitrc"); |
176 | if (!stat(path->buf, &st)) | 176 | if (!stat(path->buf, &st)) |
177 | parse_configfile(xstrdup(path->buf), &repo_config); | 177 | parse_configfile(path->buf, &repo_config); |
178 | 178 | ||
179 | strbuf_release(&rel); | 179 | strbuf_release(&rel); |
180 | } | 180 | } |