From 9f363fc85aef290d20f7102a16af0df274beadd8 Mon Sep 17 00:00:00 2001 From: Wim Vervoorn Date: Thu, 2 Jan 2020 11:52:59 +0100 Subject: drivers/intel/fsp2_0/logo.c: Correct check for logo_size The check to validate if the logo file was loaded correctly was incorrect. Now check the actual logo size. BUG=N/A TEST=build Change-Id: Ib3a808dd831986e8347512892ee88983d376d34c Signed-off-by: Wim Vervoorn Reviewed-on: https://review.coreboot.org/c/coreboot/+/38124 Reviewed-by: Paul Menzel Reviewed-by: Nico Huber Reviewed-by: Frans Hendriks Tested-by: build bot (Jenkins) --- src/drivers/intel/fsp2_0/logo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/drivers/intel/fsp2_0') diff --git a/src/drivers/intel/fsp2_0/logo.c b/src/drivers/intel/fsp2_0/logo.c index ba2b5dcbcd..1a9152f97b 100644 --- a/src/drivers/intel/fsp2_0/logo.c +++ b/src/drivers/intel/fsp2_0/logo.c @@ -26,7 +26,7 @@ const struct cbmem_entry *fsp_load_logo(UINT32 *logo_ptr, UINT32 *logo_size) if (logo_buffer) { *logo_size = cbfs_boot_load_file("logo.bmp", (void *)logo_buffer, 1 * MiB, CBFS_TYPE_RAW); - if (logo_size) + if (*logo_size) *logo_ptr = (UINT32)logo_buffer; } } -- cgit v1.2.3