ppc: switch to {get,put}_compat_sigset()
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 4 Sep 2017 16:17:38 +0000 (12:17 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Tue, 19 Sep 2017 21:56:03 +0000 (17:56 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/powerpc/kernel/signal_32.c

index 92fb1c8..1a96992 100644 (file)
  */
 static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set)
 {
-       compat_sigset_t cset;
-
-       switch (_NSIG_WORDS) {
-       case 4: cset.sig[6] = set->sig[3] & 0xffffffffull;
-               cset.sig[7] = set->sig[3] >> 32;
-       case 3: cset.sig[4] = set->sig[2] & 0xffffffffull;
-               cset.sig[5] = set->sig[2] >> 32;
-       case 2: cset.sig[2] = set->sig[1] & 0xffffffffull;
-               cset.sig[3] = set->sig[1] >> 32;
-       case 1: cset.sig[0] = set->sig[0] & 0xffffffffull;
-               cset.sig[1] = set->sig[0] >> 32;
-       }
-       return copy_to_user(uset, &cset, sizeof(*uset));
+       return put_compat_sigset(uset, set, sizeof(*uset));
 }
 
 static inline int get_sigset_t(sigset_t *set,
                               const compat_sigset_t __user *uset)
 {
-       compat_sigset_t s32;
-
-       if (copy_from_user(&s32, uset, sizeof(*uset)))
-               return -EFAULT;
-
-       /*
-        * Swap the 2 words of the 64-bit sigset_t (they are stored
-        * in the "wrong" endian in 32-bit user storage).
-        */
-       switch (_NSIG_WORDS) {
-       case 4: set->sig[3] = s32.sig[6] | (((long)s32.sig[7]) << 32);
-       case 3: set->sig[2] = s32.sig[4] | (((long)s32.sig[5]) << 32);
-       case 2: set->sig[1] = s32.sig[2] | (((long)s32.sig[3]) << 32);
-       case 1: set->sig[0] = s32.sig[0] | (((long)s32.sig[1]) << 32);
-       }
-       return 0;
+       return get_compat_sigset(set, uset);
 }
 
 #define to_user_ptr(p)         ptr_to_compat(p)