tests: ensure backtrace.c exec_dump sets jmp

With CFLAGS='-Og -g', F21 gcc hits -Werror=maybe-uninitialized on jmp in
the call "prepare_thread (pid2, jmp)".  It's fine with -O2/-O0.

The only way this could be unset is if the loop before ran to
completion, and there's already an assert against this case.

This patch initializes jmp anyway to convince gcc it's ok.

Signed-off-by: Josh Stone <jistone@redhat.com>

diff --git a/tests/ChangeLog b/tests/ChangeLog
index 5091a9b7bf62cedd46d823dfcd342899e13f1097..9d3290f27bcb39ad9ecba3eae17c8786811aec54 100644 (file)
--- a/tests/ChangeLog
+++ b/tests/ChangeLog
@@ -1,3 +1,7 @@
+2015-02-11  Josh Stone  <jistone@redhat.com>
+
+       * backtrace.c (exec_dump): Initialize jmp.
+
 2015-02-11  Petr Machata  <pmachata@redhat.com>
 
        * run-dwarf-ranges.sh: New test.

diff --git a/tests/backtrace.c b/tests/backtrace.c
index 331ba0f2bed55c27e90b6f039723694266165c57..e109654f5e41dca2acf0261c1236bc67c58a0107 100644 (file)
--- a/tests/backtrace.c
+++ b/tests/backtrace.c
@@ -377,7 +377,7 @@ exec_dump (const char *exec)
 #else /* __x86_64__ */
   is_x86_64_native = ehdr->e_ident[EI_CLASS] == ELFCLASS64;
 #endif /* __x86_64__ */
-  void (*jmp) (void);
+  void (*jmp) (void) = 0;
   if (is_x86_64_native)
     {
       // Find inferior symbol named "jmp".