x86, xsave: remove the redundant access_ok() in setup_rt_frame()
authorSuresh Siddha <suresh.b.siddha@intel.com>
Wed, 13 Aug 2008 18:38:13 +0000 (11:38 -0700)
committerIngo Molnar <mingo@elte.hu>
Thu, 14 Aug 2008 08:56:06 +0000 (10:56 +0200)
save_i387_xstate() is already doing the required access_ok(). Remove
the redundant access_ok() before it.

Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/signal_64.c

index 2621b98f5bf6635a5713bef4b0f3bbaf65ba4e41..6c581698ab56887292596e56729dc200b6f750f8 100644 (file)
@@ -208,9 +208,6 @@ static int setup_rt_frame(int sig, struct k_sigaction *ka, siginfo_t *info,
                frame = (void __user *)round_down(
                        (unsigned long)fp - sizeof(struct rt_sigframe), 16) - 8;
 
-               if (!access_ok(VERIFY_WRITE, fp, sizeof(struct _fpstate)))
-                       goto give_sigsegv;
-
                if (save_i387_xstate(fp) < 0)
                        err |= -1; 
        } else