diff options
| author | Michael Snyder <msnyder@vmware.com> | 2011-02-22 18:45:05 +0000 |
|---|---|---|
| committer | Michael Snyder <msnyder@vmware.com> | 2011-02-22 18:45:05 +0000 |
| commit | 298f437a6b0631636cd55d72b8d98d862fc8c51e (patch) | |
| tree | ca84f5d1093334090f2f76cc2a1a91266cd8ea17 | |
| parent | b7ea3126c02915e913befc54ce6bebb6446efef6 (diff) | |
| download | gdb-298f437a6b0631636cd55d72b8d98d862fc8c51e.zip gdb-298f437a6b0631636cd55d72b8d98d862fc8c51e.tar.gz gdb-298f437a6b0631636cd55d72b8d98d862fc8c51e.tar.bz2 | |
2011-02-22 Michael Snyder <msnyder@vmware.com>
* cli/cli-utils.c (number_is_in_list): Check for zero return.
| -rw-r--r-- | gdb/ChangeLog | 4 | ||||
| -rw-r--r-- | gdb/cli/cli-utils.c | 11 |
2 files changed, 12 insertions, 3 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1859bf0..2451004 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2011-02-22 Michael Snyder <msnyder@vmware.com> + + * cli/cli-utils.c (number_is_in_list): Check for zero return. + 2011-02-22 Pedro Alves <pedro@codesourcery.com> * frame-unwind.h: Fix comment to mention the this frame, not the diff --git a/gdb/cli/cli-utils.c b/gdb/cli/cli-utils.c index 8a7e5d4..34c368b 100644 --- a/gdb/cli/cli-utils.c +++ b/gdb/cli/cli-utils.c @@ -175,10 +175,15 @@ number_is_in_list (char *list, int number) if (list == NULL || *list == '\0') return 1; - while (list != NULL && *list != '\0') - if (get_number_or_range (&list) == number) - return 1; + while (*list != '\0') + { + int gotnum = get_number_or_range (&list); + if (gotnum == 0) + error (_("Args must be numbers or '$' variables.")); + if (gotnum == number) + return 1; + } return 0; } |