From: Thadeu Lima de Souza Cascardo Date: Tue, 9 Aug 2022 17:07:51 +0000 (-0300) Subject: posix-cpu-timers: Cleanup CPU timers before freeing them during exec X-Git-Tag: baikal/mips/sdk5.9~95 X-Git-Url: https://git.baikalelectronics.ru/sdk/?a=commitdiff_plain;h=06a2b06ba3d8ca48cf32e635bde32427eb1e97dc;p=kernel.git posix-cpu-timers: Cleanup CPU timers before freeing them during exec commit af5644453e8833d9af643205ee20d42635114cb3 upstream. Commit 52a323cd661b ("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: 52a323cd661b ("posix-cpu-timers: Store a reference to a pid not a task") Signed-off-by: Thadeu Lima de Souza Cascardo Signed-off-by: Thomas Gleixner Reviewed-by: Thomas Gleixner Cc: Link: https://lore.kernel.org/r/20220809170751.164716-1-cascardo@canonical.com Signed-off-by: Greg Kroah-Hartman --- diff --git a/fs/exec.c b/fs/exec.c index 76196ddefbdd0..7d424337b4ec9 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -1298,6 +1298,9 @@ int begin_new_exec(struct linux_binprm * bprm) bprm->mm = NULL; #ifdef CONFIG_POSIX_TIMERS + spin_lock_irq(&me->sighand->siglock); + posix_cpu_timers_exit(me); + spin_unlock_irq(&me->sighand->siglock); exit_itimers(me); flush_itimer_signals(); #endif