Get gid of created folders from stat()ing the folders, not from getgid().

base/file_util_unittest.cc

 // Copyright (c) 2011 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.
 
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #include <shlobj.h>
@@ skipping 1845 matching lines @@
 
     base_dir_ = temp_dir_.path().AppendASCII("base_dir");
     ASSERT_TRUE(file_util::CreateDirectory(base_dir_));
 
     sub_dir_ = base_dir_.AppendASCII("sub_dir");
     ASSERT_TRUE(file_util::CreateDirectory(sub_dir_));
 
     text_file_ = sub_dir_.AppendASCII("file.txt");
     CreateTextFile(text_file_, L"This text file has some text in it.");
 
-    // Our user and group id.
-    uid_ = getuid();
-    gid_ = getgid();
+    // Get the user and group files are created with from |base_dir_|.
+    struct stat stat_buf;
+    ASSERT_EQ(0, stat(base_dir_.value().c_str(), &stat_buf));
+    uid_ = stat_buf.st_uid;
+    gid_ = stat_buf.st_gid;
+    ASSERT_EQ(uid_, getuid());  // This process should be the owner.
 
     // To ensure that umask settings do not cause the initial state
     // of permissions to be different from what we expect, explicitly
     // set permissions on the directories we create.
     // Make all files and directories non-world-writable.
     mode_t enabled_permissions =
         S_IRWXU | // User can read, write, traverse
         S_IRWXG;  // Group can read, write, traverse
     mode_t disabled_permissions =
         S_IRWXO;  // Other users can't read, write, traverse.
 
     ASSERT_NO_FATAL_FAILURE(
         ChangePosixFilePermissions(
             base_dir_, enabled_permissions, disabled_permissions));
     ASSERT_NO_FATAL_FAILURE(
         ChangePosixFilePermissions(
             sub_dir_, enabled_permissions, disabled_permissions));
   }
 
   FilePath base_dir_;
   FilePath sub_dir_;
   FilePath text_file_;
   uid_t uid_;
   gid_t gid_;
 };
 
-#if defined(OS_MACOSX)
-// http://crbug.com/97876
-#define MAYBE_BadPaths FAILS_BadPaths
-#else
-#define MAYBE_BadPaths BadPaths
-#endif
-
-TEST_F(VerifyPathControlledByUserTest, MAYBE_BadPaths) {
+TEST_F(VerifyPathControlledByUserTest, BadPaths) {
   // File does not exist.
   FilePath does_not_exist = base_dir_.AppendASCII("does")
                                      .AppendASCII("not")
                                      .AppendASCII("exist");
 
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(
           base_dir_, does_not_exist, uid_, gid_));
 
   // |base| not a subpath of |path|.
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(sub_dir_, base_dir_, uid_, gid_));
 
   // An empty base path will fail to be a prefix for any path.
   FilePath empty;
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(empty, base_dir_, uid_, gid_));
 
   // Finding that a bad call fails proves nothing unless a good call succeeds.
   EXPECT_TRUE(
       file_util::VerifyPathControlledByUser(base_dir_, sub_dir_, uid_, gid_));
 }
 
-#if defined(OS_MACOSX)
-// http://crbug.com/97876
-#define MAYBE_Symlinks FAILS_Symlinks
-#else
-#define MAYBE_Symlinks Symlinks
-#endif
-
 TEST_F(VerifyPathControlledByUserTest, Symlinks) {
   // Symlinks in the path should cause failure.
 
   // Symlink to the file at the end of the path.
   FilePath file_link =  base_dir_.AppendASCII("file_link");
   ASSERT_TRUE(file_util::CreateSymbolicLink(text_file_, file_link))
       << "Failed to create symlink.";
 
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(base_dir_, file_link, uid_, gid_));
@@ skipping 15 matching lines @@
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(
           link_to_sub_dir, file_path_with_link, uid_, gid_));
 
   // Symlinks in parents of base path are allowed.
   EXPECT_TRUE(
       file_util::VerifyPathControlledByUser(
           file_path_with_link, file_path_with_link, uid_, gid_));
 }
 
-#if defined(OS_MACOSX)
-// http://crbug.com/97876
-#define MAYBE_OwnershipChecks FAILS_OwnershipChecks
-#else
-#define MAYBE_OwnershipChecks OwnershipChecks
-#endif
-
-TEST_F(VerifyPathControlledByUserTest, MAYBE_OwnershipChecks) {
+TEST_F(VerifyPathControlledByUserTest, OwnershipChecks) {
   // Get a uid that is not the uid of files we create.
   uid_t bad_uid = uid_ + 1;
 
   // Get a gid that is not ours.
   gid_t bad_gid = gid_ + 1;
 
   // Make all files and directories non-world-writable.
   ASSERT_NO_FATAL_FAILURE(
       ChangePosixFilePermissions(base_dir_, 0u, S_IWOTH));
   ASSERT_NO_FATAL_FAILURE(
@@ skipping 25 matching lines @@
       file_util::VerifyPathControlledByUser(
           base_dir_, sub_dir_, uid_, bad_gid));
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(
           base_dir_, text_file_, uid_, bad_gid));
   EXPECT_FALSE(
       file_util::VerifyPathControlledByUser(
           sub_dir_, text_file_, uid_, bad_gid));
 }
 
-#if defined(OS_MACOSX)
-// http://crbug.com/97876
-#define MAYBE_WriteBitChecks FAILS_WriteBitChecks
-#else
-#define MAYBE_WriteBitChecks WriteBitChecks
-#endif
-
-TEST_F(VerifyPathControlledByUserTest, MAYBE_WriteBitChecks) {
+TEST_F(VerifyPathControlledByUserTest, WriteBitChecks) {
   // Make all files and directories non-world-writable.
   ASSERT_NO_FATAL_FAILURE(
       ChangePosixFilePermissions(base_dir_, 0u, S_IWOTH));
   ASSERT_NO_FATAL_FAILURE(
       ChangePosixFilePermissions(sub_dir_, 0u, S_IWOTH));
   ASSERT_NO_FATAL_FAILURE(
       ChangePosixFilePermissions(text_file_, 0u, S_IWOTH));
 
   // Initialy, we control all parts of the path.
   EXPECT_TRUE(
@@ skipping 61 matching lines @@
       file_util::VerifyPathControlledByUser(base_dir_, sub_dir_, uid_, gid_));
   EXPECT_TRUE(
       file_util::VerifyPathControlledByUser(base_dir_, text_file_, uid_, gid_));
   EXPECT_TRUE(
       file_util::VerifyPathControlledByUser(sub_dir_, text_file_, uid_, gid_));
 }
 
 #endif  // defined(OS_POSIX)
 
 }  // namespace