From: Azeem Shaikh Date: Thu, 31 Aug 2023 19:42:12 +0000 (+0000) Subject: tracing: Replace strlcpy with strscpy in trace/events/task.h X-Git-Tag: v6.6.17~3930^2~25 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2933d3cd079d3bf6fded709de7d97c1dc71d9633;p=platform%2Fkernel%2Flinux-rpi.git tracing: Replace strlcpy with strscpy in trace/events/task.h strlcpy() reads the entire source buffer first. This read may exceed the destination size limit. This is both inefficient and can lead to linear read overflows if a source string is not NUL-terminated [1]. In an effort to remove strlcpy() completely [2], replace strlcpy() here with strscpy(). No return values were used, so direct replacement is safe. [1] https://www.kernel.org/doc/html/latest/process/deprecated.html#strlcpy [2] https://github.com/KSPP/linux/issues/89 Link: https://lore.kernel.org/linux-trace-kernel/20230831194212.1529941-1-azeemshaikh38@gmail.com Cc: Masami Hiramatsu Reviewed-by: Kees Cook Signed-off-by: Azeem Shaikh Signed-off-by: Steven Rostedt (Google) --- diff --git a/include/trace/events/task.h b/include/trace/events/task.h index 64d1609..47b5274 100644 --- a/include/trace/events/task.h +++ b/include/trace/events/task.h @@ -47,7 +47,7 @@ TRACE_EVENT(task_rename, TP_fast_assign( __entry->pid = task->pid; memcpy(entry->oldcomm, task->comm, TASK_COMM_LEN); - strlcpy(entry->newcomm, comm, TASK_COMM_LEN); + strscpy(entry->newcomm, comm, TASK_COMM_LEN); __entry->oom_score_adj = task->signal->oom_score_adj; ),