]> git.baikalelectronics.ru Git - kernel.git/commit
parisc: Fix ptrace syscall number and return value modification
authorHelge Deller <deller@gmx.de>
Tue, 19 Jan 2016 15:08:49 +0000 (16:08 +0100)
committerHelge Deller <deller@gmx.de>
Tue, 1 Mar 2016 22:06:07 +0000 (23:06 +0100)
commit8c5e823b2b567050b38e2970df59bbaae8c8ffe3
tree336ac5725ed95cac56829aaa3a0f3cea76cc176b
parentdb8ec0f93e68a396f1007889fd2c3363bed1abb3
parisc: Fix ptrace syscall number and return value modification

Mike Frysinger reported that his ptrace testcase showed strange
behaviour on parisc: It was not possible to avoid a syscall and the
return value of a syscall couldn't be changed.

To modify a syscall number, we were missing to save the new syscall
number to gr20 which is then picked up later in assembly again.

The effect that the return value couldn't be changed is a side-effect of
another bug in the assembly code. When a process is ptraced, userspace
expects each syscall to report entrance and exit of a syscall.  If a
syscall number was given which doesn't exist, we jumped to the normal
syscall exit code instead of informing userspace that the (non-existant)
syscall exits. This unexpected behaviour confuses userspace and thus the
bug was misinterpreted as if we can't change the return value.

This patch fixes both problems and was tested on 64bit kernel with
32bit userspace.

Signed-off-by: Helge Deller <deller@gmx.de>
Cc: Mike Frysinger <vapier@gentoo.org>
Cc: stable@vger.kernel.org # v4.0+
Tested-by: Mike Frysinger <vapier@gentoo.org>
arch/parisc/kernel/ptrace.c
arch/parisc/kernel/syscall.S