[MIPS] Fix sigset_t endianess swapping issues in 32-bit compat code.
authorAtsushi Nemoto <anemo@mba.ocn.ne.jp>
Wed, 14 Feb 2007 05:41:01 +0000 (14:41 +0900)
committerRalf Baechle <ralf@linux-mips.org>
Sun, 18 Feb 2007 21:31:33 +0000 (21:31 +0000)
access_ok in compat-signal.h is wrong (checking destination instead of
source) and redundant (already checked before calling this function).

Also sf_mask in struct sigframe32 should be compat_sigset_t type.

Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/kernel/signal32.c
include/asm-mips/compat-signal.h

index c28cb21514c858dc856918f22211f4036565b7c7..9f829f63024bd6c6226818672058607edacae8d6 100644 (file)
@@ -150,7 +150,7 @@ struct sigframe32 {
        u32 sf_ass[4];          /* argument save space for o32 */
        u32 sf_code[2];         /* signal trampoline */
        struct sigcontext32 sf_sc;
-       sigset_t sf_mask;
+       compat_sigset_t sf_mask;
 };
 
 struct rt_sigframe32 {
@@ -166,7 +166,7 @@ struct sigframe32 {
        u32 sf_ass[4];                  /* argument save space for o32 */
        u32 sf_pad[2];
        struct sigcontext32 sf_sc;      /* hw context */
-       sigset_t sf_mask;
+       compat_sigset_t sf_mask;
        u32 sf_code[8] ____cacheline_aligned;   /* signal trampoline */
 };
 
index 672077084aa1ffc12fe2cb2f943d1e1f1015828e..d6e7c3b4119a6bb301ae9340c02fdef68e95e204 100644 (file)
@@ -33,9 +33,6 @@ static inline int __copy_conv_sigset_from_user(sigset_t *d,
        BUG_ON(sizeof(*d) != sizeof(*s));
        BUG_ON(_NSIG_WORDS != 2);
 
-       if (unlikely(!access_ok(VERIFY_READ, d, sizeof(*d))))
-               return -EFAULT;
-
 #ifdef CONFIG_CPU_BIG_ENDIAN
        err  = __get_user(u->c.sig[1], &s->sig[0]);
        err |= __get_user(u->c.sig[0], &s->sig[1]);