]> git.baikalelectronics.ru Git - kernel.git/commit
posix-cpu-timers: Cleanup CPU timers before freeing them during exec
authorThadeu Lima de Souza Cascardo <cascardo@canonical.com>
Tue, 9 Aug 2022 17:07:51 +0000 (14:07 -0300)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 9 Aug 2022 18:02:13 +0000 (20:02 +0200)
commit958bd12e79238d69236fcac7ffe1cf2d908cf2e6
tree34ffd7469d2a0cdb545195e7b9f04523a8ea8502
parentfcf549520022b44035c736e0e993caa60c2f4650
posix-cpu-timers: Cleanup CPU timers before freeing them during exec

Commit d58d6b25301c ("posix-cpu-timers: Store a reference to a pid not a
task") started looking up tasks by PID when deleting a CPU timer.

When a non-leader thread calls execve, it will switch PIDs with the leader
process. Then, as it calls exit_itimers, posix_cpu_timer_del cannot find
the task because the timer still points out to the old PID.

That means that armed timers won't be disarmed, that is, they won't be
removed from the timerqueue_list. exit_itimers will still release their
memory, and when that list is later processed, it leads to a
use-after-free.

Clean up the timers from the de-threaded task before freeing them. This
prevents a reported use-after-free.

Fixes: d58d6b25301c ("posix-cpu-timers: Store a reference to a pid not a task")
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20220809170751.164716-1-cascardo@canonical.com
fs/exec.c