elfutils: fix build with -Os

(From OE-Core rev: c46661bf4ffe1a474e4bcfadb4a3f12a46e8cc4a)

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Martin Jansa 2015-04-10 12:18:28 +02:00 committed by Richard Purdie
parent 1b4f7174c1
commit 31b2c1ff88
2 changed files with 36 additions and 0 deletions

View File

@ -0,0 +1,35 @@
From 668accf322fd7185e273bfd50b84320e71d9de5a Mon Sep 17 00:00:00 2001
From: Martin Jansa <Martin.Jansa@gmail.com>
Date: Fri, 10 Apr 2015 00:29:18 +0200
Subject: [PATCH] elf_getarsym: Silence -Werror=maybe-uninitialized false
positive
Upstream-Status: Pending
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
libelf/elf_getarsym.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/libelf/elf_getarsym.c b/libelf/elf_getarsym.c
index d0bb28a..08954d2 100644
--- a/libelf/elf_getarsym.c
+++ b/libelf/elf_getarsym.c
@@ -165,8 +165,13 @@ elf_getarsym (elf, ptr)
int w = index64_p ? 8 : 4;
/* We have an archive. The first word in there is the number of
- entries in the table. */
- uint64_t n;
+ entries in the table.
+ Set to SIZE_MAX just to silence -Werror=maybe-uninitialized
+ elf_getarsym.c:290:9: error: 'n' may be used uninitialized in this function
+ The read_number_entries function doesn't initialize n only when returning
+ -1 which in turn ensures to jump over usage of this uninitialized variable.
+ */
+ uint64_t n = SIZE_MAX;
size_t off = elf->start_offset + SARMAG + sizeof (struct ar_hdr);
if (read_number_entries (&n, elf, &off, index64_p) < 0)
{
--
2.3.5

View File

@ -15,6 +15,7 @@ SRC_URI += "\
file://dso-link-change.patch \
file://Fix_elf_cvt_gunhash.patch \
file://fixheadercheck.patch \
file://0001-elf_getarsym-Silence-Werror-maybe-uninitialized-fals.patch \
"
# pick the patch from debian