diff options
author | Junio C Hamano <gitster@pobox.com> | 2018-02-27 21:33:54 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2018-02-27 21:33:55 +0300 |
commit | 2fb346c06a0160cac185ad57368c04c17c2578ac (patch) | |
tree | 2574f86d599946d9a364f3103578696013091f15 /builtin | |
parent | 8b49408421bba8a928c207991bc197a226b65782 (diff) | |
parent | bb1356dc643e1488ccc1924ab674f6cbbe615f74 (diff) |
Merge branch 'js/packet-read-line-check-null'
Some low level protocol codepath could crash when they get an
unexpected flush packet, which is now fixed.
* js/packet-read-line-check-null:
always check for NULL return from packet_read_line()
correct error messages for NULL packet_read_line()
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/archive.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/archive.c b/builtin/archive.c index f863465a0f..73971d0dd2 100644 --- a/builtin/archive.c +++ b/builtin/archive.c @@ -55,7 +55,7 @@ static int run_remote_archiver(int argc, const char **argv, buf = packet_read_line(fd[0], NULL); if (!buf) - die(_("git archive: expected ACK/NAK, got EOF")); + die(_("git archive: expected ACK/NAK, got a flush packet")); if (strcmp(buf, "ACK")) { if (starts_with(buf, "NACK ")) die(_("git archive: NACK %s"), buf + 5); |