From: Suresh Siddha Date: Wed, 13 Aug 2008 18:38:13 +0000 (-0700) Subject: x86, xsave: remove the redundant access_ok() in setup_rt_frame() X-Git-Tag: v2.6.28-rc1~699^2^8~9 X-Git-Url: http://pileus.org/git/?a=commitdiff_plain;h=ee2b92a8201a40021ecd1aee6f0625dc03bacc54;p=~andy%2Flinux x86, xsave: remove the redundant access_ok() in setup_rt_frame() save_i387_xstate() is already doing the required access_ok(). Remove the redundant access_ok() before it. Signed-off-by: Suresh Siddha Signed-off-by: Ingo Molnar --- diff --git a/arch/x86/kernel/signal_64.c b/arch/x86/kernel/signal_64.c index 2621b98f5bf..6c581698ab5 100644 --- a/arch/x86/kernel/signal_64.c +++ b/arch/x86/kernel/signal_64.c @@ -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