[v3,06/24] Fix "set enum-command value junk"

Message ID 20190604223444.26472-7-palves@redhat.com
State New
Headers show
  • gdb::option framework, "print -OPT", other cmd options
Related show

Commit Message

Pedro Alves June 4, 2019, 10:34 p.m.
With enum commands, we currently fail to notice junk after the value.


  (gdb) set print entry-values compact foo
  (gdb) show print entry-values foo
  Printing of function arguments at function entry is "compact".

After this fix:

 (gdb) set print entry-values compact foo
  Junk after item "compact": foo

yyyy-mm-dd  Pedro Alves  <palves@redhat.com>

	* cli/cli-setshow.c (do_set_command) <var_enum>: Detect junk
	after item.
 gdb/cli/cli-setshow.c | 4 ++++
 1 file changed, 4 insertions(+)



diff --git a/gdb/cli/cli-setshow.c b/gdb/cli/cli-setshow.c
index 96d7bf5c3c0..86ebed45a4a 100644
--- a/gdb/cli/cli-setshow.c
+++ b/gdb/cli/cli-setshow.c
@@ -413,6 +413,10 @@  do_set_command (const char *arg, int from_tty, struct cmd_list_element *c)
 	if (nmatches > 1)
 	  error (_("Ambiguous item \"%s\"."), arg);
+	const char *after = skip_spaces (arg + len);
+	if (*after != '\0')
+	  error (_("Junk after item \"%.*s\": %s"), len, arg, after);
 	if (*(const char **) c->var != match)
 	    *(const char **) c->var = match;