diff options
| author | Michael Haggerty <mhagger@alum.mit.edu> | 2015-07-28 23:08:18 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2015-07-28 14:39:10 -0700 |
| commit | c26f7d7b268c14f9ee7fa9cbeaad3bc890526d49 (patch) | |
| tree | 2f18999443f5db110f860fe65bf23123f0aa96ee /builtin | |
| parent | 9a3d637541a5b6fcd84b6f5fa057e597d1696460 (diff) | |
| download | git-c26f7d7b268c14f9ee7fa9cbeaad3bc890526d49.tar.gz | |
get_remote_group(): handle remotes with single-character names
The code for splitting a whitespace-separated list of values in
"remotes.<name>" had an off-by-one error that caused it to skip over
remotes whose names consist of a single character.
Also remove unnecessary braces.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/fetch.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/builtin/fetch.c b/builtin/fetch.c index 75a55e590b..9f7fe983a9 100644 --- a/builtin/fetch.c +++ b/builtin/fetch.c @@ -978,10 +978,9 @@ static int get_remote_group(const char *key, const char *value, void *priv) /* split list by white space */ int space = strcspn(value, " \t\n"); while (*value) { - if (space > 1) { + if (space >= 1) string_list_append(g->list, xstrndup(value, space)); - } value += space + (value[space] != '\0'); space = strcspn(value, " \t\n"); } |