diff options
| author | Andreas Krebbel <krebbel@linux.vnet.ibm.com> | 2014-02-21 15:55:37 +0100 |
|---|---|---|
| committer | Andreas Krebbel <krebbel@linux.vnet.ibm.com> | 2014-02-21 15:55:37 +0100 |
| commit | c63528fc47c79721b55f32fe1649762ff9125ae0 (patch) | |
| tree | a436ebfe5cb706632dc7477ba7a0b17fc2bb7a0b | |
| parent | ea4758f2dd33c0081abc89b011c4755453af9efc (diff) | |
| download | gdb-c63528fc47c79721b55f32fe1649762ff9125ae0.zip gdb-c63528fc47c79721b55f32fe1649762ff9125ae0.tar.gz gdb-c63528fc47c79721b55f32fe1649762ff9125ae0.tar.bz2 | |
Fix length arg in call to breakpoint_xfer_memory.
The patch "return target_xfer_status in to_xfer_partial" caused a
regression in various s390(x) test cases, because memory_xfer_partial
filled only the first byte of the read buffer from a breakpoint shadow:
https://sourceware.org/ml/gdb-patches/2014-01/msg01071.html
This patch fixes the regression.
| -rw-r--r-- | gdb/ChangeLog | 5 | ||||
| -rw-r--r-- | gdb/target.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ac3844f..9406d76 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2014-02-21 Andreas Arnez <arnez@vnet.linux.ibm.com> + + * target.c (memory_xfer_partial): Fix length arg in call to + breakpoint_xfer_memory. + 2014-02-20 Sergio Durigan Junior <sergiodj@redhat.com> PR tdep/16397 diff --git a/gdb/target.c b/gdb/target.c index 4adc094..0f3bd30 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -1348,7 +1348,7 @@ memory_xfer_partial (struct target_ops *ops, enum target_object object, xfered_len); if (res == TARGET_XFER_OK && !show_memory_breakpoints) - breakpoint_xfer_memory (readbuf, NULL, NULL, memaddr, res); + breakpoint_xfer_memory (readbuf, NULL, NULL, memaddr, *xfered_len); } else { |