diff options
author | Suresh Siddha <suresh.b.siddha@intel.com> | 2008-08-13 14:38:13 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-08-14 04:56:06 -0400 |
commit | ee2b92a8201a40021ecd1aee6f0625dc03bacc54 (patch) | |
tree | 083f53ce26f20736968af50bf96da3ebb247b334 /arch/x86 | |
parent | d4439087d316613548c70472291bf4ca646ce24b (diff) |
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 <suresh.b.siddha@intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86')
-rw-r--r-- | arch/x86/kernel/signal_64.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/arch/x86/kernel/signal_64.c b/arch/x86/kernel/signal_64.c index 2621b98f5bf6..6c581698ab56 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, | |||
208 | frame = (void __user *)round_down( | 208 | frame = (void __user *)round_down( |
209 | (unsigned long)fp - sizeof(struct rt_sigframe), 16) - 8; | 209 | (unsigned long)fp - sizeof(struct rt_sigframe), 16) - 8; |
210 | 210 | ||
211 | if (!access_ok(VERIFY_WRITE, fp, sizeof(struct _fpstate))) | ||
212 | goto give_sigsegv; | ||
213 | |||
214 | if (save_i387_xstate(fp) < 0) | 211 | if (save_i387_xstate(fp) < 0) |
215 | err |= -1; | 212 | err |= -1; |
216 | } else | 213 | } else |