Author: uwe
Date: 2007-08-26 14:12:22 +0200 (Sun, 26 Aug 2007)
New Revision: 480

Modified:
   LinuxBIOSv3/util/lar/stream.c
Log:
The checksum creation in lar starts somewhere in the lar image,
and ends at the end of a temporary buffer, which doesn't look
right to me (and segfaults if between those two there's an unmapped
region, as happened here).

Signed-off-by: Patrick Georgi <[EMAIL PROTECTED]>
Acked-by: Stefan Reinauer <[EMAIL PROTECTED]>
Acked-by: Uwe Hermann <[EMAIL PROTECTED]>



Modified: LinuxBIOSv3/util/lar/stream.c
===================================================================
--- LinuxBIOSv3/util/lar/stream.c       2007-08-26 12:06:51 UTC (rev 479)
+++ LinuxBIOSv3/util/lar/stream.c       2007-08-26 12:12:22 UTC (rev 480)
@@ -666,7 +666,7 @@
 
        csum = 0;
        for (walk = (u32 *) (lar->map + offset);
-            walk < (u32 *) (temp + complen + hlen);
+            walk < (u32 *) (lar->map + complen + hlen);
             walk++) {
                csum += ntohl(*walk);
        }


-- 
linuxbios mailing list
[email protected]
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to