gdb/tui: avoid theoretical bug with 'tui reg' command

While looking at the 'tui reg' command as part of another patch, I
spotted a theoretical bug.

The 'tui reg' command takes the name of a register group, but also
handles partial register group matches, though the partial match has to
be unique.  The current command logic goes:

With the code as currently written, if a target description named a
register group either 'prev' or 'next' then GDB would see this as an
ambiguous register name, and refuse to switch groups.

Naming a register group 'prev' or 'next' seems pretty unlikely, but,
by adding a single else block we can prevent this problem.

Now, if there's a 'prev' or 'next' register group, the user will not
be able to select the group directly, the 'prev' and 'next' names will
always iterate through the available groups instead.  But at least the
user could select their groups by iteration, rather than direct
selection.
This commit is contained in:
Andrew Burgess 2022-03-31 14:44:25 +01:00
parent 2b72890eba
commit 5783701b36

View File

@ -580,7 +580,8 @@ tui_reg_command (const char *args, int from_tty)
match = tui_reg_next (current_group, gdbarch); match = tui_reg_next (current_group, gdbarch);
else if (strncmp (args, "prev", len) == 0) else if (strncmp (args, "prev", len) == 0)
match = tui_reg_prev (current_group, gdbarch); match = tui_reg_prev (current_group, gdbarch);
else
{
/* This loop matches on the initial part of a register group /* This loop matches on the initial part of a register group
name. If this initial part in ARGS matches only one register name. If this initial part in ARGS matches only one register
group then the switch is made. */ group then the switch is made. */
@ -595,6 +596,7 @@ tui_reg_command (const char *args, int from_tty)
match = group; match = group;
} }
} }
}
if (match == NULL) if (match == NULL)
error (_("unknown register group '%s'"), args); error (_("unknown register group '%s'"), args);