samples/ftrace: Fix kthread_stop() on ERR_PTR in ftrace-direct-modify
authorHaotian Zhang <vulab@iscas.ac.cn>
Wed, 26 Aug 2026 01:50:34 +0000 (09:50 +0800)
committerSteven Rostedt <rostedt@goodmis.org>
Wed, 26 Aug 2026 19:49:15 +0000 (15:49 -0400)
commitd79fb758e7e2e7a181eab630c0545a56041c473d
treec07a4e808ca95e8ec78dd9c5f40d4ac806ca27fd
parent1704aaaf5d22bc765c168402350d191e24e245bc
samples/ftrace: Fix kthread_stop() on ERR_PTR in ftrace-direct-modify

ftrace_direct_init() assigns kthread_run()'s return value to simple_tsk
without an IS_ERR() check. When kthread_run() fails it returns
ERR_PTR(-ENOMEM), but init still returns 0, so the module loads with
simple_tsk holding an error pointer. On unload, ftrace_direct_exit()
then passes that ERR_PTR to kthread_stop(), leading to a
null-pointer-dereference.

Check the return value of kthread_run() with IS_ERR(); on failure,
unregister the ftrace direct call and propagate the error code.

Link: https://patch.msgid.link/20260826015034.10755-1-vulab@iscas.ac.cn
Fixes: ae0cc3b7e7f5 ("ftrace/samples: Add a sample module that implements modify_ftrace_direct()")
Suggested-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Haotian Zhang <vulab@iscas.ac.cn>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
samples/ftrace/ftrace-direct-modify.c