[coreboot-gerrit] New patch to review for coreboot: util/cbmem: Initialize variable

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Jul 29 21:57:30 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/15969

-gerrit

commit 842ec275ea85ddaa0d24b4ca26cc0d43a1d4c460
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Fri Jul 29 21:55:44 2016 +0200

    util/cbmem: Initialize variable
    
    There can be cases where "found" wasn't initialized, do so.
    
    Change-Id: Ifef8d61daa70e27ec39b7a8f3481d2316dfaa36e
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Found-by: Coverity Scan #1347334
---
 util/cbmem/cbmem.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbmem/cbmem.c b/util/cbmem/cbmem.c
index 5e56d11..885d460 100644
--- a/util/cbmem/cbmem.c
+++ b/util/cbmem/cbmem.c
@@ -259,7 +259,7 @@ static struct lb_cbmem_ref parse_cbmem_ref(struct lb_cbmem_ref *cbmem_ref)
 
 static int parse_cbtable(u64 address, size_t table_size, uint8_t abort_on_failure)
 {
-	int i, found, ret = 0;
+	int i, found = 0, ret = 0;
 	void *buf;
 
 	debug("Looking for coreboot table at %" PRIx64 " %zd bytes.\n",



More information about the coreboot-gerrit mailing list