Description
In the Linux kernel, the following vulnerability has been resolved: timers: Fix NULL function pointer race in timer_shutdown_sync() There is a race condition between timer_shutdown_sync() and timer expiration that can lead to hitting a WARN_ON in expire_timers(). The issue occurs when timer_shutdown_sync() clears the timer function to NULL while the timer is still running on another CPU. The race scenario looks like this: CPU0 CPU1 <SOFTIRQ> lock_timer_base() expire_timers() base->running_timer = timer; unlock_timer_base() [call_timer_fn enter] mod_timer() ... timer_shutdown_sync() lock_timer_base() // For now, will not detach the timer but only clear its function to NULL if (base->running_timer != timer) ret = detach_if_pending(timer, base, true); if (shutdown) timer->function = NULL; unlock_timer_base() [call_timer_fn exit] lock_timer_base() base->running_timer = NULL; unlock_timer_base() ... // Now timer is pending while its function set to NULL. // next timer trigger <SOFTIRQ> expire_timers() WARN_ON_ONCE(!fn) // hit ... lock_timer_base() // Now timer will detach if (base->running_timer != timer) ret = detach_if_pending(timer, base, true); if (shutdown) timer->function = NULL; unlock_timer_base() The problem is that timer_shutdown_sync() clears the timer function regardless of whether the timer is currently running. This can leave a pending timer with a NULL function pointer, which triggers the WARN_ON_ONCE(!fn) check in expire_timers(). Fix this by only clearing the timer function when actually detaching the timer. If the timer is running, leave the function pointer intact, which is safe because the timer will be properly detached when it finishes running.
Affected products
Version statuses are supplied by the assigning organization. Consult the original record for detailed ranges and platform conditions.
Linux Linux
Default version status: unaffected
debbcf812d735003c96c5e5968a3cfa4e1fbd1af to < ba43ac025c4318241f8edf94f31d2eebab86991baffected334c33aa487be406a149c8b87c38c8399d2dba8d to < 1a975716cc8977f461e45e28e3e5977d46ad7a6aaffected0cc04e80458a822300b93f82ed861a513edde194 to < 6665fbd7730b26d770c232b20d1b907e6a67a914affected0cc04e80458a822300b93f82ed861a513edde194 to < 176725f4848376530a0f0da9023f956afcc33585affected0cc04e80458a822300b93f82ed861a513edde194 to < a01efa7a780c42ac5170a949bd95c9786ffcc60aaffected0cc04e80458a822300b93f82ed861a513edde194 to < 20739af07383e6eb1ec59dcd70b72ebfa9ac362caffected6.1.158 to < 6.1.159affected
Linux Linux
Default version status: affected
6.2affected0 to < 6.2unaffected6.1.159 to ≤ 6.1.*unaffected6.6.118 to ≤ 6.6.*unaffected6.12.60 to ≤ 6.12.*unaffected6.17.10 to ≤ 6.17.*unaffected6.18 to ≤ *unaffected
References
- https://git.kernel.org/stable/c/ba43ac025c4318241f8edf94f31d2eebab86991b ↗
- https://git.kernel.org/stable/c/1a975716cc8977f461e45e28e3e5977d46ad7a6a ↗
- https://git.kernel.org/stable/c/6665fbd7730b26d770c232b20d1b907e6a67a914 ↗
- https://git.kernel.org/stable/c/176725f4848376530a0f0da9023f956afcc33585 ↗
- https://git.kernel.org/stable/c/a01efa7a780c42ac5170a949bd95c9786ffcc60a ↗
- https://git.kernel.org/stable/c/20739af07383e6eb1ec59dcd70b72ebfa9ac362c ↗