git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: git show loop

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 26, 2012, 04:34 UTC
Message-ID
<7v7gy8rne3.fsf@alter.siamese.dyndns.org>
In-Reply-To
<4F6DE5D6.4010408@lsrfire.ath.cx>
René Scharfe <rene.scharfe@lsrfire.ath.cx> writes:
Show 9 quoted lines
> Subject: [PATCH] combine-diff: fix loop index underflow
>
> If both la and context are zero at the start of the loop, la wraps around
> and we end up reading from memory far away.  Skip the loop in that case
> instead.
>
> Reported-by: Julia Lawall <julia.lawall@lip6.fr>
> Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
> ---
Thanks. This dates back to 1.3.0.
Will queue.
Show 16 quoted lines
>  combine-diff.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/combine-diff.c b/combine-diff.c
> index a2e8dcf..9786680 100644
> --- a/combine-diff.c
> +++ b/combine-diff.c
> @@ -423,7 +423,7 @@ static int make_hunks(struct sline *sline, unsigned long cnt,
>  						     hunk_begin, j);
>  				la = (la + context < cnt + 1) ?
>  					(la + context) : cnt + 1;
> -				while (j <= --la) {
> +				while (la && j <= --la) {
>  					if (sline[la].flag & mark) {
>  						contin = 1;
>  						break;
Previous: René Scharfe
Message 5 of 5 in “git show loop”
  1. Julia LawallMar 24, 2012
  2. Zbigniew Jędrzejewski-SzmekMar 24, 2012
  3. Zbigniew Jędrzejewski-SzmekMar 24, 2012
  4. René ScharfeMar 24, 2012
  5. Junio C HamanoMar 26, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.