Commit fabca2c0 authored by Roland McGrath's avatar Roland McGrath Committed by Paul Mackerras

[POWERPC] Add CHECK_FULL_REGS in several places in ptrace code

This restores the CHECK_FULL_REGS sanity check to every place that can
access the nonvolatile GPRs for ptrace.  This is already done for
native-bitwidth PTRACE_PEEKUSR, but was omitted for many other cases
(32-bit ptrace, PTRACE_GETREGS, etc.); I think there may have been more
uniform checks before that were lost in the recent cleanup of GETREGS et al.
Signed-off-by: default avatarRoland McGrath <roland@redhat.com>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent fec60470
...@@ -331,6 +331,7 @@ static long arch_ptrace_old(struct task_struct *child, long request, long addr, ...@@ -331,6 +331,7 @@ static long arch_ptrace_old(struct task_struct *child, long request, long addr,
unsigned long *reg = &((unsigned long *)child->thread.regs)[0]; unsigned long *reg = &((unsigned long *)child->thread.regs)[0];
unsigned long __user *tmp = (unsigned long __user *)addr; unsigned long __user *tmp = (unsigned long __user *)addr;
CHECK_FULL_REGS(child->thread.regs);
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
ret = put_user(*reg, tmp); ret = put_user(*reg, tmp);
if (ret) if (ret)
...@@ -346,6 +347,7 @@ static long arch_ptrace_old(struct task_struct *child, long request, long addr, ...@@ -346,6 +347,7 @@ static long arch_ptrace_old(struct task_struct *child, long request, long addr,
unsigned long *reg = &((unsigned long *)child->thread.regs)[0]; unsigned long *reg = &((unsigned long *)child->thread.regs)[0];
unsigned long __user *tmp = (unsigned long __user *)addr; unsigned long __user *tmp = (unsigned long __user *)addr;
CHECK_FULL_REGS(child->thread.regs);
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
ret = get_user(*reg, tmp); ret = get_user(*reg, tmp);
if (ret) if (ret)
...@@ -517,6 +519,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data) ...@@ -517,6 +519,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
ret = -EIO; ret = -EIO;
break; break;
} }
CHECK_FULL_REGS(child->thread.regs);
ret = 0; ret = 0;
for (ui = 0; ui < PT_REGS_COUNT; ui ++) { for (ui = 0; ui < PT_REGS_COUNT; ui ++) {
ret |= __put_user(ptrace_get_reg(child, ui), ret |= __put_user(ptrace_get_reg(child, ui),
...@@ -537,6 +540,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data) ...@@ -537,6 +540,7 @@ long arch_ptrace(struct task_struct *child, long request, long addr, long data)
ret = -EIO; ret = -EIO;
break; break;
} }
CHECK_FULL_REGS(child->thread.regs);
ret = 0; ret = 0;
for (ui = 0; ui < PT_REGS_COUNT; ui ++) { for (ui = 0; ui < PT_REGS_COUNT; ui ++) {
ret = __get_user(tmp, (unsigned long __user *) data); ret = __get_user(tmp, (unsigned long __user *) data);
......
...@@ -53,6 +53,7 @@ static long compat_ptrace_old(struct task_struct *child, long request, ...@@ -53,6 +53,7 @@ static long compat_ptrace_old(struct task_struct *child, long request,
unsigned long *reg = &((unsigned long *)child->thread.regs)[0]; unsigned long *reg = &((unsigned long *)child->thread.regs)[0];
unsigned int __user *tmp = (unsigned int __user *)addr; unsigned int __user *tmp = (unsigned int __user *)addr;
CHECK_FULL_REGS(child->thread.regs);
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
ret = put_user(*reg, tmp); ret = put_user(*reg, tmp);
if (ret) if (ret)
...@@ -68,6 +69,7 @@ static long compat_ptrace_old(struct task_struct *child, long request, ...@@ -68,6 +69,7 @@ static long compat_ptrace_old(struct task_struct *child, long request,
unsigned long *reg = &((unsigned long *)child->thread.regs)[0]; unsigned long *reg = &((unsigned long *)child->thread.regs)[0];
unsigned int __user *tmp = (unsigned int __user *)addr; unsigned int __user *tmp = (unsigned int __user *)addr;
CHECK_FULL_REGS(child->thread.regs);
for (i = 0; i < 32; i++) { for (i = 0; i < 32; i++) {
ret = get_user(*reg, tmp); ret = get_user(*reg, tmp);
if (ret) if (ret)
...@@ -164,6 +166,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -164,6 +166,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
if ((addr & 3) || (index > PT_FPSCR32)) if ((addr & 3) || (index > PT_FPSCR32))
break; break;
CHECK_FULL_REGS(child->thread.regs);
if (index < PT_FPR0) { if (index < PT_FPR0) {
tmp = ptrace_get_reg(child, index); tmp = ptrace_get_reg(child, index);
} else { } else {
...@@ -210,6 +213,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -210,6 +213,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
if ((addr & 3) || numReg > PT_FPSCR) if ((addr & 3) || numReg > PT_FPSCR)
break; break;
CHECK_FULL_REGS(child->thread.regs);
if (numReg >= PT_FPR0) { if (numReg >= PT_FPR0) {
flush_fp_to_thread(child); flush_fp_to_thread(child);
tmp = ((unsigned long int *)child->thread.fpr)[numReg - PT_FPR0]; tmp = ((unsigned long int *)child->thread.fpr)[numReg - PT_FPR0];
...@@ -270,6 +274,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -270,6 +274,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
if ((addr & 3) || (index > PT_FPSCR32)) if ((addr & 3) || (index > PT_FPSCR32))
break; break;
CHECK_FULL_REGS(child->thread.regs);
if (index < PT_FPR0) { if (index < PT_FPR0) {
ret = ptrace_put_reg(child, index, data); ret = ptrace_put_reg(child, index, data);
} else { } else {
...@@ -307,6 +312,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -307,6 +312,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
*/ */
if ((addr & 3) || (numReg > PT_FPSCR)) if ((addr & 3) || (numReg > PT_FPSCR))
break; break;
CHECK_FULL_REGS(child->thread.regs);
if (numReg < PT_FPR0) { if (numReg < PT_FPR0) {
unsigned long freg = ptrace_get_reg(child, numReg); unsigned long freg = ptrace_get_reg(child, numReg);
if (index % 2) if (index % 2)
...@@ -342,6 +348,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -342,6 +348,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
ret = -EIO; ret = -EIO;
break; break;
} }
CHECK_FULL_REGS(child->thread.regs);
ret = 0; ret = 0;
for (ui = 0; ui < PT_REGS_COUNT; ui ++) { for (ui = 0; ui < PT_REGS_COUNT; ui ++) {
ret |= __put_user(ptrace_get_reg(child, ui), ret |= __put_user(ptrace_get_reg(child, ui),
...@@ -359,6 +366,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr, ...@@ -359,6 +366,7 @@ long compat_sys_ptrace(int request, int pid, unsigned long addr,
ret = -EIO; ret = -EIO;
break; break;
} }
CHECK_FULL_REGS(child->thread.regs);
ret = 0; ret = 0;
for (ui = 0; ui < PT_REGS_COUNT; ui ++) { for (ui = 0; ui < PT_REGS_COUNT; ui ++) {
ret = __get_user(tmp, (unsigned int __user *) data); ret = __get_user(tmp, (unsigned int __user *) data);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment