Jeff King wrote: > But is it noisy about a missing pipe? We do not get EPIPE for reading. Ah, that makes more sense. [...] >>> + len = packet_read_from_buf(line, sizeof(line), &last->buf, &last->len); >>> + if (len && line[len - 1] == '\n') >>> + len--; >> >> Was anything guaranteeing that buffer.len < 1000 before this change? > > No. That's discussed in point (3) of the "implications" in the commit > message. Thanks. Sorry I missed it the first time. -- 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