[coreboot-gerrit] New patch to review for coreboot: 6dad95c lib/selfboot: s_srcaddr is uninitialized.

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Fri Feb 28 23:34:29 CET 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5314

-gerrit

commit 6dad95cd87ecfe03055c49721c877a459da7fd28
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat Mar 1 09:27:37 2014 +1100

    lib/selfboot: s_srcaddr is uninitialized.
    
    s_srcaddr is uninitialized in the BSS section, leading to a
    garbage valued operand on the LHS of a '<' on line 383.
    
    Change-Id: Ie4fec91b09c70fb1d91ad3918ac3f60653fa1d83
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/lib/selfboot.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/lib/selfboot.c b/src/lib/selfboot.c
index 222eae2..2d12a2b 100644
--- a/src/lib/selfboot.c
+++ b/src/lib/selfboot.c
@@ -352,6 +352,7 @@ static int build_self_segment_list(
 				 	ntohl(segment->mem_len));
 			new = malloc(sizeof(*new));
 			new->s_filesz = 0;
+			new->s_srcaddr = NULL;
 			new->s_dstaddr = ntohll(segment->load_addr);
 			new->s_memsz = ntohl(segment->mem_len);
 			break;



More information about the coreboot-gerrit mailing list