]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
ufs: core: tracing: Do not dereference pointers in TP_printk()
authorSteven Rostedt <rostedt@goodmis.org>
Tue, 30 Jun 2026 22:54:12 +0000 (18:54 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Tue, 7 Jul 2026 14:42:28 +0000 (10:42 -0400)
commit535fcf4b8a261fbb8cc4f91e4597343c135a90f2
tree06616c7622ed2c0bb5d3e01d18fe23c011c57cce
parentc3e94604675e3db186111b8942650d86577df9b0
ufs: core: tracing: Do not dereference pointers in TP_printk()

The trace events in drivers/ufs/core/ufs_trace.h were converted to take a
pointer to the hba structure as an argument for the tracepoint and then in
TP_printk() the printing of the dev_name from the ring buffer was
converted to using the dev dereferenced pointer from the hba saved
pointer.

This is not allowed as the TP_printk() is executed at the time the trace
event is read from /sys/kernel/tracing/trace file. That can happen
literally, seconds, minutes, hours, weeks, days, or even months later!
There is no guarantee that the hba pointer will still exist by the time it
is dereferenced when the "trace" file is read.

Instead, save the device name from the hba pointer at the time the
tracepoint is called and place it into the ring buffer event. Then the
TP_printk() can read the name directly from the ring buffer and remove the
possibility that it will read a freed pointer and crash the kernel.

This was detected when testing the trace event code that looks for
TP_printk() parameters doing illegal derferences[1]

[1] https://lore.kernel.org/all/20260630184836.74d477b6@gandalf.local.home/

Cc: stable@vger.kernel.org
Link: https://patch.msgid.link/20260630185412.283c26c5@gandalf.local.home
Fixes: 583e518e71003 ("scsi: ufs: core: Add hba parameter to trace events")
Reviewed-by: Peter Wang <peter.wang@mediatek.com>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
drivers/ufs/core/ufs_trace.h