diff options
| author | Hrushiraj Gandhi <hrushirajg23@gmail.com> | 2026-08-31 20:19:53 +0530 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2026-09-29 23:14:53 -0700 |
| commit | c2dde78d53e03fee2de1978b7ea16e50d6705497 (patch) | |
| tree | 92d7597bf7ee895a4f0d9abbee23a73c678bde27 /kernel | |
| parent | 5436671ff5e89e4d9b69184849f7ff7709717409 (diff) | |
| download | linux-next-c2dde78d53e03fee2de1978b7ea16e50d6705497.tar.gz linux-next-c2dde78d53e03fee2de1978b7ea16e50d6705497.zip | |
gcov: use strscpy() instead of strcpy() in init_node()
node->name is a flexible array member sized to exactly strlen(name) + 1
bytes at allocation time in new_node(), so this copy can never actually
overflow. Still, prefer the bounded strscpy() over strcpy() on general
principle; pass the same strlen(name) + 1 bound the allocation used, since
sizeof() cannot be applied to a flexible array member.
No functional change.
Link: https://lore.kernel.org/20260831144953.324441-1-hrushirajg23@gmail.com
Signed-off-by: Hrushiraj Gandhi <hrushirajg23@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Reviewed-by: Bradley Morgan <brads@mainlining.org>
Cc: Peter Oberparleiter <oberpar@linux.ibm.com>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/gcov/fs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/gcov/fs.c b/kernel/gcov/fs.c index 1d19b1be207a..764918570de1 100644 --- a/kernel/gcov/fs.c +++ b/kernel/gcov/fs.c @@ -529,7 +529,7 @@ static void init_node(struct gcov_node *node, struct gcov_info *info, } node->parent = parent; if (name) - strcpy(node->name, name); + strscpy(node->name, name, strlen(name) + 1); } /* |