From 56b30364c5db6367279ffe88929f286f15680b40 Mon Sep 17 00:00:00 2001 From: Michael Brown Date: Thu, 15 Sep 2022 15:04:01 +0100 Subject: [pci] Check for wraparound in callers of pci_find_next() The semantics of the bus:dev.fn parameter passed to pci_find_next() are "find the first existent PCI device at this address or higher", with the caller expected to increment the address between finding devices. This does not allow the parameter to distinguish between the two cases "start from address zero" and "wrapped after incrementing maximal possible address", which could therefore lead to an infinite loop in the degenerate case that a device with address ffff:ff:1f.7 really exists. Fix by checking for wraparound in the caller (which is already responsible for performing the increment). Signed-off-by: Michael Brown --- src/include/ipxe/errfile.h | 1 + 1 file changed, 1 insertion(+) (limited to 'src/include/ipxe/errfile.h') diff --git a/src/include/ipxe/errfile.h b/src/include/ipxe/errfile.h index 359e4d2ce..9b955e574 100644 --- a/src/include/ipxe/errfile.h +++ b/src/include/ipxe/errfile.h @@ -397,6 +397,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL ); #define ERRFILE_linux_sysfs ( ERRFILE_OTHER | 0x00560000 ) #define ERRFILE_linux_acpi ( ERRFILE_OTHER | 0x00570000 ) #define ERRFILE_dynkeymap ( ERRFILE_OTHER | 0x00580000 ) +#define ERRFILE_pci_cmd ( ERRFILE_OTHER | 0x00590000 ) /** @} */ -- cgit