[coreboot-gerrit] New patch to review for coreboot: libpayload: Drop superfluous "continue"

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Jul 29 16:34:40 CEST 2016


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

-gerrit

commit fdd76a9d2c55428c9698fa5c9440ac74beabb4ef
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Fri Jul 29 16:33:48 2016 +0200

    libpayload: Drop superfluous "continue"
    
    Change-Id: I5a1d1ce8ba268b08d1275f392f0b9e602860c6ab
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Found-by: Coverity Scan #1260729
---
 payloads/libpayload/libc/coreboot.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/payloads/libpayload/libc/coreboot.c b/payloads/libpayload/libc/coreboot.c
index 7ad8e06..9fc6a69 100644
--- a/payloads/libpayload/libc/coreboot.c
+++ b/payloads/libpayload/libc/coreboot.c
@@ -291,7 +291,6 @@ int cb_parse_header(void *addr, int len, struct sysinfo_t *info)
 			forward = phys_to_virt((void *)(unsigned long)
 					       ((struct cb_forward *)rec)->forward);
 			return cb_parse_header(forward, len, info);
-			continue;
 		case CB_TAG_MEMORY:
 			cb_parse_memory(ptr, info);
 			break;



More information about the coreboot-gerrit mailing list