[PATCH v2 15/20] try_remove_empty_parents(): don't accommodate consecutive slashes
- From
Michael Haggerty <mhagger@alum.mit.edu>
- Date
- Feb 25, 2016, 13:16 UTC
- Message-ID
- <c9ae141d685e6ab835838252b726c5da519cb0fd.1456405699.git.mhagger@alum.mit.edu>
- In-Reply-To
- <cover.1456405698.git.mhagger@alum.mit.edu>
"refname" has already been checked by check_refname_format(), so it cannot have consecutive slashes.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> --- refs/files-backend.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/refs/files-backend.c b/refs/files-backend.c index 1fcbb6f..46dcc23 100644 --- a/refs/files-backend.c +++ b/refs/files-backend.c @@ -2212,15 +2212,14 @@ static void try_remove_empty_parents(const char *refname) for (i = 0; i < 2; i++) { /* refs/{heads,tags,...}/ */ while (*p && *p != '/') p++; - /* tolerate duplicate slashes; see check_refname_format() */ - while (*p == '/') + if (*p == '/') p++; } q = buf.buf + buf.len; while (1) { while (q > p && *q != '/') q--; - while (q > p && *(q-1) == '/') + if (q > p && *(q-1) == '/') q--; if (q == p) break;
-- 2.7.0