Patrick Georgi (pgeorgi@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@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@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);