[coreboot-gerrit] New patch to review for coreboot: 4a68515 libpayload: Remove useless continue after return

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Dec 29 19:32:29 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/7969

-gerrit

commit 4a685159b177716e763cd4ba3ef7589005aaa47a
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Mon Dec 29 19:30:37 2014 +0100

    libpayload: Remove useless continue after return
    
    Change-Id: I7209ceaf4ad686bbb07eb0c6105f77834d5511fd
    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 deletion(-)

diff --git a/payloads/libpayload/arch/x86/coreboot.c b/payloads/libpayload/arch/x86/coreboot.c
index 320b8c2..8ccd4cb 100644
--- a/payloads/libpayload/arch/x86/coreboot.c
+++ b/payloads/libpayload/arch/x86/coreboot.c
@@ -217,7 +217,6 @@ static int cb_parse_header(void *addr, int len, struct sysinfo_t *info)
 		case CB_TAG_FORWARD:
 			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