Fix duplicate symbol in ARM build of nacl_helper_nonsfi_unittests

components/nacl/loader/nonsfi/nonsfi_sandbox_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Sanitizers internally use some syscalls which non-SFI NaCl disallows.
 #if !defined(ADDRESS_SANITIZER) && !defined(THREAD_SANITIZER) && \
     !defined(MEMORY_SANITIZER) && !defined(LEAK_SANITIZER)
 
 #include "components/nacl/loader/nonsfi/nonsfi_sandbox.h"
 
@@ skipping 184 matching lines @@
 BPF_DEATH_TEST_C(NaClNonSfiSandboxTest,
                  prctl_SET_DUMPABLE,
                  DEATH_SEGV_MESSAGE(
                      sandbox::GetPrctlErrorMessageContentForTests()),
                  nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
   syscall(__NR_prctl, PR_SET_DUMPABLE, 1UL);
 }
 
 #if defined(OS_NACL_NONSFI)
 BPF_DEATH_TEST_C(NaClNonsfiSandboxTest,
-                 socketpair,
+                 socketpair_af_unix_disallowed,
                  DEATH_SEGV_MESSAGE(sandbox::GetErrorMessageContentForTests()),
                  nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
   int tmp_fds[2];
   socketpair(AF_UNIX, SOCK_STREAM, 0, tmp_fds);
 }
 #else
 BPF_TEST_C(NaClNonSfiSandboxTest,
            socketcall_allowed,
            nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
   base::ScopedFD fds[2];
   struct msghdr msg = {};
   struct iovec iov;
   std::string payload("foo");
   iov.iov_base = &payload[0];
   iov.iov_len = payload.size();
   msg.msg_iov = &iov;
   msg.msg_iovlen = 1;
   DoSocketpair(fds);
   BPF_ASSERT_EQ(static_cast<int>(payload.size()),
                 HANDLE_EINTR(sendmsg(fds[1].get(), &msg, 0)));
   BPF_ASSERT_EQ(static_cast<int>(payload.size()),
                 HANDLE_EINTR(recvmsg(fds[0].get(), &msg, 0)));
   BPF_ASSERT_EQ(0, shutdown(fds[0].get(), SHUT_RDWR));
 }
 #endif
 
+// On arm and x86_64 the argument to socketpair are passed in registers

[Mark Seaborn, 2015/05/28 08:44:10]

Nit: "arguments ... are"

+// so that can be filtered by seccomp-bpf.  This filter cannot be applied

[Mark Seaborn, 2015/05/28 08:44:10]

"...registers so that" -> "...registers, so they"

+// on x86_32 as they arguments are passed in memory.

[Mark Seaborn, 2015/05/28 08:44:10]

"the arguments"

+#if defined(__x86_64__) || defined(__arm__)
+BPF_DEATH_TEST_C(NaClNonSfiSandboxTest,
+                 socketpair_af_inet_disallowed,
+                 DEATH_SEGV_MESSAGE(sandbox::GetErrorMessageContentForTests()),
+                 nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
+  int fds[2];
+  socketpair(AF_INET, SOCK_STREAM, 0, fds);
+}
+#endif
+
 BPF_DEATH_TEST_C(NaClNonSfiSandboxTest,
                  accept,
                  DEATH_SEGV_MESSAGE(sandbox::GetErrorMessageContentForTests()),
                  nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
 #if defined(__i386__)
   uintptr_t args[] = {0, 0, 0};
   syscall(__NR_socketcall, SYS_ACCEPT, args);
 #else
   syscall(__NR_accept, 0, 0, 0);
 #endif
@@ skipping 144 matching lines @@
                  DEATH_SEGV_MESSAGE(sandbox::GetErrorMessageContentForTests()),
                  nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
 #if defined(__i386__)
   uintptr_t args[] = {0, 0, 0};
   syscall(__NR_socketcall, SYS_SOCKET, args);
 #else
   syscall(__NR_socket, 0, 0, 0);
 #endif
 }
 
-#if defined(__x86_64__) || defined(__arm__)
-BPF_DEATH_TEST_C(NaClNonSfiSandboxTest,
-                 socketpair,
-                 DEATH_SEGV_MESSAGE(sandbox::GetErrorMessageContentForTests()),
-                 nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
-  int fds[2];
-  socketpair(AF_INET, SOCK_STREAM, 0, fds);
-}
-#endif
-
 BPF_TEST_C(NaClNonSfiSandboxTest,
            fcntl_SETFD_allowed,
            nacl::nonsfi::NaClNonSfiBPFSandboxPolicy) {
   base::ScopedFD fds[2];
   DoPipe(fds);
   BPF_ASSERT_EQ(0, fcntl(fds[0].get(), F_SETFD, FD_CLOEXEC));
 }
 
 BPF_DEATH_TEST_C(NaClNonSfiSandboxTest,
                  fcntl_SETFD,
@@ skipping 258 matching lines @@
 RESTRICT_SYSCALL_EPERM_TEST(ptrace);
 RESTRICT_SYSCALL_EPERM_TEST(set_robust_list);
 #if defined(__i386__) || defined(__x86_64__)
 RESTRICT_SYSCALL_EPERM_TEST(time);
 #endif
 
 }  // namespace
 
 #endif  // !ADDRESS_SANITIZER && !THREAD_SANITIZER &&
         // !MEMORY_SANITIZER && !LEAK_SANITIZER