timers: Mask invalid flags in do_init_timer()
authorQianli Zhao <zhaoqianli@xiaomi.com>
Thu, 13 Aug 2020 15:03:14 +0000 (23:03 +0800)
committerThomas Gleixner <tglx@linutronix.de>
Thu, 24 Sep 2020 20:12:18 +0000 (22:12 +0200)
commitb952caf2d5ca898cc10d63be7722ae7a5daca696
treea05437d2478cc8c8752ca41c560f01d45666b8fb
parentec02821c1d35f93b821bc9fdfa83a5f3e9d7275d
timers: Mask invalid flags in do_init_timer()

do_init_timer() accepts any combination of timer flags handed in by the
caller without a sanity check, but only TIMER_DEFFERABLE, TIMER_PINNED and
TIMER_IRQSAFE are valid.

If the supplied flags have other bits set, this could result in
malfunction. If bits are set in TIMER_CPUMASK the first timer usage could
deference a cpu base which is outside the range of possible CPUs. If
TIMER_MIGRATION is set, then the switch_timer_base() will live lock.

Prevent that with a sanity check which warns when invalid flags are
supplied and masks them out.

[ tglx: Made it WARN_ON_ONCE() and added context to the changelog ]

Signed-off-by: Qianli Zhao <zhaoqianli@xiaomi.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/9d79a8aa4eb56713af7379f99f062dedabcde140.1597326756.git.zhaoqianli@xiaomi.com
include/linux/timer.h
kernel/time/timer.c