]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
syscall_get_arch: add "struct task_struct *" argument
authorDmitry V. Levin <ldv@altlinux.org>
Sun, 17 Mar 2019 23:30:18 +0000 (02:30 +0300)
committerPaul Moore <paul@paul-moore.com>
Thu, 21 Mar 2019 01:12:36 +0000 (21:12 -0400)
This argument is required to extend the generic ptrace API with
PTRACE_GET_SYSCALL_INFO request: syscall_get_arch() is going
to be called from ptrace_request() along with syscall_get_nr(),
syscall_get_arguments(), syscall_get_error(), and
syscall_get_return_value() functions with a tracee as their argument.

The primary intent is that the triple (audit_arch, syscall_nr, arg1..arg6)
should describe what system call is being called and what its arguments
are.

Reverts: 5e937a9ae913 ("syscall_get_arch: remove useless function arguments")
Reverts: 1002d94d3076 ("syscall.h: fix doc text for syscall_get_arch()")
Reviewed-by: Andy Lutomirski <luto@kernel.org> # for x86
Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
Acked-by: Paul Moore <paul@paul-moore.com>
Acked-by: Paul Burton <paul.burton@mips.com> # MIPS parts
Acked-by: Michael Ellerman <mpe@ellerman.id.au> (powerpc)
Acked-by: Kees Cook <keescook@chromium.org> # seccomp parts
Acked-by: Mark Salter <msalter@redhat.com> # for the c6x bit
Cc: Elvira Khabirova <lineprinter@altlinux.org>
Cc: Eugene Syromyatnikov <esyr@redhat.com>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: x86@kernel.org
Cc: linux-alpha@vger.kernel.org
Cc: linux-snps-arc@lists.infradead.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-c6x-dev@linux-c6x.org
Cc: uclinux-h8-devel@lists.sourceforge.jp
Cc: linux-hexagon@vger.kernel.org
Cc: linux-ia64@vger.kernel.org
Cc: linux-m68k@lists.linux-m68k.org
Cc: linux-mips@vger.kernel.org
Cc: nios2-dev@lists.rocketboards.org
Cc: openrisc@lists.librecores.org
Cc: linux-parisc@vger.kernel.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-riscv@lists.infradead.org
Cc: linux-s390@vger.kernel.org
Cc: linux-sh@vger.kernel.org
Cc: sparclinux@vger.kernel.org
Cc: linux-um@lists.infradead.org
Cc: linux-xtensa@linux-xtensa.org
Cc: linux-arch@vger.kernel.org
Cc: linux-audit@redhat.com
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Paul Moore <paul@paul-moore.com>
30 files changed:
arch/alpha/include/asm/syscall.h
arch/arc/include/asm/syscall.h
arch/arm/include/asm/syscall.h
arch/arm64/include/asm/syscall.h
arch/c6x/include/asm/syscall.h
arch/csky/include/asm/syscall.h
arch/h8300/include/asm/syscall.h
arch/hexagon/include/asm/syscall.h
arch/ia64/include/asm/syscall.h
arch/m68k/include/asm/syscall.h
arch/microblaze/include/asm/syscall.h
arch/mips/include/asm/syscall.h
arch/mips/kernel/ptrace.c
arch/nds32/include/asm/syscall.h
arch/nios2/include/asm/syscall.h
arch/openrisc/include/asm/syscall.h
arch/parisc/include/asm/syscall.h
arch/powerpc/include/asm/syscall.h
arch/riscv/include/asm/syscall.h
arch/s390/include/asm/syscall.h
arch/sh/include/asm/syscall_32.h
arch/sh/include/asm/syscall_64.h
arch/sparc/include/asm/syscall.h
arch/unicore32/include/asm/syscall.h
arch/x86/include/asm/syscall.h
arch/x86/um/asm/syscall.h
arch/xtensa/include/asm/syscall.h
include/asm-generic/syscall.h
kernel/auditsc.c
kernel/seccomp.c

index d73a6fcb519cb79d4ccb318aa76b0f30932de892..11c688c1d7ecf74cd116bbf6a9b926f50320f62c 100644 (file)
@@ -4,7 +4,7 @@
 
 #include <uapi/linux/audit.h>
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_ALPHA;
 }
index c7fc4c0c3bcb1bd5a7afb16f23e0213e208b3e37..caf2697ef5b7791596efd3693123f0c2b18aad72 100644 (file)
@@ -70,7 +70,7 @@ syscall_get_arguments(struct task_struct *task, struct pt_regs *regs,
 }
 
 static inline int
-syscall_get_arch(void)
+syscall_get_arch(struct task_struct *task)
 {
        return IS_ENABLED(CONFIG_ISA_ARCOMPACT)
                ? (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN)
index 06dea6bce293b934e1146d26aa316ea8e36e80b5..3940ceac0bdcbe1c7eb4431688779c587d3d9ed5 100644 (file)
@@ -104,7 +104,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        memcpy(&regs->ARM_r0 + i, args, n * sizeof(args[0]));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        /* ARM tasks don't change audit architectures on the fly. */
        return AUDIT_ARCH_ARM;
index ad8be16a39c9d18bdbd406f522c02432529c4cf6..1870df03f7743224d3163df3993f62697a5c774b 100644 (file)
@@ -117,9 +117,9 @@ static inline void syscall_set_arguments(struct task_struct *task,
  * We don't care about endianness (__AUDIT_ARCH_LE bit) here because
  * AArch64 has the same system calls both on little- and big- endian.
  */
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
-       if (is_compat_task())
+       if (is_compat_thread(task_thread_info(task)))
                return AUDIT_ARCH_ARM;
 
        return AUDIT_ARCH_AARCH64;
index 39dbd1ef994c8a6deb545ba07d410845789e20c7..595057191c9cc9245e2287ee7aa03e3e84c8552f 100644 (file)
@@ -121,7 +121,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        }
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return IS_ENABLED(CONFIG_CPU_BIG_ENDIAN)
                ? AUDIT_ARCH_C6XBE : AUDIT_ARCH_C6X;
index d637445737b78fd5c78c9994173a1e7c73eb3d1f..150ffb894fa205ceeeb399fb0eba03a5fc5fafc2 100644 (file)
@@ -70,7 +70,7 @@ syscall_set_arguments(struct task_struct *task, struct pt_regs *regs,
 }
 
 static inline int
-syscall_get_arch(void)
+syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_CSKY;
 }
index 5135910616e24908f5dc37308f566c3b36c0e8b4..d316c3d40d4e3fce2830df039876d96b81594145 100644 (file)
@@ -49,7 +49,7 @@ syscall_get_arguments(struct task_struct *task, struct pt_regs *regs,
 }
 
 static inline int
-syscall_get_arch(void)
+syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_H8300;
 }
index de3917aad3fd0921730c309abfb623d4c26f01d7..47b0bc3f16be5a1ad7d8f598277e2a8782092ad5 100644 (file)
@@ -46,7 +46,7 @@ static inline void syscall_get_arguments(struct task_struct *task,
        memcpy(args, &(&regs->r00)[i], n * sizeof(args[0]));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_HEXAGON;
 }
index 1d0b875fec44fc0fd8a70876e8ee6191f66f6005..47ab33f5448aa623b9946c80215f091d5cc6d2f5 100644 (file)
@@ -81,7 +81,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        ia64_syscall_get_set_arguments(task, regs, i, n, args, 1);
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_IA64;
 }
index d4d7deda8d50b5fdd894d3274400787780d27e9b..465ac039be09a1b8ccbb34a4fc940cff6a4e36d5 100644 (file)
@@ -4,7 +4,7 @@
 
 #include <uapi/linux/audit.h>
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_M68K;
 }
index 220decd605a4aded46a99b445e54bf27c4adc821..77a86fafa974f4e746800d7ecf13c671a65cf2d5 100644 (file)
@@ -101,7 +101,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
 asmlinkage unsigned long do_syscall_trace_enter(struct pt_regs *regs);
 asmlinkage void do_syscall_trace_leave(struct pt_regs *regs);
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_MICROBLAZE;
 }
index 6cf8ffb5367ec3fb725aac26c701d0ae5d81923c..6a22c9352ef6af5199a658f178dab6d49dcea128 100644 (file)
@@ -140,14 +140,14 @@ extern const unsigned long sys_call_table[];
 extern const unsigned long sys32_call_table[];
 extern const unsigned long sysn32_call_table[];
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        int arch = AUDIT_ARCH_MIPS;
 #ifdef CONFIG_64BIT
-       if (!test_thread_flag(TIF_32BIT_REGS)) {
+       if (!test_tsk_thread_flag(task, TIF_32BIT_REGS)) {
                arch |= __AUDIT_ARCH_64BIT;
                /* N32 sets only TIF_32BIT_ADDR */
-               if (test_thread_flag(TIF_32BIT_ADDR))
+               if (test_tsk_thread_flag(task, TIF_32BIT_ADDR))
                        arch |= __AUDIT_ARCH_CONVENTION_MIPS64_N32;
        }
 #endif
index 0057c910bc2f34de0f518c43d2e234c845db0da1..2ead6ff919b768e8c75aa47aa811b8dc474de4b1 100644 (file)
@@ -1418,7 +1418,7 @@ asmlinkage long syscall_trace_enter(struct pt_regs *regs, long syscall)
                unsigned long args[6];
 
                sd.nr = syscall;
-               sd.arch = syscall_get_arch();
+               sd.arch = syscall_get_arch(current);
                syscall_get_arguments(current, regs, 0, 6, args);
                for (i = 0; i < 6; i++)
                        sd.args[i] = args[i];
index cc56a3962f8b164415468d2aad1193a4b23286b6..7501e376a6b137c95480c79e87e82f19ee8674c3 100644 (file)
@@ -188,7 +188,7 @@ void syscall_set_arguments(struct task_struct *task, struct pt_regs *regs,
 }
 
 static inline int
-syscall_get_arch(void)
+syscall_get_arch(struct task_struct *task)
 {
        return IS_ENABLED(CONFIG_CPU_BIG_ENDIAN)
                ? AUDIT_ARCH_NDS32BE : AUDIT_ARCH_NDS32;
index cf35e210fc4dafeb787bb0aa6d2b3502a98b0a63..f0f6ae208e78980b72e469018ddf30ea50a282ed 100644 (file)
@@ -136,7 +136,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        }
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_NIOS2;
 }
index 2db9f1cf0694c0f2c6bdaec77953f62fb4fe6372..46b10c674bd203e85a09632a0235ff5f6988325d 100644 (file)
@@ -72,7 +72,7 @@ syscall_set_arguments(struct task_struct *task, struct pt_regs *regs,
        memcpy(&regs->gpr[3 + i], args, n * sizeof(args[0]));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_OPENRISC;
 }
index 8bff1a58c97f1b107dabf79e172f5ecb56c5db2d..c04ffc6ac92840abcf56c1211465930ef7dd8c11 100644 (file)
@@ -62,11 +62,11 @@ static inline void syscall_rollback(struct task_struct *task,
        /* do nothing */
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        int arch = AUDIT_ARCH_PARISC;
 #ifdef CONFIG_64BIT
-       if (!is_compat_task())
+       if (!__is_compat_task(task))
                arch = AUDIT_ARCH_PARISC64;
 #endif
        return arch;
index 1a0e7a8b1c811cf5d089c5ac68eb96d189ad702d..efb50429c9f4ba2aa65c599b780f78db14fae33a 100644 (file)
@@ -99,9 +99,15 @@ static inline void syscall_set_arguments(struct task_struct *task,
                regs->orig_gpr3 = args[0];
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
-       int arch = is_32bit_task() ? AUDIT_ARCH_PPC : AUDIT_ARCH_PPC64;
+       int arch;
+
+       if (IS_ENABLED(CONFIG_PPC64) && !test_tsk_thread_flag(task, TIF_32BIT))
+               arch = AUDIT_ARCH_PPC64;
+       else
+               arch = AUDIT_ARCH_PPC;
+
 #ifdef __LITTLE_ENDIAN__
        arch |= __AUDIT_ARCH_LE;
 #endif
index bba3da6ef1572f41db64e59ca203ae32b9139180..ca120a36a037a2b99fa200dd23ad9e58699bb1e5 100644 (file)
@@ -100,7 +100,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        memcpy(&regs->a1 + i * sizeof(regs->a1), args, n * sizeof(regs->a0));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
 #ifdef CONFIG_64BIT
        return AUDIT_ARCH_RISCV64;
index 96f9a9151fde02fc6f76633d76d292f47512d364..5a40ea8b90ea88e2639179a2d264810554f44b0e 100644 (file)
@@ -92,10 +92,10 @@ static inline void syscall_set_arguments(struct task_struct *task,
                regs->orig_gpr2 = args[0];
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
 #ifdef CONFIG_COMPAT
-       if (test_tsk_thread_flag(current, TIF_31BIT))
+       if (test_tsk_thread_flag(task, TIF_31BIT))
                return AUDIT_ARCH_S390;
 #endif
        return AUDIT_ARCH_S390X;
index 6e118799831c32dc37b8cf21960d284c5bec3646..08de429eccd4228e3c12ee810b68e19a94dee742 100644 (file)
@@ -95,7 +95,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        }
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        int arch = AUDIT_ARCH_SH;
 
index 43882580c7f99bec93e519f1b4182c1daad2fbf0..9b62a2404531a3cfa51c53f76d4b7191512b16e7 100644 (file)
@@ -63,7 +63,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        memcpy(&regs->regs[2 + i], args, n * sizeof(args[0]));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        int arch = AUDIT_ARCH_SH;
 
index 053989e3f6a6f1435323873ea010723ac09736bd..9ffb367c17fd8c3da629839aaa5fd34b966b7721 100644 (file)
@@ -128,10 +128,11 @@ static inline void syscall_set_arguments(struct task_struct *task,
                regs->u_regs[UREG_I0 + i + j] = args[j];
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
 #if defined(CONFIG_SPARC64) && defined(CONFIG_COMPAT)
-       return in_compat_syscall() ? AUDIT_ARCH_SPARC : AUDIT_ARCH_SPARC64;
+       return test_tsk_thread_flag(task, TIF_32BIT)
+               ? AUDIT_ARCH_SPARC : AUDIT_ARCH_SPARC64;
 #elif defined(CONFIG_SPARC64)
        return AUDIT_ARCH_SPARC64;
 #else
index 3a6b885476b40d654ef61e3abf1eeca97442e869..607961797fff1dd4ccc410704ee502c7ecc57dad 100644 (file)
@@ -4,7 +4,7 @@
 
 #include <uapi/linux/audit.h>
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_UNICORE;
 }
index d653139857af2a1121f877b611c8d56d4e4690f0..435f3f09279c3825a7e718c58c9ba93e2f7e8726 100644 (file)
@@ -107,7 +107,7 @@ static inline void syscall_set_arguments(struct task_struct *task,
        memcpy(&regs->bx + i, args, n * sizeof(args[0]));
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_I386;
 }
@@ -236,10 +236,12 @@ static inline void syscall_set_arguments(struct task_struct *task,
                }
 }
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        /* x32 tasks should be considered AUDIT_ARCH_X86_64. */
-       return in_ia32_syscall() ? AUDIT_ARCH_I386 : AUDIT_ARCH_X86_64;
+       return (IS_ENABLED(CONFIG_IA32_EMULATION) &&
+               task->thread_info.status & TS_COMPAT)
+               ? AUDIT_ARCH_I386 : AUDIT_ARCH_X86_64;
 }
 #endif /* CONFIG_X86_32 */
 
index ef898af102d164270cb27f0655780df60a45c1a9..56a2f0913e3c0307958eb47815a7634447cb1390 100644 (file)
@@ -9,7 +9,7 @@ typedef asmlinkage long (*sys_call_ptr_t)(unsigned long, unsigned long,
                                          unsigned long, unsigned long,
                                          unsigned long, unsigned long);
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
 #ifdef CONFIG_X86_32
        return AUDIT_ARCH_I386;
index a168bf81c7f4701a036abaa251fa5dce7fad6c19..0681ca656809bf6606bd8cb14578cd839cadb138 100644 (file)
@@ -14,7 +14,7 @@
 #include <asm/ptrace.h>
 #include <uapi/linux/audit.h>
 
-static inline int syscall_get_arch(void)
+static inline int syscall_get_arch(struct task_struct *task)
 {
        return AUDIT_ARCH_XTENSA;
 }
index 0c938a4354f6f58f67e0bb3555dca24460afadb7..e0d060b4332152120b03adca104bcb93a9fd9214 100644 (file)
@@ -144,14 +144,15 @@ void syscall_set_arguments(struct task_struct *task, struct pt_regs *regs,
 
 /**
  * syscall_get_arch - return the AUDIT_ARCH for the current system call
+ * @task:      task of interest, must be blocked
  *
  * Returns the AUDIT_ARCH_* based on the system call convention in use.
  *
- * It's only valid to call this when current is stopped on entry to a system
+ * It's only valid to call this when @task is stopped on entry to a system
  * call, due to %TIF_SYSCALL_TRACE, %TIF_SYSCALL_AUDIT, or %TIF_SECCOMP.
  *
  * Architectures which permit CONFIG_HAVE_ARCH_SECCOMP_FILTER must
  * provide an implementation of this.
  */
-int syscall_get_arch(void);
+int syscall_get_arch(struct task_struct *task);
 #endif /* _ASM_SYSCALL_H */
index 17b0007fafc25cd6b302ef4549706eaa95d1d4f1..98a98e6dca0508382815f24dca0771199aac46ed 100644 (file)
@@ -1636,7 +1636,7 @@ void __audit_syscall_entry(int major, unsigned long a1, unsigned long a2,
                        return;
        }
 
-       context->arch       = syscall_get_arch();
+       context->arch       = syscall_get_arch(current);
        context->major      = major;
        context->argv[0]    = a1;
        context->argv[1]    = a2;
@@ -2590,7 +2590,7 @@ void audit_seccomp(unsigned long syscall, long signr, int code)
                return;
        audit_log_task(ab);
        audit_log_format(ab, " sig=%ld arch=%x syscall=%ld compat=%d ip=0x%lx code=0x%x",
-                        signr, syscall_get_arch(), syscall,
+                        signr, syscall_get_arch(current), syscall,
                         in_compat_syscall(), KSTK_EIP(current), code);
        audit_log_end(ab);
 }
index 54a0347ca8128f09cdbbcc83e2e8f8eea633a7ab..36f36ab00f488754bbb2ecac1b236c7589ebfc8a 100644 (file)
@@ -148,7 +148,7 @@ static void populate_seccomp_data(struct seccomp_data *sd)
        unsigned long args[6];
 
        sd->nr = syscall_get_nr(task, regs);
-       sd->arch = syscall_get_arch();
+       sd->arch = syscall_get_arch(task);
        syscall_get_arguments(task, regs, 0, 6, args);
        sd->args[0] = args[0];
        sd->args[1] = args[1];
@@ -591,7 +591,7 @@ static void seccomp_init_siginfo(kernel_siginfo_t *info, int syscall, int reason
        info->si_code = SYS_SECCOMP;
        info->si_call_addr = (void __user *)KSTK_EIP(current);
        info->si_errno = reason;
-       info->si_arch = syscall_get_arch();
+       info->si_arch = syscall_get_arch(current);
        info->si_syscall = syscall;
 }