diff options
| author | Andrew Burgess <andrew.burgess@embecosm.com> | 2021-01-28 17:17:08 +0000 |
|---|---|---|
| committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2021-02-08 11:01:07 +0000 |
| commit | 93a01471f3df559fffc567247288b3301620c4d7 (patch) | |
| tree | ca8460281314070cdfc55776b9aee79056414a3d /sim/rx/gdb-if.c | |
| parent | 0309f9549d8ab359f60790427ca97579dd26eafb (diff) | |
sim/rx: fix an issue where we try to modify a const string
While experimenting with switching on warnings for the rx simulator I
discovered this bug. In sim_do_command we get passed a 'const char *'
argument. We create a copy of this string to work with locally, but
then while processing this we accidentally switch back to reference
the original string.
sim/rx/ChangeLog:
* gdb-if.c (sim_do_command): Work with a copy of the command.
Diffstat (limited to 'sim/rx/gdb-if.c')
| -rw-r--r-- | sim/rx/gdb-if.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sim/rx/gdb-if.c b/sim/rx/gdb-if.c index 6f8bfbd1c5..55eb13d12e 100644 --- a/sim/rx/gdb-if.c +++ b/sim/rx/gdb-if.c @@ -804,13 +804,13 @@ sim_do_command (SIM_DESC sd, const char *cmd) p++; /* Find the extent of the command word. */ - for (p = cmd; *p; p++) + for (; *p != '\0'; p++) if (isspace (*p)) break; /* Null-terminate the command word, and record the start of any further arguments. */ - if (*p) + if (*p != '\0') { *p = '\0'; args = p + 1; |