]> git.ipfire.org Git - thirdparty/ipxe.git/commit
[pci] Ensure that pci_read_config() initialises all fields
authorMichael Brown <mcb30@ipxe.org>
Wed, 16 Feb 2022 12:30:02 +0000 (12:30 +0000)
committerMichael Brown <mcb30@ipxe.org>
Wed, 16 Feb 2022 12:55:28 +0000 (12:55 +0000)
commit04288974f6d81019314cbf9cbd72ab1fae95496f
treee34966223f084e094ebf08f827ce1596d59bfb6e
parent5d22307c4161dde453d50e8dc7bef8b3a2f6c9b3
[pci] Ensure that pci_read_config() initialises all fields

As per the general pattern for initialisation functions in iPXE,
pci_init() saves code size by assuming that the caller has already
zeroed the underlying storage (e.g. as part of zeroing a larger
containing structure).  There are several places within the code where
pci_init() is deliberately used to initialise a transient struct
pci_device without zeroing the entire structure, because the calling
code knows that only the PCI bus:dev.fn address is required to be
initialised (e.g. when reading from PCI configuration space).

Ensure that using pci_init() followed by pci_read_config() will fully
initialise the struct pci_device even if the caller did not previously
zero the underlying storage, since Coverity reports that there are
several places in the code that rely upon this.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/drivers/bus/pci.c