Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=755f21bb894d3fe57017f5aa8f7ace8808fcb1e3
Commit:     755f21bb894d3fe57017f5aa8f7ace8808fcb1e3
Parent:     8a03d9a498eaf02c8a118752050a5154852c13bf
Author:     Atsushi Nemoto <[EMAIL PROTECTED]>
AuthorDate: Wed Feb 14 14:41:01 2007 +0900
Committer:  Ralf Baechle <[EMAIL PROTECTED]>
CommitDate: Sun Feb 18 21:31:33 2007 +0000

    [MIPS] Fix sigset_t endianess swapping issues in 32-bit compat code.
    
    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 <[EMAIL PROTECTED]>
    Signed-off-by: Ralf Baechle <[EMAIL PROTECTED]>
---
 arch/mips/kernel/signal32.c      |    4 ++--
 include/asm-mips/compat-signal.h |    3 ---
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/arch/mips/kernel/signal32.c b/arch/mips/kernel/signal32.c
index c28cb21..9f829f6 100644
--- a/arch/mips/kernel/signal32.c
+++ b/arch/mips/kernel/signal32.c
@@ -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 */
 };
 
diff --git a/include/asm-mips/compat-signal.h b/include/asm-mips/compat-signal.h
index 6720770..d6e7c3b 100644
--- a/include/asm-mips/compat-signal.h
+++ b/include/asm-mips/compat-signal.h
@@ -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]);
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to