[coreboot-gerrit] [XS] Change in coreboot[main]: mb/google/brya/var/bb/nissa: Fix potential null pointer dereference

[Subrata Banik (Code Review)]

Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/82628?usp=email )

Change subject: mb/google/brya/var/bb/nissa: Fix potential null pointer dereference
......................................................................

mb/google/brya/var/bb/nissa: Fix potential null pointer dereference

* Introduce a null check before calling `gpio_padbased_override`
  in `variant_configure_pads`.
* This prevents potential errors in cases where the
`variant_gpio_override_table` function returns a null pointer,
indicating that there are no override pads to configure.

BUG=b:334826281
TEST=Able to avoid hang incase there is no GPIO override.

Change-Id: I733210a08091b37eda6e6b0d6924aafd5e7e6280
Signed-off-by: Subrata Banik 
---
M src/mainboard/google/brya/variants/baseboard/nissa/ramstage.c
1 file changed, 2 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/82628/1

diff --git a/src/mainboard/google/brya/variants/baseboard/nissa/ramstage.c b/src/mainboard/google/brya/variants/baseboard/nissa/ramstage.c
index dc686fe..64e1b14 100644
--- a/src/mainboard/google/brya/variants/baseboard/nissa/ramstage.c
+++ b/src/mainboard/google/brya/variants/baseboard/nissa/ramstage.c
@@ -16,7 +16,8 @@
    base_pads = variant_gpio_table(&base_num);
    gpio_padbased_override(padbased_table, base_pads, base_num);
    override_pads = variant_gpio_override_table(&override_num);
-	gpio_padbased_override(padbased_table, override_pads, override_num);
+	if (override_pads != NULL)
+		gpio_padbased_override(padbased_table, override_pads, override_num);
    fw_config_gpio_padbased_override(padbased_table);
    gpio_configure_pads_with_padbased(padbased_table);
    free(padbased_table);