[coreboot-gerrit] New patch to review for coreboot: Fix GCC 5.1 compile issue seen at Linux Arch

Anatol Pomozov (anatol.pomozov@gmail.com) gerrit at coreboot.org
Sat Jul 11 02:37:07 CEST 2015


Anatol Pomozov (anatol.pomozov at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10881

-gerrit

commit 552f80ea2c12358e91d22a6b3a9a9764ce60bb8d
Author: Anatol Pomozov <anatol.pomozov at gmail.com>
Date:   Fri Jul 10 17:30:01 2015 -0700

    Fix GCC 5.1 compile issue seen at Linux Arch
    
    rmodule.c: In function ‘rmodule_create’:
    rmodule.c:287:29: error: ‘phdr’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
            (phdr->p_vaddr + phdr->p_memsz))) {
                                 ^
    rmodule.c:204:14: note: ‘phdr’ was declared here
      Elf64_Phdr *phdr;
                  ^
    
    Change-Id: I94a235253610348484eef218ec855103a3bb5da5
    Signed-off-by: Anatol Pomozov <anatol.pomozov at gmail.com>
---
 util/cbfstool/rmodule.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbfstool/rmodule.c b/util/cbfstool/rmodule.c
index b94e0c2..f93f4f6 100644
--- a/util/cbfstool/rmodule.c
+++ b/util/cbfstool/rmodule.c
@@ -201,7 +201,7 @@ static int find_program_segment(struct rmod_context *ctx)
 	int i;
 	int nsegments;
 	struct parsed_elf *pelf;
-	Elf64_Phdr *phdr;
+	Elf64_Phdr *phdr = NULL;
 
 	pelf = &ctx->pelf;
 



More information about the coreboot-gerrit mailing list