[LinuxBIOS] [PATCH] LAR no-segment crash (was: Re: r2758 - in trunk: LinuxBIOSv2/util util)

Alex Beregszaszi alex at rtfs.hu
Wed Sep 5 02:09:13 CEST 2007


Hi,

> > As of revision 2762, the v2 repository is still BROKEN and superiotool
> > doesn't appear in v3 either (checked revision 486).
> > 
> > Such commits are one reason why I think self-acked commits should not be
> > allowed.
> 
> Well. Half-hearted code reviews are an illusion of safety, too.
> 
> The current code's lar handling is pretty much broken even though that
> code was reviewed, but obviously nobody even tested it:
> 
> Wrote LinuxBIOS table at: 0x00000500 - 0x00000a54  checksum 401f
> Show all devs...
> cpus: Unknown device path type: 0
> Stage2 code done.
> LAR: Attempting to open 'normal/payload'.
> LAR: Attempting to open 'normal/payload/segment0'.
> LAR: load_file: No such file 'normal/payload/segment0'
> 
> It just hangs somewhere bogus and never gets to the defined error in 
> 
>   arch/x86/stage1.c:194: die("FATAL: No usable payload found.\n");
> 
> also the current code contains quite some warnings again regarding
> int-to-pointer casts in lar.c and pointer-to-int cast and uninitialized
> variable use in stage1.c

run_address gets called with entry=0.

Attached patch fixes it.

(If thread hi-jacking is forbidden, sorry)

--
Alex

-------------- next part --------------
A non-text attachment was scrubbed...
Name: lar-segment-nullpointer.diff
Type: text/x-patch
Size: 373 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070905/059c92ee/attachment.diff>


More information about the coreboot mailing list