save_i387_xstate() is already doing the required access_ok(). Remove the redundant access_ok() before it. Signed-off-by: Suresh Siddha --- 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/