[coreboot-gerrit] Change in coreboot[master]: soc/skylake: prevent null pointer dereferences

[Angel Pons (Code Review)]

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35173 )

Change subject: soc/skylake: prevent null pointer dereferences
......................................................................

soc/skylake: prevent null pointer dereferences

Change-Id: Ide10223e7fc37a6c4bfa408234ef3efe1846236a
Signed-off-by: Angel Pons 
---
M src/soc/intel/skylake/chip.c
M src/soc/intel/skylake/chip_fsp20.c
2 files changed, 19 insertions(+), 20 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/35173/1

diff --git a/src/soc/intel/skylake/chip.c b/src/soc/intel/skylake/chip.c
index a7d5872..812572a 100644
--- a/src/soc/intel/skylake/chip.c
+++ b/src/soc/intel/skylake/chip.c
@@ -153,10 +153,7 @@

/* Enable ISH if device is on */
    dev = pcidev_path_on_root(PCH_DEVFN_ISH);
-	if (dev)
-		params->IshEnable = dev->enabled;
-	else
-		params->IshEnable = 0;
+	params->IshEnable = (dev) ? dev->enabled : 0;

params->EnableAzalia = config->EnableAzalia;
    params->IoBufferOwnership = config->IoBufferOwnership;
@@ -210,23 +207,24 @@
     * do the changes and then lock it back in coreboot
     *
     */
-	if (config->HeciEnabled == 0)
-		params->PsfUnlock = 1;
-	else
-		params->PsfUnlock = 0;
+	params->PsfUnlock = !config->HeciEnabled;

for (i = 0; i < ARRAY_SIZE(config->domain_vr_config); i++)
    	fill_vr_domain_config(params, i, &config->domain_vr_config[i]);

/* Show SPI controller if enabled in devicetree.cb */
    dev = pcidev_path_on_root(PCH_DEVFN_SPI);
-	params->ShowSpiController = dev->enabled;
+	params->ShowSpiController = (dev) ? dev->enabled : 0;

/* Enable xDCI controller if enabled in devicetree and allowed */
    dev = pcidev_path_on_root(PCH_DEVFN_USBOTG);
-	if (!xdci_can_enable())
-		dev->enabled = 0;
-	params->XdciEnable = dev->enabled;
+	if (dev) {
+		if (!xdci_can_enable())
+			dev->enabled = 0;
+		params->XdciEnable = dev->enabled;
+	} else {
+		params->XdciEnable = 0;
+	}

params->SendVrMbxCmd = config->SendVrMbxCmd;

diff --git a/src/soc/intel/skylake/chip_fsp20.c b/src/soc/intel/skylake/chip_fsp20.c
index 064f71e..d594bbb 100644
--- a/src/soc/intel/skylake/chip_fsp20.c
+++ b/src/soc/intel/skylake/chip_fsp20.c
@@ -354,10 +354,7 @@

/* If ISH is enabled, enable ISH elements */
    dev = pcidev_path_on_root(PCH_DEVFN_ISH);
-	if (dev)
-		params->PchIshEnable = dev->enabled;
-	else
-		params->PchIshEnable = 0;
+	params->PchIshEnable = (dev) ? dev->enabled : 0;

params->PchHdaEnable = config->EnableAzalia;
    params->PchHdaIoBufferOwnership = config->IoBufferOwnership;
@@ -433,13 +430,17 @@

/* Show SPI controller if enabled in devicetree.cb */
    dev = pcidev_path_on_root(PCH_DEVFN_SPI);
-	params->ShowSpiController = dev->enabled;
+	params->ShowSpiController = (dev) ? dev->enabled : 0;

/* Enable xDCI controller if enabled in devicetree and allowed */
    dev = pcidev_path_on_root(PCH_DEVFN_USBOTG);
-	if (!xdci_can_enable())
-		dev->enabled = 0;
-	params->XdciEnable = dev->enabled;
+	if (dev) {
+		if (!xdci_can_enable())
+			dev->enabled = 0;
+		params->XdciEnable = dev->enabled;
+	} else {
+		params->XdciEnable = 0;
+	}

/*
     * Send VR specific mailbox commands: