diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2018-03-25 13:34:50 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-03-25 13:34:50 -0400 |
commit | 9fd64e8ac2632929fb7f5d1245a15ae45ba21da6 (patch) | |
tree | fb57c31cac8fa1d6d64d7da6634ab3f7548c8aa9 | |
parent | bf45bae961d503b4cbf6c78d3ed71fe35e8f168d (diff) | |
parent | 19b558db12f9f4e45a22012bae7b4783e62224da (diff) |
Merge branch 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull timer fix from Ingo Molnar:
"Make posix clock ID usage Spectre-safe"
* 'timers-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
posix-timers: Protect posix clock array access against speculation
-rw-r--r-- | kernel/time/posix-timers.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/kernel/time/posix-timers.c b/kernel/time/posix-timers.c index 75043046914e..10b7186d0638 100644 --- a/kernel/time/posix-timers.c +++ b/kernel/time/posix-timers.c | |||
@@ -50,6 +50,7 @@ | |||
50 | #include <linux/export.h> | 50 | #include <linux/export.h> |
51 | #include <linux/hashtable.h> | 51 | #include <linux/hashtable.h> |
52 | #include <linux/compat.h> | 52 | #include <linux/compat.h> |
53 | #include <linux/nospec.h> | ||
53 | 54 | ||
54 | #include "timekeeping.h" | 55 | #include "timekeeping.h" |
55 | #include "posix-timers.h" | 56 | #include "posix-timers.h" |
@@ -1346,11 +1347,15 @@ static const struct k_clock * const posix_clocks[] = { | |||
1346 | 1347 | ||
1347 | static const struct k_clock *clockid_to_kclock(const clockid_t id) | 1348 | static const struct k_clock *clockid_to_kclock(const clockid_t id) |
1348 | { | 1349 | { |
1349 | if (id < 0) | 1350 | clockid_t idx = id; |
1351 | |||
1352 | if (id < 0) { | ||
1350 | return (id & CLOCKFD_MASK) == CLOCKFD ? | 1353 | return (id & CLOCKFD_MASK) == CLOCKFD ? |
1351 | &clock_posix_dynamic : &clock_posix_cpu; | 1354 | &clock_posix_dynamic : &clock_posix_cpu; |
1355 | } | ||
1352 | 1356 | ||
1353 | if (id >= ARRAY_SIZE(posix_clocks) || !posix_clocks[id]) | 1357 | if (id >= ARRAY_SIZE(posix_clocks)) |
1354 | return NULL; | 1358 | return NULL; |
1355 | return posix_clocks[id]; | 1359 | |
1360 | return posix_clocks[array_index_nospec(idx, ARRAY_SIZE(posix_clocks))]; | ||
1356 | } | 1361 | } |