[coreboot-gerrit] New patch to review for coreboot: 6e9f4ac libpayload: Add missing break statement

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Dec 29 19:32:30 CET 2014


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7970

-gerrit

commit 6e9f4accacd1ab3792dc602e642c049867f2a1a9
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Mon Dec 29 19:31:35 2014 +0100

    libpayload: Add missing break statement
    
    Change-Id: I073b9bd30f4f9b0cdf6c45c10e4549889342d5f3
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Found-by: Coverity Scan
---
 payloads/libpayload/arch/x86/coreboot.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/payloads/libpayload/arch/x86/coreboot.c b/payloads/libpayload/arch/x86/coreboot.c
index 8ccd4cb..a463557 100644
--- a/payloads/libpayload/arch/x86/coreboot.c
+++ b/payloads/libpayload/arch/x86/coreboot.c
@@ -270,6 +270,7 @@ static int cb_parse_header(void *addr, int len, struct sysinfo_t *info)
 #endif
 		case CB_TAG_MAINBOARD:
 			info->mainboard = (struct cb_mainboard *)ptr;
+			break;
 #ifdef CONFIG_LP_CHROMEOS
 		case CB_TAG_GPIO:
 			cb_parse_gpios(ptr, info);



More information about the coreboot-gerrit mailing list