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.
This commit is contained in:
John Baldwin 2016-06-11 07:26:09 -07:00
parent 99a54ef6f7
commit bec734b212
2 changed files with 5 additions and 0 deletions

View File

@ -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.

View File

@ -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);