linux-kernel - 回复: [PATCH] tracing: Add down_write(trace_event

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <3ba7b2fd200642ec9a5f2a2c98b7def0@luxshare-ict.com>
Date: Fri, 18 Jul 2025 01:47:14 +0000
From: Fusheng Huang(黄富生)
	<Fusheng.Huang@...share-ict.com>
To: Steven Rostedt <rostedt@...dmis.org>, LKML <linux-kernel@...r.kernel.org>,
	Linux trace kernel <linux-trace-kernel@...r.kernel.org>
CC: Masami Hiramatsu <mhiramat@...nel.org>, Mathieu Desnoyers
	<mathieu.desnoyers@...icios.com>, GuoJin Dai(戴国金)
	<GuoJin.Dai@...share-ict.com>
Subject: 回复: [PATCH] tracing: Add down_write(trace_event_sem) when adding trace event

Hi, steven：

   Thank you very much, we will test this patch and feedback ASAP.

-----邮件原件-----
发件人: Steven Rostedt <rostedt@...dmis.org> 
发送时间: 2025年7月18日 1:12
收件人: LKML <linux-kernel@...r.kernel.org>; Linux trace kernel <linux-trace-kernel@...r.kernel.org>
抄送: Masami Hiramatsu <mhiramat@...nel.org>; Mathieu Desnoyers <mathieu.desnoyers@...icios.com>; Fusheng Huang(黄富生) <Fusheng.Huang@...share-ict.com>
主题: [PATCH] tracing: Add down_write(trace_event_sem) when adding trace event

【外部邮件】请勿点击链接或附件，除非您认识发件人并且知道内容是安全的。
[External Email] Do not click on links or attachments unless you recognize the sender and ensure the content is safe.
[Thư bên ngoài] Không nhấp vào liên kết hoặc tệp đính kèm trừ khi bạn biết người gửi và biết nội dung đó an toàn.

From: Steven Rostedt <rostedt@...dmis.org>

When a module is loaded, it adds trace events defined by the module. It may also need to modify the modules trace printk formats to replace enum names with their values.

If two modules are loaded at the same time, the adding of the event to the ftrace_events list can corrupt the walking of the list in the code that is modifying the printk format strings and crash the kernel.

The addition of the event should take the trace_event_sem for write while it adds the new event.

Also add a lockdep_assert_held_read() on that semaphore in
__trace_add_event_dirs() as it iterates the list.

Cc: stable@...r.kernel.org
Reported-by: Fusheng Huang(黄富生)  <Fusheng.Huang@...share-ict.com>
Closes: https://lore.kernel.org/all/20250717105007.46ccd18f@batman.local.home/
Fixes: 110bf2b764eb6 ("tracing: add protection around module events unload")
Signed-off-by: Steven Rostedt (Google) <rostedt@...dmis.org>
---
 kernel/trace/trace_events.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index 120531268abf..552c92911e2c 100644
--- a/kernel/trace/trace_events.c
+++ b/kernel/trace/trace_events.c
@@ -3136,7 +3136,10 @@ __register_event(struct trace_event_call *call, struct module *mod)
        if (ret < 0)
                return ret;

+       down_write(&trace_event_sem);
        list_add(&call->list, &ftrace_events);
+       up_write(&trace_event_sem);
+
        if (call->flags & TRACE_EVENT_FL_DYNAMIC)
                atomic_set(&call->refcnt, 0);
        else
@@ -3750,6 +3753,8 @@ __trace_add_event_dirs(struct trace_array *tr)
        struct trace_event_call *call;
        int ret;

+       lockdep_assert_held_read(&trace_event_sem);
+
        list_for_each_entry(call, &ftrace_events, list) {
                ret = __trace_add_new_event(call, tr);
                if (ret < 0)
--
2.47.2