[PATCH] ARM/audit: use correct arch on audit_syscall_enter
Will Deacon
will.deacon at arm.com
Tue Feb 21 09:58:22 EST 2012
Hi Eric,
On Tue, Feb 21, 2012 at 01:47:39PM +0000, Eric Paris wrote:
> ARM has both LE and BE versions however the audit code was called as if
> it was always BE. If audit userspace were to try to interpret the bits
> it got from a LE system it would obviously do so incorrectly. Fix this
> by using the right arch flag on the right system.
>
> Signed-off-by: Eric Paris <eparis at redhat.com>
> ---
> arch/arm/kernel/ptrace.c | 7 ++++++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
It might be worth adding the #include <linux/audit.h> in this patch as well,
since they're both fixes for problems introduced by the same commit. That
way we only need to deal with one patch as well.
> diff --git a/arch/arm/kernel/ptrace.c b/arch/arm/kernel/ptrace.c
> index e33870f..8009e84 100644
> --- a/arch/arm/kernel/ptrace.c
> +++ b/arch/arm/kernel/ptrace.c
> @@ -915,10 +915,15 @@ asmlinkage int syscall_trace(int why, struct pt_regs *regs, int scno)
> ip = regs->ARM_ip;
> regs->ARM_ip = why;
>
> +#ifdef __ARMEB__
> +#define AUDIT_ARCH_NR AUDIT_ARCH_ARMEB
> +#else
> +#define AUDIT_ARCH_NR AUDIT_ARCH_ARM
> +#endif
Pedantry, but I prefer the #defines outside of the function scope (I'd probably
just stick them before the function given that they're not used anywhere else).
> if (!ip)
> audit_syscall_exit(regs);
> else
> - audit_syscall_entry(AUDIT_ARCH_ARMEB, scno, regs->ARM_r0,
> + audit_syscall_entry(AUDIT_ARCH_NR, scno, regs->ARM_r0,
> regs->ARM_r1, regs->ARM_r2, regs->ARM_r3);
>
> if (!test_thread_flag(TIF_SYSCALL_TRACE))
Cheers,
Will
More information about the linux-arm-kernel
mailing list