aboutsummaryrefslogtreecommitdiffstats
path: root/arch/xtensa
diff options
context:
space:
mode:
authorDan Rosenberg <drosenberg@vsecurity.com>2011-07-25 20:11:53 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2011-07-25 23:57:07 -0400
commit0d0138ebe24b94065580bd2601f8bb7eb6152f56 (patch)
tree13ab981dbddd25b57bb821394d0a3c78bf8fb25a /arch/xtensa
parent67db392d1124e14684e23deb572de2a63b9b3b69 (diff)
xtensa: prevent arbitrary read in ptrace
Prevent an arbitrary kernel read. Check the user pointer with access_ok() before copying data in. [akpm@linux-foundation.org: s/EIO/EFAULT/] Signed-off-by: Dan Rosenberg <drosenberg@vsecurity.com> Cc: Christian Zankel <chris@zankel.net> Cc: Oleg Nesterov <oleg@redhat.com> Cc: <stable@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/xtensa')
-rw-r--r--arch/xtensa/kernel/ptrace.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/arch/xtensa/kernel/ptrace.c b/arch/xtensa/kernel/ptrace.c
index c72c9473ef99..a0d042aa2967 100644
--- a/arch/xtensa/kernel/ptrace.c
+++ b/arch/xtensa/kernel/ptrace.c
@@ -147,6 +147,9 @@ int ptrace_setxregs(struct task_struct *child, void __user *uregs)
147 elf_xtregs_t *xtregs = uregs; 147 elf_xtregs_t *xtregs = uregs;
148 int ret = 0; 148 int ret = 0;
149 149
150 if (!access_ok(VERIFY_READ, uregs, sizeof(elf_xtregs_t)))
151 return -EFAULT;
152
150#if XTENSA_HAVE_COPROCESSORS 153#if XTENSA_HAVE_COPROCESSORS
151 /* Flush all coprocessors before we overwrite them. */ 154 /* Flush all coprocessors before we overwrite them. */
152 coprocessor_flush_all(ti); 155 coprocessor_flush_all(ti);