diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2025-10-11 02:47:47 +0200 |
|---|---|---|
| committer | Paul Walmsley <pjw@kernel.org> | 2025-10-27 23:30:01 -0600 |
| commit | 5228ed2c624449d1a53d1c84ba01c021d2df95c0 (patch) | |
| tree | a0af8f79cd806246b11a9a20eedc8d1ea5cc732e /arch/riscv/kernel/kgdb.c | |
| parent | riscv: asm: use .insn for making custom instructions (diff) | |
| download | linux-5228ed2c624449d1a53d1c84ba01c021d2df95c0.tar.gz linux-5228ed2c624449d1a53d1c84ba01c021d2df95c0.zip | |
riscv: KGDB: Replace deprecated strcpy in kgdb_arch_handle_qxfer_pkt
strcpy() is deprecated because it can cause a buffer overflow when the
sizes of the source and the destination are not known at compile time.
Use strscpy() instead.
Link: https://github.com/KSPP/linux/issues/88
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://lore.kernel.org/r/20251011004750.461954-1-thorsten.blum@linux.dev
Signed-off-by: Paul Walmsley <pjw@kernel.org>
Diffstat (limited to '')
| -rw-r--r-- | arch/riscv/kernel/kgdb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/riscv/kernel/kgdb.c b/arch/riscv/kernel/kgdb.c index 9f3db3503dab..15fec5d1e6de 100644 --- a/arch/riscv/kernel/kgdb.c +++ b/arch/riscv/kernel/kgdb.c @@ -265,10 +265,10 @@ void kgdb_arch_handle_qxfer_pkt(char *remcom_in_buffer, { if (!strncmp(remcom_in_buffer, gdb_xfer_read_target, sizeof(gdb_xfer_read_target))) - strcpy(remcom_out_buffer, riscv_gdb_stub_target_desc); + strscpy(remcom_out_buffer, riscv_gdb_stub_target_desc, BUFMAX); else if (!strncmp(remcom_in_buffer, gdb_xfer_read_cpuxml, sizeof(gdb_xfer_read_cpuxml))) - strcpy(remcom_out_buffer, riscv_gdb_stub_cpuxml); + strscpy(remcom_out_buffer, riscv_gdb_stub_cpuxml, BUFMAX); } static inline void kgdb_arch_update_addr(struct pt_regs *regs, |