kthread: Never put_user the set_child_tid address
authorEric W. Biederman <ebiederm@xmission.com>
Wed, 22 Dec 2021 22:57:50 +0000 (16:57 -0600)
committerEric W. Biederman <ebiederm@xmission.com>
Wed, 22 Dec 2021 22:57:50 +0000 (16:57 -0600)
commit00580f03af5eb2a527875b4a80a5effd95bda2fa
tree43cf743e630c66d1c711b67c179e2bbe03025ba9
parentdd621ee0cf8eb32445c8f5f26d3b7555953071d8
kthread: Never put_user the set_child_tid address

Kernel threads abuse set_child_tid.  Historically that has been fine
as set_child_tid was initialized after the kernel thread had been
forked.  Unfortunately storing struct kthread in set_child_tid after
the thread is running makes struct kthread being unusable for storing
result codes of the thread.

When set_child_tid is set to struct kthread during fork that results
in schedule_tail writing the thread id to the beggining of struct
kthread (if put_user does not realize it is a kernel address).

Solve this by skipping the put_user for all kthreads.

Reported-by: Nathan Chancellor <nathan@kernel.org>
Link: https://lkml.kernel.org/r/YcNsG0Lp94V13whH@archlinux-ax161
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
kernel/sched/core.c