Chromium Code Reviews| Index: sandbox/linux/seccomp-bpf/trap.cc |
| diff --git a/sandbox/linux/seccomp-bpf/trap.cc b/sandbox/linux/seccomp-bpf/trap.cc |
| index 4c42111c2b07659f5c7ed9358655e30d07c952cd..e3a8b52c1835d6c375ddc481b7d42bdd780cc2f1 100644 |
| --- a/sandbox/linux/seccomp-bpf/trap.cc |
| +++ b/sandbox/linux/seccomp-bpf/trap.cc |
| @@ -150,9 +150,19 @@ void Trap::SigSys(int nr, siginfo_t* info, void* void_context) { |
| struct arch_sigsys sigsys; |
| memcpy(&sigsys, &info->_sifields, sizeof(sigsys)); |
| +#if defined(__mips__) |
| + // When indirect syscall (syscall(__NR_foo, ...)) is made on Mips, the |
| + // number in register SECCOMP_SYSCALL(ctx) is always __NR_syscall and the |
| + // real number of a syscall (__NR_foo) is in SECCOMP_PARM1(ctx) |
| + bool sigsys_nr_is_bad = sigsys.nr != static_cast<int>(SECCOMP_SYSCALL(ctx)) |
| + && sigsys.nr != static_cast<int>(SECCOMP_PARM1(ctx)); |
|
jln (very slow on Chromium)
2014/06/20 00:37:06
Nit: indent. (please use "git cl format")
nedeljko
2014/06/20 14:09:51
Done.
|
| +#else |
| + bool sigsys_nr_is_bad = sigsys.nr != static_cast<int>(SECCOMP_SYSCALL(ctx)); |
| +#endif |
| + |
| // Some more sanity checks. |
| if (sigsys.ip != reinterpret_cast<void*>(SECCOMP_IP(ctx)) || |
| - sigsys.nr != static_cast<int>(SECCOMP_SYSCALL(ctx)) || |
| + sigsys_nr_is_bad || |
| sigsys.arch != SECCOMP_ARCH) { |
| // TODO(markus): |
| // SANDBOX_DIE() can call LOG(FATAL). This is not normally async-signal |
| @@ -168,7 +178,7 @@ void Trap::SigSys(int nr, siginfo_t* info, void* void_context) { |
| if (sigsys.nr == __NR_clone) { |
| RAW_SANDBOX_DIE("Cannot call clone() from an UnsafeTrap() handler."); |
| } |
| - rc = Syscall::Call(sigsys.nr, |
| + rc = Syscall::Call(SECCOMP_SYSCALL(ctx), |
| SECCOMP_PARM1(ctx), |
| SECCOMP_PARM2(ctx), |
| SECCOMP_PARM3(ctx), |
| @@ -185,7 +195,9 @@ void Trap::SigSys(int nr, siginfo_t* info, void* void_context) { |
| // is what we are showing to TrapFnc callbacks that the system call |
| // evaluator registered with the sandbox. |
| struct arch_seccomp_data data = { |
| - sigsys.nr, SECCOMP_ARCH, reinterpret_cast<uint64_t>(sigsys.ip), |
| + static_cast<int> SECCOMP_SYSCALL(ctx), |
| + SECCOMP_ARCH, |
| + reinterpret_cast<uint64_t>(sigsys.ip), |
| {static_cast<uint64_t>(SECCOMP_PARM1(ctx)), |
| static_cast<uint64_t>(SECCOMP_PARM2(ctx)), |
| static_cast<uint64_t>(SECCOMP_PARM3(ctx)), |
| @@ -201,7 +213,7 @@ void Trap::SigSys(int nr, siginfo_t* info, void* void_context) { |
| // Update the CPU register that stores the return code of the system call |
| // that we just handled, and restore "errno" to the value that it had |
| // before entering the signal handler. |
| - SECCOMP_RESULT(ctx) = static_cast<greg_t>(rc); |
| + Syscall::PutValueInUcontext(rc, ctx); |
| errno = old_errno; |
| return; |