Fix build failure in symfile.c::unmap_overlay_command (GCC5 bug)
authorAlexander Klimov <alserkli@inbox.ru>
Tue, 27 Jan 2015 17:56:45 +0000 (19:56 +0200)
committerJoel Brobecker <brobecker@adacore.com>
Thu, 29 Jan 2015 07:28:02 +0000 (11:28 +0400)
Compilation of (GDB) 7.9.50.20150127-cvs with (GCC) 5.0.0 20150127
fails with

In file included from symfile.c:32:0:
symfile.c: In function 'unmap_overlay_command':
objfiles.h:628:3: error: 'sec' may be used uninitialized in this
function [-Werror=maybe-uninitialized]
   for (osect = objfile->sections; osect < objfile->sections_end; osect++) \
   ^
symfile.c:3442:23: note: 'sec' was declared here
   struct obj_section *sec;
                       ^
cc1: all warnings being treated as errors
make[2]: *** [symfile.o] Error 1
make[2]: Leaving directory `gdb/gdb'

While the bug was reported to GCC as
<https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64823>,
the attached patch simply initializes sec with NULL.

gdb/ChangeLog:

        * symfile.c (unmap_overlay_command): Initialize sec to NULL.

Tested on x86_64-linux.

gdb/ChangeLog
gdb/symfile.c

index 03b871239b61f50301c814ef99af6065352af774..db4e0b146dce077460fe5b7c2c8143163927b919 100644 (file)
@@ -1,3 +1,7 @@
+2015-01-29  Joel Brobecker  <brobecker@adacore.com>  (tiny patch)
+
+       * symfile.c (unmap_overlay_command): Initialize sec to NULL.
+
 2015-01-27  Doug Evans  <dje@google.com>
 
        * NEWS: Mention gdb.Objfile.username.
index d55e36186c046ae1ff36763d6d4a8164d65c5f50..86a758a52cd61245eee2dacc05d7852cd19e9f51 100644 (file)
@@ -3439,7 +3439,7 @@ static void
 unmap_overlay_command (char *args, int from_tty)
 {
   struct objfile *objfile;
-  struct obj_section *sec;
+  struct obj_section *sec = NULL;
 
   if (!overlay_debugging)
     error (_("Overlay debugging not enabled.  "