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 /scan-tree.c | |
| parent | cgit.c: remove useless null check (diff) | |
| download | cgit-687cdf69689edcaff8e9035af93baf1a9e71cf32.tar.gz 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>
Diffstat (limited to 'scan-tree.c')
| -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 | } |