binutils-gdb/gdbsupport/agent.h
Tom Tromey 81aa19c303 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>
2023-02-14 09:01:18 -07:00

69 lines
2.2 KiB
C

/* Shared utility routines for GDB to interact with agent.
Copyright (C) 2009-2023 Free Software Foundation, Inc.
This file is part of GDB.
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 3 of the License, or
(at your option) any later version.
This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
#ifndef COMMON_AGENT_H
#define COMMON_AGENT_H
#include "gdbsupport/preprocessor.h"
int agent_run_command (int pid, char *cmd, int len);
int agent_look_up_symbols (void *);
#define IPA_SYM_EXPORTED_NAME(SYM) gdb_agent_ ## SYM
/* Define an entry in an IPA symbol list array. If IPA_SYM is used, the macro
IPA_SYM_STRUCT_NAME must be defined to the structure name holding the IPA
symbol addresses in that particular file, before including
gdbsupport/agent.h. */
#define IPA_SYM(SYM) \
{ \
STRINGIFY (IPA_SYM_EXPORTED_NAME (SYM)), \
offsetof (IPA_SYM_STRUCT_NAME, addr_ ## SYM) \
}
/* The size in bytes of the buffer used to talk to the IPA helper
thread. */
#define IPA_CMD_BUF_SIZE 1024
bool agent_loaded_p (void);
extern bool debug_agent;
extern bool use_agent;
/* Capability of agent. Different agents may have different capabilities,
such as installing fast tracepoint or evaluating breakpoint conditions.
Capabilities are represented by bit-maps, and each capability occupies one
bit. */
enum agent_capa
{
/* Capability to install fast tracepoint. */
AGENT_CAPA_FAST_TRACE = 0x1,
/* Capability to install static tracepoint. */
AGENT_CAPA_STATIC_TRACE = (0x1 << 1),
};
bool agent_capability_check (enum agent_capa);
void agent_capability_invalidate (void);
#endif /* COMMON_AGENT_H */