[coreboot-gerrit] Change in coreboot[master]: device/pci_device.c: Return if the scan parameter is invalid

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63038 )

Change subject: device/pci_device.c: Return if the scan parameter is invalid
......................................................................

device/pci_device.c: Return if the scan parameter is invalid

Clang is unhappy about codepath of an invalid parameter because
variables remain unset.

Change-Id: I1ba392a48cf3f81a29d9645e5cf220b122d588af
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63038
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/device/pci_device.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 22ccf59..90b26c1 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -1518,6 +1518,8 @@
    	primary = parent->secondary;
    	secondary = link->secondary;
    	subordinate = link->subordinate;
+	} else {
+		return;
    }

if (state == PCI_ROUTE_SCAN) {