summaryrefslogtreecommitdiff
path: root/credential.c
diff options
context:
space:
mode:
authorLibravatar Jon Simons <jon@jonsimons.org>2018-02-08 13:47:50 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2018-02-08 12:37:40 -0800
commitbb1356dc643e1488ccc1924ab674f6cbbe615f74 (patch)
tree88bf6c08ac171e299567bd53df1f2820d577c691 /credential.c
parentcorrect error messages for NULL packet_read_line() (diff)
downloadtgif-bb1356dc643e1488ccc1924ab674f6cbbe615f74.tar.xz
always check for NULL return from packet_read_line()
The packet_read_line() function will die if it sees any protocol or socket errors. But it will return NULL for a flush packet; some callers which are not expecting this may dereference NULL if they get an unexpected flush. This would involve the other side breaking protocol, but we should flag the error rather than segfault. Signed-off-by: Jon Simons <jon@jonsimons.org> Reviewed-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'credential.c')
0 files changed, 0 insertions, 0 deletions