Initialize 'ra' to zero to avoid uninitialized use.

If the instruction in this case does not include an RA field, then 'ra'
is used uninitialized.  Use the same idiom used elsewhere in this file of
initializing ra to zero before check for an RA field.

gdb/ChangeLog:

	* rs6000-tdep.c (ppc_process_record_op31): Initialize ra.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1bd309f3e2b9829a3fe3093a075d866e8471d797..c0d4a3f2b7b4560adadd5d693afddd0f11c88c51 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2016-06-14  John Baldwin  <jhb@FreeBSD.org>
+
+       * rs6000-tdep.c (ppc_process_record_op31): Initialize ra.
+
 2016-06-13  Nick Clifton  <nickc@redhat.com>
 
        * gdbtypes.c (replace_type): Fix assertion.

diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c
index 645fe9245c6b715be5155f965293e455517f0eec..1ed1354f2d605a6079f6ac1824b4160cbcf02c68 100644 (file)
--- a/gdb/rs6000-tdep.c
+++ b/gdb/rs6000-tdep.c
@@ -4631,6 +4631,7 @@ ppc_process_record_op31 (struct gdbarch *gdbarch, struct regcache *regcache,
          || at_dcsz == 0)
        at_dcsz = 128; /* Assume 128-byte cache line size (POWER8)  */
 
+      ra = 0;
       if (PPC_RA (insn) != 0)
        regcache_raw_read_unsigned (regcache,
                                    tdep->ppc_gp0_regnum + PPC_RA (insn), &ra);