Sebastian Schuberth <sschuberth@xxxxxxxxx> writes: > - if (is_bare) { > - struct strbuf result = STRBUF_INIT; > - strbuf_addf(&result, "%.*s.git", (int)(end - start), start); > - dir = strbuf_detach(&result, NULL); > - } else > + if (is_bare) > + dir = xstrfmt("%.*s.git", (int)len, start); > + else > dir = xstrndup(start, end - start); The last one needs to be adjusted with s/end - start/len/. The last-minute rewrite without testing shows; your first two patches correctly used "len" ;-) No need to resend. Will locally tweak before queuing. Thanks. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html