Sverre Rabbelier wrote:
> --- a/remote-curl.c
> +++ b/remote-curl.c
> @@ -857,6 +857,8 @@ int main(int argc, const char **argv)
> do {
> if (strbuf_getline(&buf, stdin, '\n') == EOF)
> break;
> + if (buf.len == 0)
> + break;
Thanks. I wonder if that first "if" should be something like
if (strbuf_getline(&buf, stdin, '\n') == EOF) {
if (ferror(stdin))
fprintf(stderr, "Error reading command stream\n");
else
fprintf(stderr, "Unexpected end of command stream\n");
return 1;
}
to catch I/O errors (e.g., the transport-helper exiting early).
--
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