diff options
author | Jim Blandy <jimb@codesourcery.com> | 2001-12-20 08:55:31 +0000 |
---|---|---|
committer | Jim Blandy <jimb@codesourcery.com> | 2001-12-20 08:55:31 +0000 |
commit | 9233e27e3b6ddd05fd4210b6758b4ed8fcf1b1af (patch) | |
tree | fa28cbfeed0c758adb43954a6854e05499a71e8a /gdb | |
parent | c6fabecf08b2dc3d18f4742a8bd3a6df8b733b54 (diff) | |
download | gdb-9233e27e3b6ddd05fd4210b6758b4ed8fcf1b1af.tar.gz |
* s390-tdep.c (s390_pop_frame_regular): Make sure the saved_regs
array is actually set before we try to use it.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 3 | ||||
-rw-r--r-- | gdb/s390-tdep.c | 29 |
2 files changed, 19 insertions, 13 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 432c507acf6..b3823897d74 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,8 @@ 2001-12-19 Jim Blandy <jimb@redhat.com> + * s390-tdep.c (s390_pop_frame_regular): Make sure the saved_regs + array is actually set before we try to use it. + * s390-tdep.c (s390_frame_saved_pc_nofix): If we get the saved PC out of the return address register, cache that in the frame's extra info, just as if we'd gotten it from the saved regs array; diff --git a/gdb/s390-tdep.c b/gdb/s390-tdep.c index c524657a57a..7b4972b097d 100644 --- a/gdb/s390-tdep.c +++ b/gdb/s390-tdep.c @@ -1225,20 +1225,23 @@ s390_pop_frame_regular (struct frame_info *frame) write_register (S390_PC_REGNUM, FRAME_SAVED_PC (frame)); /* Restore any saved registers. */ - for (regnum = 0; regnum < NUM_REGS; regnum++) - if (frame->saved_regs[regnum] != 0) - { - ULONGEST value; - - value = read_memory_unsigned_integer (frame->saved_regs[regnum], - REGISTER_RAW_SIZE (regnum)); - write_register (regnum, value); - } + if (frame->saved_regs) + { + for (regnum = 0; regnum < NUM_REGS; regnum++) + if (frame->saved_regs[regnum] != 0) + { + ULONGEST value; + + value = read_memory_unsigned_integer (frame->saved_regs[regnum], + REGISTER_RAW_SIZE (regnum)); + write_register (regnum, value); + } - /* Actually cut back the stack. Remember that the SP's element of - saved_regs is the old SP itself, not the address at which it is - saved. */ - write_register (S390_SP_REGNUM, frame->saved_regs[S390_SP_REGNUM]); + /* Actually cut back the stack. Remember that the SP's element of + saved_regs is the old SP itself, not the address at which it is + saved. */ + write_register (S390_SP_REGNUM, frame->saved_regs[S390_SP_REGNUM]); + } /* Throw away any cached frame information. */ flush_cached_frames (); |