Commit 4e8b1452 authored by John Stultz's avatar John Stultz Committed by Thomas Gleixner

time: Improve sanity checking of timekeeping inputs

Unexpected behavior could occur if the time is set to a value large
enough to overflow a 64bit ktime_t (which is something larger then the
year 2262).

Also unexpected behavior could occur if large negative offsets are
injected via adjtimex.

So this patch improves the sanity check timekeeping inputs by
improving the timespec_valid() check, and then makes better use of
timespec_valid() to make sure we don't set the time to an invalid
negative value or one that overflows ktime_t.

Note: This does not protect from setting the time close to overflowing
ktime_t and then letting natural accumulation cause the overflow.
Reported-by: default avatarCAI Qian <caiqian@redhat.com>
Reported-by: default avatarSasha Levin <levinsasha928@gmail.com>
Signed-off-by: default avatarJohn Stultz <john.stultz@linaro.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Prarit Bhargava <prarit@redhat.com>
Cc: Zhouping Liu <zliu@redhat.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: stable@vger.kernel.org
Link: http://lkml.kernel.org/r/1344454580-17031-1-git-send-email-john.stultz@linaro.orgSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 3bf671af
...@@ -58,13 +58,6 @@ union ktime { ...@@ -58,13 +58,6 @@ union ktime {
typedef union ktime ktime_t; /* Kill this */ typedef union ktime ktime_t; /* Kill this */
#define KTIME_MAX ((s64)~((u64)1 << 63))
#if (BITS_PER_LONG == 64)
# define KTIME_SEC_MAX (KTIME_MAX / NSEC_PER_SEC)
#else
# define KTIME_SEC_MAX LONG_MAX
#endif
/* /*
* ktime_t definitions when using the 64-bit scalar representation: * ktime_t definitions when using the 64-bit scalar representation:
*/ */
......
...@@ -107,11 +107,29 @@ static inline struct timespec timespec_sub(struct timespec lhs, ...@@ -107,11 +107,29 @@ static inline struct timespec timespec_sub(struct timespec lhs,
return ts_delta; return ts_delta;
} }
#define KTIME_MAX ((s64)~((u64)1 << 63))
#if (BITS_PER_LONG == 64)
# define KTIME_SEC_MAX (KTIME_MAX / NSEC_PER_SEC)
#else
# define KTIME_SEC_MAX LONG_MAX
#endif
/* /*
* Returns true if the timespec is norm, false if denorm: * Returns true if the timespec is norm, false if denorm:
*/ */
#define timespec_valid(ts) \ static inline bool timespec_valid(const struct timespec *ts)
(((ts)->tv_sec >= 0) && (((unsigned long) (ts)->tv_nsec) < NSEC_PER_SEC)) {
/* Dates before 1970 are bogus */
if (ts->tv_sec < 0)
return false;
/* Can't have more nanoseconds then a second */
if ((unsigned long)ts->tv_nsec >= NSEC_PER_SEC)
return false;
/* Disallow values that could overflow ktime_t */
if ((unsigned long long)ts->tv_sec >= KTIME_SEC_MAX)
return false;
return true;
}
extern void read_persistent_clock(struct timespec *ts); extern void read_persistent_clock(struct timespec *ts);
extern void read_boot_clock(struct timespec *ts); extern void read_boot_clock(struct timespec *ts);
......
...@@ -427,7 +427,7 @@ int do_settimeofday(const struct timespec *tv) ...@@ -427,7 +427,7 @@ int do_settimeofday(const struct timespec *tv)
struct timespec ts_delta, xt; struct timespec ts_delta, xt;
unsigned long flags; unsigned long flags;
if ((unsigned long)tv->tv_nsec >= NSEC_PER_SEC) if (!timespec_valid(tv))
return -EINVAL; return -EINVAL;
write_seqlock_irqsave(&tk->lock, flags); write_seqlock_irqsave(&tk->lock, flags);
...@@ -463,6 +463,8 @@ int timekeeping_inject_offset(struct timespec *ts) ...@@ -463,6 +463,8 @@ int timekeeping_inject_offset(struct timespec *ts)
{ {
struct timekeeper *tk = &timekeeper; struct timekeeper *tk = &timekeeper;
unsigned long flags; unsigned long flags;
struct timespec tmp;
int ret = 0;
if ((unsigned long)ts->tv_nsec >= NSEC_PER_SEC) if ((unsigned long)ts->tv_nsec >= NSEC_PER_SEC)
return -EINVAL; return -EINVAL;
...@@ -471,10 +473,17 @@ int timekeeping_inject_offset(struct timespec *ts) ...@@ -471,10 +473,17 @@ int timekeeping_inject_offset(struct timespec *ts)
timekeeping_forward_now(tk); timekeeping_forward_now(tk);
/* Make sure the proposed value is valid */
tmp = timespec_add(tk_xtime(tk), *ts);
if (!timespec_valid(&tmp)) {
ret = -EINVAL;
goto error;
}
tk_xtime_add(tk, ts); tk_xtime_add(tk, ts);
tk_set_wall_to_mono(tk, timespec_sub(tk->wall_to_monotonic, *ts)); tk_set_wall_to_mono(tk, timespec_sub(tk->wall_to_monotonic, *ts));
error: /* even if we error out, we forwarded the time, so call update */
timekeeping_update(tk, true); timekeeping_update(tk, true);
write_sequnlock_irqrestore(&tk->lock, flags); write_sequnlock_irqrestore(&tk->lock, flags);
...@@ -482,7 +491,7 @@ int timekeeping_inject_offset(struct timespec *ts) ...@@ -482,7 +491,7 @@ int timekeeping_inject_offset(struct timespec *ts)
/* signal hrtimers about time change */ /* signal hrtimers about time change */
clock_was_set(); clock_was_set();
return 0; return ret;
} }
EXPORT_SYMBOL(timekeeping_inject_offset); EXPORT_SYMBOL(timekeeping_inject_offset);
...@@ -649,7 +658,20 @@ void __init timekeeping_init(void) ...@@ -649,7 +658,20 @@ void __init timekeeping_init(void)
struct timespec now, boot, tmp; struct timespec now, boot, tmp;
read_persistent_clock(&now); read_persistent_clock(&now);
if (!timespec_valid(&now)) {
pr_warn("WARNING: Persistent clock returned invalid value!\n"
" Check your CMOS/BIOS settings.\n");
now.tv_sec = 0;
now.tv_nsec = 0;
}
read_boot_clock(&boot); read_boot_clock(&boot);
if (!timespec_valid(&boot)) {
pr_warn("WARNING: Boot clock returned invalid value!\n"
" Check your CMOS/BIOS settings.\n");
boot.tv_sec = 0;
boot.tv_nsec = 0;
}
seqlock_init(&tk->lock); seqlock_init(&tk->lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment