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

Stefan Reinauer stepan at coresystems.de
Thu Aug 23 16:54:15 CEST 2007


* Patrick Georgi <patrick at georgi-clan.de> [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 <stepan at coresystems.de>


-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/




More information about the coreboot mailing list