aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarcin Kościelnicki <koriakin@0x04.net>2016-02-06 16:30:01 +0100
committerMarcin Kościelnicki <koriakin@0x04.net>2016-02-10 14:50:22 +0100
commit473b99e5721ad623cc5826e03a73e2cf6cef4eb5 (patch)
tree9c15e2a648d6ad0423cb9547740562bc13d7b3fc
parent4670103e86f59a80259fd593a6949d693382e536 (diff)
downloadgdb-473b99e5721ad623cc5826e03a73e2cf6cef4eb5.zip
gdb-473b99e5721ad623cc5826e03a73e2cf6cef4eb5.tar.gz
gdb-473b99e5721ad623cc5826e03a73e2cf6cef4eb5.tar.bz2
gdb.trace: Fix off-by-one in tfile_fetch_registers.
This resulted in the last register being considered unavailable. On plain x86_64 (without AVX), this happened to be orig_rax. gdb/ChangeLog: * tracefile-tfile.c (tfile_fetch_registers): Fix off-by-one in bounds check.
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/tracefile-tfile.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1bca0d7..9f8aa1d 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2016-02-10 Marcin Kościelnicki <koriakin@0x04.net>
+
+ * tracefile-tfile.c (tfile_fetch_registers): Fix off-by-one in bounds
+ check.
+
2016-02-10 Joel Brobecker <brobecker@adacore.com>
* NEWS: Create a new section for the next release branch.
diff --git a/gdb/tracefile-tfile.c b/gdb/tracefile-tfile.c
index b761894..dc7b05a 100644
--- a/gdb/tracefile-tfile.c
+++ b/gdb/tracefile-tfile.c
@@ -816,7 +816,7 @@ tfile_fetch_registers (struct target_ops *ops,
{
regsize = register_size (gdbarch, regn);
/* Make sure we stay within block bounds. */
- if (offset + regsize >= trace_regblock_size)
+ if (offset + regsize > trace_regblock_size)
break;
if (regcache_register_status (regcache, regn) == REG_UNKNOWN)
{