XCNEW gdbarch_tdep in rl78 and rx
authorYao Qi <yao.qi@linaro.org>
Thu, 13 Apr 2017 07:29:43 +0000 (08:29 +0100)
committerYao Qi <yao.qi@linaro.org>
Thu, 13 Apr 2017 07:29:43 +0000 (08:29 +0100)
commite6ddc3bfedb4665c9d4baa2c85037af25167cdf3
tree2cfee48ab641a2e953b6469564691a2d569e11da
parentbfb8cf9091a174b42beeff3d014173084413af4d
XCNEW gdbarch_tdep in rl78 and rx

"struct gdbarch_tdep" is XNEW'ed in rl78 and rx, so the memory is not
cleared.  As the result, tdep->rl78_psw_type is never initialized
properly.

  struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);

  if (tdep->rl78_psw_type == NULL)
    {
      tdep->rl78_psw_type = arch_flags_type (gdbarch,
     "builtin_type_rl78_psw", 1);

The bug is found by my unit test in the following patch.

gdb:

2017-04-13  Yao Qi  <yao.qi@linaro.org>

* rl78-tdep.c (rl78_gdbarch_init): Use XCNEW instead of XNEW.
* rx-tdep.c (rx_gdbarch_init): Likewise.
gdb/ChangeLog
gdb/rl78-tdep.c
gdb/rx-tdep.c
This page took 0.026499 seconds and 4 git commands to generate.