Am 21.12.2015 um 01:03 schrieb Mickaël Salaün: > This fix two related bugs: > * PTRACE_GETREGS doesn't get the right orig_ax (syscall) value > * PTRACE_SETREGS can't set the orig_ax value (erased by initial value) > > Remove the now useless and error-prone get_syscall(). > > Signed-off-by: Mickaël Salaün <m...@digikod.net> > Cc: Jeff Dike <jd...@addtoit.com> > Cc: Richard Weinberger <rich...@nod.at> > Cc: Thomas Gleixner <t...@linutronix.de> > Cc: Kees Cook <keesc...@chromium.org> > Cc: Andy Lutomirski <l...@amacapital.net> > Cc: Will Drewry <w...@chromium.org> > Cc: Thomas Meyer <tho...@m3y3r.de> > Cc: Nicolas Iooss <nicolas.iooss_li...@m4x.org> > Cc: Anton Ivanov <aiva...@brocade.com> > Cc: linux-kernel@vger.kernel.org > Cc: user-mode-linux-de...@lists.sourceforge.net > Cc: user-mode-linux-u...@lists.sourceforge.net > Cc: Meredydd Luff <mered...@senatehouse.org> > Cc: David Drysdale <drysd...@google.com> > --- > arch/um/kernel/skas/syscall.c | 7 ++++++- > arch/um/os-Linux/skas/process.c | 7 ------- > 2 files changed, 6 insertions(+), 8 deletions(-) > > diff --git a/arch/um/kernel/skas/syscall.c b/arch/um/kernel/skas/syscall.c > index 1683b8e..65f0d1a 100644 > --- a/arch/um/kernel/skas/syscall.c > +++ b/arch/um/kernel/skas/syscall.c > @@ -7,6 +7,7 @@ > #include <linux/ptrace.h> > #include <kern_util.h> > #include <sysdep/ptrace.h> > +#include <sysdep/ptrace_user.h> > #include <sysdep/syscalls.h> > #include <os.h> > > @@ -16,12 +17,16 @@ void handle_syscall(struct uml_pt_regs *r) > long result; > int syscall; > > + /* Save the syscall register. */ > + UPT_SYSCALL_NR(r) = PT_SYSCALL_NR(r->gp); > + > if (syscall_trace_enter(regs)) { > result = -ENOSYS; > goto out; > } > > - syscall = get_syscall(r); > + /* Get the syscall after being potentially updated with ptrace. */ > + syscall = UPT_SYSCALL_NR(r);
Doesn't this break the support for changing syscall numbers using PTRACE_SETREGS? Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/