diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-26 22:43:20 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-26 22:43:20 -0500 |
commit | 24e55910e4801d772f95becde20b526b8b10388d (patch) | |
tree | fe02c8e39d21e940562624e95fbe6641a590e734 | |
parent | dcad0fceae528e8007610308bad7e5a3370e5c39 (diff) | |
parent | 00f4e13c4634b10f6a16b26a980b22a53dfa6bb5 (diff) |
Merge branch 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull timer fixes from Ingo Molnar.
* 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
clocksource : Nomadik-mtu : fix missing irq initialization
posix-timer: Don't call idr_find() with out-of-range ID
-rw-r--r-- | drivers/clocksource/nomadik-mtu.c | 1 | ||||
-rw-r--r-- | kernel/posix-timers.c | 7 |
2 files changed, 8 insertions, 0 deletions
diff --git a/drivers/clocksource/nomadik-mtu.c b/drivers/clocksource/nomadik-mtu.c index 435e54d55bbd..071f6eadfea2 100644 --- a/drivers/clocksource/nomadik-mtu.c +++ b/drivers/clocksource/nomadik-mtu.c | |||
@@ -240,6 +240,7 @@ void __init nmdk_timer_init(void __iomem *base, int irq) | |||
240 | /* Timer 1 is used for events, register irq and clockevents */ | 240 | /* Timer 1 is used for events, register irq and clockevents */ |
241 | setup_irq(irq, &nmdk_timer_irq); | 241 | setup_irq(irq, &nmdk_timer_irq); |
242 | nmdk_clkevt.cpumask = cpumask_of(0); | 242 | nmdk_clkevt.cpumask = cpumask_of(0); |
243 | nmdk_clkevt.irq = irq; | ||
243 | clockevents_config_and_register(&nmdk_clkevt, rate, 2, 0xffffffffU); | 244 | clockevents_config_and_register(&nmdk_clkevt, rate, 2, 0xffffffffU); |
244 | 245 | ||
245 | mtu_delay_timer.read_current_timer = &nmdk_timer_read_current_timer; | 246 | mtu_delay_timer.read_current_timer = &nmdk_timer_read_current_timer; |
diff --git a/kernel/posix-timers.c b/kernel/posix-timers.c index 10349d5f2ec3..7edfe4b901e7 100644 --- a/kernel/posix-timers.c +++ b/kernel/posix-timers.c | |||
@@ -639,6 +639,13 @@ static struct k_itimer *__lock_timer(timer_t timer_id, unsigned long *flags) | |||
639 | { | 639 | { |
640 | struct k_itimer *timr; | 640 | struct k_itimer *timr; |
641 | 641 | ||
642 | /* | ||
643 | * timer_t could be any type >= int and we want to make sure any | ||
644 | * @timer_id outside positive int range fails lookup. | ||
645 | */ | ||
646 | if ((unsigned long long)timer_id > INT_MAX) | ||
647 | return NULL; | ||
648 | |||
642 | rcu_read_lock(); | 649 | rcu_read_lock(); |
643 | timr = idr_find(&posix_timers_id, (int)timer_id); | 650 | timr = idr_find(&posix_timers_id, (int)timer_id); |
644 | if (timr) { | 651 | if (timr) { |