Merge branch 'x86-x32-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git...
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 2 Apr 2014 19:51:41 +0000 (12:51 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 2 Apr 2014 19:51:41 +0000 (12:51 -0700)
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)

1  2 
fs/compat.c
include/linux/compat.h
ipc/compat.c
ipc/compat_mq.c
kernel/compat.c

diff --cc fs/compat.c
index 19252b97f0cc9662940c2782c76e6782c7fa787a,62092f43375905d15cbdd84eb81387db13e3f8e4..f86df85dff612b2910e6e33f35fc8d85c13b635b
@@@ -504,9 -506,14 +504,9 @@@ COMPAT_SYSCALL_DEFINE5(io_getevents, co
        struct timespec t;
        struct timespec __user *ut = NULL;
  
 -      ret = -EFAULT;
 -      if (unlikely(!access_ok(VERIFY_WRITE, events, 
 -                              nr * sizeof(struct io_event))))
 -              goto out;
        if (timeout) {
-               if (get_compat_timespec(&t, timeout))
+               if (compat_get_timespec(&t, timeout))
 -                      goto out;
 +                      return -EFAULT;
  
                ut = compat_alloc_user_space(sizeof(*ut));
                if (copy_to_user(ut, &t, sizeof(t)) )
Simple merge
diff --cc ipc/compat.c
index 98b9016cab6c84e4e86151458a5e9b8063ffc6f2,1048522479e556b421be164b96a18f9370464764..a4695ada3275742e0bd5258e55ab046751fb38b4
@@@ -749,18 -749,11 +749,12 @@@ COMPAT_SYSCALL_DEFINE3(shmctl, int, fir
        return err;
  }
  
 -long compat_sys_semtimedop(int semid, struct sembuf __user *tsems,
 -              unsigned nsops, const struct compat_timespec __user *timeout)
 +COMPAT_SYSCALL_DEFINE4(semtimedop, int, semid, struct sembuf __user *, tsems,
 +                     unsigned, nsops,
 +                     const struct compat_timespec __user *, timeout)
  {
-       struct timespec __user *ts64 = NULL;
-       if (timeout) {
-               struct timespec ts;
-               ts64 = compat_alloc_user_space(sizeof(*ts64));
-               if (get_compat_timespec(&ts, timeout))
-                       return -EFAULT;
-               if (copy_to_user(ts64, &ts, sizeof(ts)))
-                       return -EFAULT;
-       }
+       struct timespec __user *ts64;
+       if (compat_convert_timespec(&ts64, timeout))
+               return -EFAULT;
        return sys_semtimedop(semid, tsems, nsops, ts64);
  }
diff --cc ipc/compat_mq.c
index d5874729377296acc8a95cd0c27b7ab5eb903a76,a9cf16378d7a6438e685fe6473daf879b88ba35d..90d29f59cac6e525f1904bdb7a1d623e34ae4fe2
@@@ -64,24 -64,10 +64,10 @@@ COMPAT_SYSCALL_DEFINE4(mq_open, const c
        return sys_mq_open(u_name, oflag, mode, p);
  }
  
- static int compat_prepare_timeout(struct timespec __user **p,
-                                 const struct compat_timespec __user *u)
- {
-       struct timespec ts;
-       if (!u) {
-               *p = NULL;
-               return 0;
-       }
-       *p = compat_alloc_user_space(sizeof(ts));
-       if (get_compat_timespec(&ts, u) || copy_to_user(*p, &ts, sizeof(ts)))
-               return -EFAULT;
-       return 0;
- }
 -asmlinkage long compat_sys_mq_timedsend(mqd_t mqdes,
 -                      const char __user *u_msg_ptr,
 -                      size_t msg_len, unsigned int msg_prio,
 -                      const struct compat_timespec __user *u_abs_timeout)
 +COMPAT_SYSCALL_DEFINE5(mq_timedsend, mqd_t, mqdes,
 +                     const char __user *, u_msg_ptr,
 +                     compat_size_t, msg_len, unsigned int, msg_prio,
 +                     const struct compat_timespec __user *, u_abs_timeout)
  {
        struct timespec __user *u_ts;
  
                        msg_prio, u_ts);
  }
  
 -asmlinkage ssize_t compat_sys_mq_timedreceive(mqd_t mqdes,
 -                      char __user *u_msg_ptr,
 -                      size_t msg_len, unsigned int __user *u_msg_prio,
 -                      const struct compat_timespec __user *u_abs_timeout)
 +COMPAT_SYSCALL_DEFINE5(mq_timedreceive, mqd_t, mqdes,
 +                     char __user *, u_msg_ptr,
 +                     compat_size_t, msg_len, unsigned int __user *, u_msg_prio,
 +                     const struct compat_timespec __user *, u_abs_timeout)
  {
        struct timespec __user *u_ts;
-       if (compat_prepare_timeout(&u_ts, u_abs_timeout))
+       if (compat_convert_timespec(&u_ts, u_abs_timeout))
                return -EFAULT;
  
        return sys_mq_timedreceive(mqdes, u_msg_ptr, msg_len,
diff --cc kernel/compat.c
index 488ff8c4cf48ec071ad6cdb824fe60af735b5c8f,7076b57fa52eb4031bf1d046b6625a1bcda0bc01..e40b0430b562856df203ec26a7150d6a745ab562
@@@ -127,18 -105,21 +105,21 @@@ COMPAT_SYSCALL_DEFINE2(gettimeofday, st
        return 0;
  }
  
 -asmlinkage long compat_sys_settimeofday(struct compat_timeval __user *tv,
 -              struct timezone __user *tz)
 +COMPAT_SYSCALL_DEFINE2(settimeofday, struct compat_timeval __user *, tv,
 +                     struct timezone __user *, tz)
  {
-       struct timespec kts;
-       struct timezone ktz;
+       struct timeval user_tv;
+       struct timespec new_ts;
+       struct timezone new_tz;
  
        if (tv) {
-               if (compat_get_timeval_convert(&kts, tv))
+               if (compat_get_timeval(&user_tv, tv))
                        return -EFAULT;
+               new_ts.tv_sec = user_tv.tv_sec;
+               new_ts.tv_nsec = user_tv.tv_usec * NSEC_PER_USEC;
        }
        if (tz) {
-               if (copy_from_user(&ktz, tz, sizeof(ktz)))
+               if (copy_from_user(&new_tz, tz, sizeof(*tz)))
                        return -EFAULT;
        }