diff options
| author | Tom Tromey <tromey@adacore.com> | 2023-02-14 07:03:11 -0700 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-02-14 09:01:18 -0700 |
| commit | 81aa19c303c94f549cb9ae343cfe4b635b4e888c (patch) | |
| tree | 804ffe2eec706e0e18732b5724ebcf5d737f9e7a /gdbsupport/agent.h | |
| parent | 5bed9dc992a0136d403a7addb29a2ed822fd4fd2 (diff) | |
| download | gdb-81aa19c303c94f549cb9ae343cfe4b635b4e888c.zip gdb-81aa19c303c94f549cb9ae343cfe4b635b4e888c.tar.gz gdb-81aa19c303c94f549cb9ae343cfe4b635b4e888c.tar.bz2 | |
Do not cast away const in agent_run_command
While investigating something else, I noticed some weird code in
agent_run_command (use of memcpy rather than strcpy). Then I noticed
that 'cmd' is used as both an in and out parameter, despite being
const.
Casting away const like this is bad. This patch removes the const and
fixes the memcpy. I also added a static assert to assure myself that
the code in gdbserver is correct -- gdbserver is passing its own
buffer directly to agent_run_command.
Reviewed-By: Andrew Burgess <aburgess@redhat.com>
Diffstat (limited to 'gdbsupport/agent.h')
| -rw-r--r-- | gdbsupport/agent.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdbsupport/agent.h b/gdbsupport/agent.h index dceb33f..7a258e2 100644 --- a/gdbsupport/agent.h +++ b/gdbsupport/agent.h @@ -22,7 +22,7 @@ #include "gdbsupport/preprocessor.h" -int agent_run_command (int pid, const char *cmd, int len); +int agent_run_command (int pid, char *cmd, int len); int agent_look_up_symbols (void *); |