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

[Edward O'Callaghan]

Edward O'Callaghan ([e-mail]) 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 
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 
---
 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;