[patch 1/3] x86, xsave: remove the redundant access_ok() in setup_rt_frame()

From: Suresh Siddha
Date: Wed Aug 13 2008 - 14:42:20 EST


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@xxxxxxxxx>
---

Index: tip/arch/x86/kernel/signal_64.c
===================================================================
--- tip.orig/arch/x86/kernel/signal_64.c 2008-08-11 15:26:47.000000000 -0700
+++ tip/arch/x86/kernel/signal_64.c 2008-08-11 15:28:52.000000000 -0700
@@ -208,9 +208,6 @@
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

--

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/