Re: [LinuxBIOS] [PATCH][v3] small lar fix

[Stefan Reinauer]

* Patrick Georgi  [070822 14:57]:
> Index: util/lar/stream.c
>
> --- util/lar/stream.c	(revision 478)
> +++ util/lar/stream.c	(working copy)
> @@ -666,7 +666,7 @@
>
> csum = 0;
>    for (walk = (u32 *) (lar->map + offset);
>     walk < (u32 *) (temp + complen + hlen);
>     walk < (u32 *) (lar->map + complen + hlen);

I would think it should be

walk < (u32 *) (lar->map + offset + complen + hlen)

because temp was copied to lar->map + offset before.

But I agree this needs fixing:

Acked-by: Stefan Reinauer