commit fc30bfe2a4b1a84465b4c80f3159d7d714c08d37
author Elliott Hughes <enh@google.com> Tue Oct 24 22:54:34 2017 -0700
committer Elliott Hughes <enh@google.com> Tue Oct 24 22:54:34 2017 -0700
tree 476f6a982c11885624a15f2bdff25189f4d7f3ec
parent 88c5ddb6d20d8eec56bfff3a209a86ef2bfce60b

Don't try to link(2) in a test if we're not root.

Bug: http://b/68055982
Test: ran tests
Change-Id: Icdbe3dd58fb84f3661e5d9ed3ff6884cc0ea2ff8

tests/fcntl_test.cpp

diff --git a/tests/fcntl_test.cpp b/tests/fcntl_test.cpp
index cb00c3a..0a83dff 100644
--- a/tests/fcntl_test.cpp
+++ b/tests/fcntl_test.cpp
@@ -319,6 +319,9 @@
   ASSERT_EQ(0, fstat(fd, &sb));
   ASSERT_EQ(perms, (sb.st_mode & ~S_IFMT));
 
+  // On Android if we're not root, we won't be able to create links anyway...
+  if (getuid() != 0) return;
+
   std::string final_path = android::base::StringPrintf("%s/named_now", dir.dirname);
   ASSERT_EQ(0, linkat(AT_FDCWD, android::base::StringPrintf("/proc/self/fd/%d", fd).c_str(),
                       AT_FDCWD, final_path.c_str(),