diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2026-04-01 17:20:40 +0200 |
|---|---|---|
| committer | Kees Cook <kees@kernel.org> | 2026-04-01 12:26:07 -0700 |
| commit | 10cd6758e054e4002ccb409fef7dd2c6b7bbd549 (patch) | |
| tree | 8e4c7907372a2a8980cbf77731dca811d0defe3b | |
| parent | 4ced4cf5c9d172d91f181df3accdf949d3761aab (diff) | |
exec: use strnlen() in __set_task_comm
Use strnlen() to limit source string scanning to 'TASK_COMM_LEN - 1'
bytes.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Reviewed-by: Jan Kara <jack@suse.cz>
Link: https://patch.msgid.link/20260401152039.724811-3-thorsten.blum@linux.dev
Signed-off-by: Kees Cook <kees@kernel.org>
| -rw-r--r-- | fs/exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/exec.c b/fs/exec.c index 9ea3a775d51e..ba12b4c466f6 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -1074,7 +1074,7 @@ static int unshare_sighand(struct task_struct *me) */ void __set_task_comm(struct task_struct *tsk, const char *buf, bool exec) { - size_t len = min(strlen(buf), sizeof(tsk->comm) - 1); + size_t len = strnlen(buf, sizeof(tsk->comm) - 1); trace_task_rename(tsk, buf); memcpy(tsk->comm, buf, len); |