From: Jeff King Date: Tue, 18 Oct 2022 01:01:17 +0000 (-0400) Subject: diffstat_consume(): assert non-zero length X-Git-Tag: v2.39.0-rc0~80^2~11 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0e5a87e0422ff308641199d1afe94b58e1249bc0;p=thirdparty%2Fgit.git diffstat_consume(): assert non-zero length The callback interface for xdiff_emit_line_fn gives us a line/len pair, but diffstat_consume() never looks at "len". At first glance this seems like a bug that could cause us to read further than xdiff intends. But in practice, we read only the first character, and xdiff would never pass us an empty line. Let's add a run-time assertion that this is true, which clarifies our assumption and silences -Wunused-parameter. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- diff --git a/diff.c b/diff.c index 648f6717a5..bba888a34a 100644 --- a/diff.c +++ b/diff.c @@ -2488,6 +2488,9 @@ static int diffstat_consume(void *priv, char *line, unsigned long len) struct diffstat_t *diffstat = priv; struct diffstat_file *x = diffstat->files[diffstat->nr - 1]; + if (!len) + BUG("xdiff fed us an empty line"); + if (line[0] == '+') x->added++; else if (line[0] == '-')