* avr-tdep.c (avr_io_reg_read_command): Fix to handle case when the

number of io registers reported by remote target is not a multiple of step.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 329e864..b9eee63 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2003-03-04  Theodore A. Roth  <troth@openavr.org>
+
+       * avr-tdep.c (avr_io_reg_read_command): Fix to handle case when the
+       number of io registers reported by remote target is not a multiple of
+       step.
+
 2003-03-04  David Carlton  <carlton@math.stanford.edu>
 
        * symtab.c (lookup_partial_symbol): Add linkage_name argument.

diff --git a/gdb/avr-tdep.c b/gdb/avr-tdep.c
index 952b4e4..975132a 100644 (file)
--- a/gdb/avr-tdep.c
+++ b/gdb/avr-tdep.c
@@ -1279,7 +1279,10 @@ avr_io_reg_read_command (char *args, int from_tty)
 
   for (i = 0; i < nreg; i += step)
     {
-      j = step - (nreg % step);        /* how many registers this round? */
+      /* how many registers this round? */
+      j = step;
+      if ((i+j) >= nreg)
+        j = nreg - i;           /* last block is less than 8 registers */
 
       snprintf (query, sizeof (query) - 1, "avr.io_reg:%x,%x", i, j);
       target_query ((int) 'R', query, buf, &bufsiz);