diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2014-04-02 15:51:41 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-04-02 15:51:41 -0400 |
commit | 7125764c5d1a5c72d522f1011b6cc8b8100b48fe (patch) | |
tree | 678f3355ac872a4379b28dbe36f5beddd0e284d8 /ipc/compat.c | |
parent | c6f21243ce1e8d81ad8361da4d2eaa5947b667c4 (diff) | |
parent | dce44e03b0a3448ad11ac6c6e0cbe299e0400791 (diff) |
Merge branch 'x86-x32-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull compat time conversion changes from Peter Anvin:
"Despite the branch name this is really neither an x86 nor an
x32-specific patchset, although it the implementation of the
discussions that followed the x32 security hole a few months ago.
This removes get/put_compat_timespec/val() and replaces them with
compat_get/put_timespec/val() which are savvy as to the current status
of COMPAT_USE_64BIT_TIME.
It removes several unused and/or incorrect/misleading functions (like
compat_put_timeval_convert which doesn't in fact do any conversion)
and also replaces several open-coded implementations what is now
called compat_convert_timespec() with that function"
* 'x86-x32-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
compat: Fix sparse address space warnings
compat: Get rid of (get|put)_compat_time(val|spec)
Diffstat (limited to 'ipc/compat.c')
-rw-r--r-- | ipc/compat.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/ipc/compat.c b/ipc/compat.c index 98b9016cab6c..a4695ada3275 100644 --- a/ipc/compat.c +++ b/ipc/compat.c | |||
@@ -753,14 +753,8 @@ COMPAT_SYSCALL_DEFINE4(semtimedop, int, semid, struct sembuf __user *, tsems, | |||
753 | unsigned, nsops, | 753 | unsigned, nsops, |
754 | const struct compat_timespec __user *, timeout) | 754 | const struct compat_timespec __user *, timeout) |
755 | { | 755 | { |
756 | struct timespec __user *ts64 = NULL; | 756 | struct timespec __user *ts64; |
757 | if (timeout) { | 757 | if (compat_convert_timespec(&ts64, timeout)) |
758 | struct timespec ts; | 758 | return -EFAULT; |
759 | ts64 = compat_alloc_user_space(sizeof(*ts64)); | ||
760 | if (get_compat_timespec(&ts, timeout)) | ||
761 | return -EFAULT; | ||
762 | if (copy_to_user(ts64, &ts, sizeof(ts))) | ||
763 | return -EFAULT; | ||
764 | } | ||
765 | return sys_semtimedop(semid, tsems, nsops, ts64); | 759 | return sys_semtimedop(semid, tsems, nsops, ts64); |
766 | } | 760 | } |