diff options
| author | Pedro Alves <palves@redhat.com> | 2013-09-02 14:14:58 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2013-09-02 14:14:58 +0000 |
| commit | c31e67091e3052399e180e5ba2e22e8999c76003 (patch) | |
| tree | 62b5754e55b7f2b5c81269d01d1b970178b93da9 /gdb/gdbserver/ChangeLog | |
| parent | 2d8bbd00ecdfe7d148a7d24083a4c96eda08a1cd (diff) | |
| download | gdb-c31e67091e3052399e180e5ba2e22e8999c76003.tar.gz | |
server.c:gdb_read_memory: Fix error return.
When I added gdb_read_memory, with bits factored out from elsewhere, I
missed adjusting this error return. gdb_read_memory has an interface
similar to Like GDB's xfer_partial:
> /* Read trace frame or inferior memory. Returns the number of bytes
> actually read, zero when no further transfer is possible, and -1 on
> error. Return of a positive value smaller than LEN does not
> indicate there's no more to be read, only the end of the transfer.
Returning EIO, a positive value, is obviously bogus, for the caller
will confuse it with a successful partial transfer.
Found by inspection.
Tested on x86_64 Fedora 17.
gdb/gdbserver/
2013-09-02 Pedro Alves <palves@redhat.com>
* server.c (gdb_read_memory): Return -1 on traceframe memory read
error instead of EIO.
Diffstat (limited to 'gdb/gdbserver/ChangeLog')
| -rw-r--r-- | gdb/gdbserver/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 913fe42c45f..1e15f77c4c1 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,8 @@ +2013-09-02 Pedro Alves <palves@redhat.com> + + * server.c (gdb_read_memory): Return -1 on traceframe memory read + error instead of EIO. + 2013-08-28 Jan Kratochvil <jan.kratochvil@redhat.com> PR server/15604 |