github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-07-27 17:47:41 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

riscv: ptdump: Replace unbounded sprintf() in dump_prot()

Browse Source 
Replace the unbounded sprintf("%s", ...) with the faster and safer
strscpy(). Replace all other sprintf() calls with the safer snprintf().

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20260127110543.436242-1-thorsten.blum@linux.dev
Signed-off-by: Paul Walmsley <pjw@kernel.org>
This commit is contained in:
Thorsten Blum 2026-06-06 20:17:52 -06:00 committed by Paul Walmsley
parent a24c7f3523
commit 1a9e47c2c3
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
arch/riscv/mm/ptdump.c
View File

@ -7,6 +7,7 @@
#include <linux/init.h>
#include <linux/debugfs.h>
#include <linux/seq_file.h>
#include <linux/string.h>
#include <linux/ptdump.h>
#include <linux/pgtable.h>
@ -213,21 +214,21 @@ static void dump_prot(struct pg_state *st)
val = st->current_prot & pte_bits[i].mask;
if (val) {
if (pte_bits[i].mask == _PAGE_SOFT)
sprintf(s, pte_bits[i].set, val >> 8);
snprintf(s, sizeof(s), pte_bits[i].set, val >> 8);
#ifdef CONFIG_64BIT
else if (pte_bits[i].mask == _PAGE_MTMASK_SVPBMT) {
if (val == _PAGE_NOCACHE_SVPBMT)
sprintf(s, pte_bits[i].set, "NC");
snprintf(s, sizeof(s), pte_bits[i].set, "NC");
else if (val == _PAGE_IO_SVPBMT)
sprintf(s, pte_bits[i].set, "IO");
snprintf(s, sizeof(s), pte_bits[i].set, "IO");
else
sprintf(s, pte_bits[i].set, "??");
snprintf(s, sizeof(s), pte_bits[i].set, "??");
}
#endif
else
sprintf(s, "%s", pte_bits[i].set);
strscpy(s, pte_bits[i].set);
} else {
sprintf(s, "%s", pte_bits[i].clear);
strscpy(s, pte_bits[i].clear);
}
pt_dump_seq_printf(st->seq, " %s", s);