[coreboot-gerrit] Patch set updated for coreboot: soc/intel/quark: Fix uninitialized variable d_variant

Leroy P Leahy (leroy.p.leahy@intel.com) gerrit at coreboot.org
Thu Apr 21 20:21:22 CEST 2016


Leroy P Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14338

-gerrit

commit 576432c122692acf485ad6266c1298eaf1630e02
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Tue Apr 12 13:37:02 2016 -0700

    soc/intel/quark: Fix uninitialized variable d_variant
    
    Initialize the d_variant variable.
    
    Found-by: CID 1353356 Uninitialized variable
    
    TEST=Build and run on Galileo Gen2
    
    Change-Id: I26fba4e77f91d53b6ff9028669aa0186d3174639
    Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
---
 src/soc/intel/quark/romstage/report_platform.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/soc/intel/quark/romstage/report_platform.c b/src/soc/intel/quark/romstage/report_platform.c
index 7ede746..5ed8a5f 100644
--- a/src/soc/intel/quark/romstage/report_platform.c
+++ b/src/soc/intel/quark/romstage/report_platform.c
@@ -86,6 +86,7 @@ static void report_cpu_info(void)
 		& B_ROM_FUSE_IN_SECURE_SKU));
 
 	/* TODO: Determine if this is a D variant */
+	d_variant = 0;
 	if (ecc_enabled && secure_boot)
 		d_variant = 0;	/* or 1 */
 



More information about the coreboot-gerrit mailing list