[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block: Add NULL check for 'ctx' pointer

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/45411 )

Change subject: soc/intel/common/block: Add NULL check for 'ctx' pointer
......................................................................

soc/intel/common/block: Add NULL check for 'ctx' pointer

Found-by: Klocwork, Avoid NULL pointer (ctx) dereference.

Signed-off-by: Subrata Banik 
Change-Id: I16015b538112e0b125b4a5e145c26263c456953c
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45411
Reviewed-by: Furquan Shaikh 
Reviewed-by: Angel Pons 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/common/block/cse/cse_lite.c
1 file changed, 4 insertions(+), 3 deletions(-)

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

diff --git a/src/soc/intel/common/block/cse/cse_lite.c b/src/soc/intel/common/block/cse/cse_lite.c
index edb08da..7daa35e 100644
--- a/src/soc/intel/common/block/cse/cse_lite.c
+++ b/src/soc/intel/common/block/cse/cse_lite.c
@@ -169,13 +169,14 @@
    /* Log CSE Firmware Status Registers to help debugging */
    cse_log_status_registers();
    if (CONFIG(VBOOT)) {
-		struct vb2_context *ctx;
-		ctx = vboot_get_context();
+		struct vb2_context *ctx = vboot_get_context();
+		if (ctx == NULL)
+			goto failure;
    	vb2api_fail(ctx, VB2_RECOVERY_INTEL_CSE_LITE_SKU, rec_sub_code);
    	vboot_save_data(ctx);
    	vboot_reboot();
    }
-
+failure:
    die("cse_lite: Failed to trigger recovery mode(recovery subcode:%d)\n", rec_sub_code);
 }