[coreboot] [PATCH] v3: Fix two NULL pointer dereferences in device code
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Wed Mar 5 13:56:21 CET 2008
On 05.03.2008 03:54, ron minnich wrote:
> Acked-by: Ronald G. Minnich <rminnich at gmail.com>
>
Thanks, committed in r631.
> This was on db800 right? I need to get a failure case.
>
db800, norwich, probably any AMD board triggered the NULL dereference in
device/device.c. There is still one NULL dereference left in
device/device.c and I need you to answer my question to fix it
completely. Then again, you probably asked for a way to reproduce to see
this. With the patch applied, look for "ERROR: bus->dev is NULL" in the
logs and notice if that appears, there is garbage in the log later on.
>> Ron? You can probably tell if bus->dev==NULL in read_resources(bus)
>> is an error or if we need to handle it cleanly.
>>
All boards (including qemu) triggered the NULL pointer in
device/pci_device.c. Look for "None (not found)" in the logs with the
patch applied. You'll see it 33 times in the qemu boot log.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list