mirror of
https://github.com/hanzoai/visor.git
synced 2026-09-24 17:04:14 +00:00
Replace raw hex literals with VFS_CAP_REVISION_2 and VFS_CAP_FLAGS_EFFECTIVE from linux/capability.h in the capability xattr tests. Add the missing #include <string> in chown.cc that was blocking the internal linter.
400 lines
14 KiB
C++
400 lines
14 KiB
C++
// Copyright 2018 The gVisor Authors.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#include <errno.h>
|
|
#include <fcntl.h>
|
|
#include <grp.h>
|
|
#include <linux/capability.h>
|
|
#include <stdint.h>
|
|
#include <sys/types.h>
|
|
#include <sys/xattr.h>
|
|
#include <unistd.h>
|
|
|
|
#include <string>
|
|
#include <vector>
|
|
|
|
#include "gmock/gmock.h"
|
|
#include "gtest/gtest.h"
|
|
#include "absl/flags/flag.h"
|
|
#include "absl/synchronization/notification.h"
|
|
#include "test/util/file_descriptor.h"
|
|
#include "test/util/fs_util.h"
|
|
#include "test/util/linux_capability_util.h"
|
|
#include "test/util/posix_error.h"
|
|
#include "test/util/temp_path.h"
|
|
#include "test/util/test_util.h"
|
|
#include "test/util/thread_util.h"
|
|
|
|
ABSL_FLAG(int32_t, scratch_uid1, 65534, "first scratch UID");
|
|
ABSL_FLAG(int32_t, scratch_uid2, 65533, "second scratch UID");
|
|
ABSL_FLAG(int32_t, scratch_gid, 65534, "first scratch GID");
|
|
|
|
namespace gvisor {
|
|
namespace testing {
|
|
|
|
namespace {
|
|
|
|
struct VfsCapData {
|
|
uint32_t magic_etc;
|
|
uint32_t permitted_lo;
|
|
uint32_t inheritable_lo;
|
|
uint32_t permitted_hi;
|
|
uint32_t inheritable_hi;
|
|
};
|
|
|
|
int SetSecurityCapability(const std::string& path) {
|
|
VfsCapData cap_data = {};
|
|
cap_data.magic_etc = VFS_CAP_REVISION_2 | VFS_CAP_FLAGS_EFFECTIVE;
|
|
cap_data.permitted_lo = 1u << CAP_SETUID;
|
|
|
|
if (setxattr(path.c_str(), "security.capability", &cap_data, sizeof(cap_data),
|
|
0) < 0) {
|
|
return errno;
|
|
}
|
|
|
|
char buf[sizeof(cap_data)] = {};
|
|
const ssize_t size =
|
|
getxattr(path.c_str(), "security.capability", buf, sizeof(buf));
|
|
if (size != sizeof(cap_data)) {
|
|
return size < 0 ? errno : EIO;
|
|
}
|
|
return 0;
|
|
}
|
|
|
|
TEST(ChownTest, FchownBadF) {
|
|
ASSERT_THAT(fchown(-1, 0, 0), SyscallFailsWithErrno(EBADF));
|
|
}
|
|
|
|
TEST(ChownTest, FchownatBadF) {
|
|
ASSERT_THAT(fchownat(-1, "fff", 0, 0, 0), SyscallFailsWithErrno(EBADF));
|
|
}
|
|
|
|
TEST(ChownTest, FchownFileWithOpath) {
|
|
auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile());
|
|
FileDescriptor fd = ASSERT_NO_ERRNO_AND_VALUE(Open(file.path(), O_PATH));
|
|
|
|
ASSERT_THAT(fchown(fd.get(), geteuid(), getegid()),
|
|
SyscallFailsWithErrno(EBADF));
|
|
}
|
|
|
|
TEST(ChownTest, FchownDirWithOpath) {
|
|
const auto dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
|
|
const auto fd =
|
|
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path(), O_DIRECTORY | O_PATH));
|
|
|
|
ASSERT_THAT(fchown(fd.get(), geteuid(), getegid()),
|
|
SyscallFailsWithErrno(EBADF));
|
|
}
|
|
|
|
TEST(ChownTest, FchownPipeFileSucceeds) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_CHOWN)));
|
|
const auto uid = absl::GetFlag(FLAGS_scratch_uid1);
|
|
const auto gid = absl::GetFlag(FLAGS_scratch_gid);
|
|
int fds[2];
|
|
ASSERT_THAT(pipe2(fds, O_CLOEXEC), SyscallSucceeds());
|
|
for (const auto& fd : fds) {
|
|
ASSERT_THAT(fchown(fd, uid, gid), SyscallSucceeds());
|
|
struct stat s = {};
|
|
ASSERT_THAT(fstat(fd, &s), SyscallSucceeds());
|
|
EXPECT_EQ(s.st_uid, uid);
|
|
EXPECT_EQ(s.st_gid, gid);
|
|
}
|
|
}
|
|
|
|
TEST(ChownTest, FchownPipeFileFails) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SETUID)));
|
|
int fds[2];
|
|
ASSERT_THAT(pipe2(fds, O_CLOEXEC), SyscallSucceeds());
|
|
ScopedThread([&] {
|
|
// Drop privileges.
|
|
AutoCapability cap(CAP_CHOWN, false);
|
|
|
|
// Change EUID and EGID.
|
|
//
|
|
// See note about POSIX below.
|
|
EXPECT_THAT(
|
|
syscall(SYS_setresgid, -1, absl::GetFlag(FLAGS_scratch_gid), -1),
|
|
SyscallSucceeds());
|
|
EXPECT_THAT(
|
|
syscall(SYS_setresuid, -1, absl::GetFlag(FLAGS_scratch_uid1), -1),
|
|
SyscallSucceeds());
|
|
EXPECT_THAT(fchown(fds[1], geteuid(), getegid()),
|
|
SyscallFailsWithErrno(EPERM));
|
|
});
|
|
}
|
|
|
|
TEST(ChownTest, FchownatWithOpath) {
|
|
const auto dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
|
|
auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFileIn(dir.path()));
|
|
const auto dirfd =
|
|
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path(), O_DIRECTORY | O_PATH));
|
|
ASSERT_THAT(
|
|
fchownat(dirfd.get(), file.path().c_str(), geteuid(), getegid(), 0),
|
|
SyscallSucceeds());
|
|
}
|
|
|
|
TEST(ChownTest, FchownatWithOpathEmtpyPath) {
|
|
const auto dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
|
|
const auto dirfd =
|
|
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path(), O_DIRECTORY | O_PATH));
|
|
ASSERT_THAT(fchownat(dirfd.get(), "", geteuid(), getegid(), AT_EMPTY_PATH),
|
|
SyscallSucceeds());
|
|
}
|
|
|
|
TEST(ChownTest, FchownatEmptyPath) {
|
|
const auto dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
|
|
const auto fd =
|
|
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path(), O_DIRECTORY | O_RDONLY));
|
|
ASSERT_THAT(fchownat(fd.get(), "", 0, 0, 0), SyscallFailsWithErrno(ENOENT));
|
|
}
|
|
|
|
TEST(ChownTest, ChownRemovesSecurityCapability) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_CHOWN)));
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SETFCAP)));
|
|
|
|
const auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile());
|
|
const int set_cap_errno = SetSecurityCapability(file.path());
|
|
if (set_cap_errno == EOPNOTSUPP || set_cap_errno == EPERM) {
|
|
GTEST_SKIP() << "setting security.capability failed with errno "
|
|
<< set_cap_errno;
|
|
}
|
|
ASSERT_EQ(set_cap_errno, 0);
|
|
|
|
ASSERT_THAT(chown(file.path().c_str(), absl::GetFlag(FLAGS_scratch_uid1), -1),
|
|
SyscallSucceeds());
|
|
|
|
char buf[sizeof(VfsCapData)] = {};
|
|
EXPECT_THAT(
|
|
getxattr(file.path().c_str(), "security.capability", buf, sizeof(buf)),
|
|
SyscallFailsWithErrno(ENODATA));
|
|
}
|
|
|
|
TEST(ChownTest, SetGroupIdBitDeniedWithoutCapFsetIdOrRelevantGroup) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SETGID)));
|
|
const auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile());
|
|
AutoCapability cap(CAP_FSETID, false);
|
|
|
|
// Changing task creds in the main thread can mess with other tests.
|
|
ScopedThread([&] {
|
|
const mode_t chown_mode = 0755 | S_ISGID;
|
|
const gid_t oddGid = 12345;
|
|
struct stat stats;
|
|
mode_t stat_mode;
|
|
|
|
// 1) Without CAP_FSETID, and no relevant group, S_ISGID is not applied.
|
|
ASSERT_THAT(syscall(SYS_setresgid, oddGid, oddGid, oddGid),
|
|
SyscallSucceeds());
|
|
std::vector<gid_t> no_extra_groups;
|
|
ASSERT_THAT(
|
|
syscall(SYS_setgroups, no_extra_groups.size(), no_extra_groups.data()),
|
|
SyscallSucceeds());
|
|
|
|
ASSERT_THAT(chmod(file.path().c_str(), chown_mode), SyscallSucceeds());
|
|
ASSERT_THAT(stat(file.path().c_str(), &stats), SyscallSucceeds());
|
|
stat_mode = stats.st_mode & ~S_IFMT;
|
|
|
|
EXPECT_EQ(stat_mode & S_ISGID, 0); // S_ISGID is not applied.
|
|
EXPECT_EQ(stat_mode, chown_mode & ~S_ISGID); // but the rest is.
|
|
|
|
// 2) Without CAP_FSETID, but with a relevant group, S_ISGID is applied.
|
|
gid_t file_gid = stats.st_gid;
|
|
ASSERT_THAT(syscall(SYS_setresgid, file_gid, file_gid, file_gid),
|
|
SyscallSucceeds());
|
|
|
|
ASSERT_THAT(chmod(file.path().c_str(), chown_mode), SyscallSucceeds());
|
|
ASSERT_THAT(stat(file.path().c_str(), &stats), SyscallSucceeds());
|
|
stat_mode = stats.st_mode & ~S_IFMT;
|
|
|
|
EXPECT_EQ(stat_mode & S_ISGID, S_ISGID); // S_ISGID is applied.
|
|
EXPECT_EQ(stat_mode, chown_mode); // as is the rest.
|
|
|
|
// 3) With CAP_FSETID, but without a relevant group, S_ISGID is applied.
|
|
AutoCapability cap_inner(CAP_FSETID, true);
|
|
ASSERT_THAT(syscall(SYS_setresgid, oddGid, oddGid, oddGid),
|
|
SyscallSucceeds());
|
|
|
|
ASSERT_THAT(chmod(file.path().c_str(), chown_mode), SyscallSucceeds());
|
|
ASSERT_THAT(stat(file.path().c_str(), &stats), SyscallSucceeds());
|
|
stat_mode = stats.st_mode & ~S_IFMT;
|
|
|
|
EXPECT_EQ(stat_mode & S_ISGID, S_ISGID); // S_ISGID is applied.
|
|
EXPECT_EQ(stat_mode, chown_mode); // as is the rest.
|
|
});
|
|
}
|
|
|
|
using Chown =
|
|
std::function<PosixError(const std::string&, uid_t owner, gid_t group)>;
|
|
|
|
class ChownParamTest : public ::testing::TestWithParam<Chown> {};
|
|
|
|
TEST_P(ChownParamTest, ChownFileSucceeds) {
|
|
AutoCapability cap(CAP_CHOWN, false);
|
|
|
|
const auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile());
|
|
|
|
// At least *try* setting to a group other than the EGID.
|
|
gid_t gid;
|
|
EXPECT_THAT(gid = getegid(), SyscallSucceeds());
|
|
int num_groups;
|
|
EXPECT_THAT(num_groups = getgroups(0, nullptr), SyscallSucceeds());
|
|
if (num_groups > 0) {
|
|
std::vector<gid_t> list(num_groups);
|
|
EXPECT_THAT(getgroups(list.size(), list.data()), SyscallSucceeds());
|
|
// Scan the list of groups for a valid gid. Note that if a group is not
|
|
// defined in this local user namespace, then we will see 65534, and the
|
|
// group will not chown below as expected. So only change if we find a
|
|
// valid group in this list.
|
|
for (const gid_t other_gid : list) {
|
|
if (other_gid != 65534) {
|
|
gid = other_gid;
|
|
break;
|
|
}
|
|
}
|
|
}
|
|
|
|
EXPECT_NO_ERRNO(GetParam()(file.path(), geteuid(), gid));
|
|
|
|
struct stat s = {};
|
|
ASSERT_THAT(stat(file.path().c_str(), &s), SyscallSucceeds());
|
|
EXPECT_EQ(s.st_uid, geteuid());
|
|
EXPECT_EQ(s.st_gid, gid);
|
|
}
|
|
|
|
TEST_P(ChownParamTest, ChownFilePermissionDenied) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SETUID)));
|
|
|
|
const auto file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFileMode(0777));
|
|
EXPECT_THAT(chmod(GetAbsoluteTestTmpdir().c_str(), 0777), SyscallSucceeds());
|
|
|
|
// Drop privileges and change IDs only in child thread, or else this parent
|
|
// thread won't be able to open some log files after the test ends.
|
|
ScopedThread([&] {
|
|
// Drop privileges.
|
|
AutoCapability cap(CAP_CHOWN, false);
|
|
|
|
// Change EUID and EGID.
|
|
//
|
|
// See note about POSIX below.
|
|
EXPECT_THAT(
|
|
syscall(SYS_setresgid, -1, absl::GetFlag(FLAGS_scratch_gid), -1),
|
|
SyscallSucceeds());
|
|
EXPECT_THAT(
|
|
syscall(SYS_setresuid, -1, absl::GetFlag(FLAGS_scratch_uid1), -1),
|
|
SyscallSucceeds());
|
|
|
|
EXPECT_THAT(GetParam()(file.path(), geteuid(), getegid()),
|
|
PosixErrorIs(EPERM, ::testing::ContainsRegex("chown")));
|
|
});
|
|
}
|
|
|
|
TEST_P(ChownParamTest, ChownFileSucceedsAsRoot) {
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability((CAP_CHOWN))));
|
|
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability((CAP_SETUID))));
|
|
|
|
const std::string filename = NewTempAbsPath();
|
|
EXPECT_THAT(chmod(GetAbsoluteTestTmpdir().c_str(), 0777), SyscallSucceeds());
|
|
|
|
absl::Notification fileCreated, fileChowned;
|
|
// Change UID only in child thread, or else this parent thread won't be able
|
|
// to open some log files after the test ends.
|
|
ScopedThread t([&] {
|
|
// POSIX requires that all threads in a process share the same UIDs, so
|
|
// the NPTL setresuid wrappers use signals to make all threads execute the
|
|
// setresuid syscall. However, we want this thread to have its own set of
|
|
// credentials different from the parent process, so we use the raw
|
|
// syscall.
|
|
EXPECT_THAT(
|
|
syscall(SYS_setresuid, -1, absl::GetFlag(FLAGS_scratch_uid2), -1),
|
|
SyscallSucceeds());
|
|
|
|
// Create file and immediately close it.
|
|
FileDescriptor fd =
|
|
ASSERT_NO_ERRNO_AND_VALUE(Open(filename, O_CREAT | O_RDWR, 0644));
|
|
fd.reset(); // Close the fd.
|
|
|
|
fileCreated.Notify();
|
|
fileChowned.WaitForNotification();
|
|
|
|
EXPECT_THAT(open(filename.c_str(), O_RDWR), SyscallFailsWithErrno(EACCES));
|
|
FileDescriptor fd2 = ASSERT_NO_ERRNO_AND_VALUE(Open(filename, O_RDONLY));
|
|
});
|
|
|
|
fileCreated.WaitForNotification();
|
|
|
|
// Set file's owners to someone different.
|
|
EXPECT_NO_ERRNO(GetParam()(filename, absl::GetFlag(FLAGS_scratch_uid1),
|
|
absl::GetFlag(FLAGS_scratch_gid)));
|
|
|
|
struct stat s;
|
|
EXPECT_THAT(stat(filename.c_str(), &s), SyscallSucceeds());
|
|
EXPECT_EQ(s.st_uid, absl::GetFlag(FLAGS_scratch_uid1));
|
|
EXPECT_EQ(s.st_gid, absl::GetFlag(FLAGS_scratch_gid));
|
|
|
|
fileChowned.Notify();
|
|
}
|
|
|
|
PosixError errorFromReturn(const std::string& name, int ret) {
|
|
if (ret == -1) {
|
|
return PosixError(errno, absl::StrCat(name, " failed"));
|
|
}
|
|
return NoError();
|
|
}
|
|
|
|
INSTANTIATE_TEST_SUITE_P(
|
|
ChownKinds, ChownParamTest,
|
|
::testing::Values(
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
int rc = chown(path.c_str(), owner, group);
|
|
MaybeSave();
|
|
return errorFromReturn("chown", rc);
|
|
},
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
int rc = lchown(path.c_str(), owner, group);
|
|
MaybeSave();
|
|
return errorFromReturn("lchown", rc);
|
|
},
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
ASSIGN_OR_RETURN_ERRNO(auto fd, Open(path, O_RDWR));
|
|
int rc = fchown(fd.get(), owner, group);
|
|
MaybeSave();
|
|
return errorFromReturn("fchown", rc);
|
|
},
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
ASSIGN_OR_RETURN_ERRNO(auto fd, Open(path, O_RDWR));
|
|
int rc = fchownat(fd.get(), "", owner, group, AT_EMPTY_PATH);
|
|
MaybeSave();
|
|
return errorFromReturn("fchownat-fd", rc);
|
|
},
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
ASSIGN_OR_RETURN_ERRNO(auto dirfd, Open(std::string(Dirname(path)),
|
|
O_DIRECTORY | O_RDONLY));
|
|
int rc = fchownat(dirfd.get(), std::string(Basename(path)).c_str(),
|
|
owner, group, 0);
|
|
MaybeSave();
|
|
return errorFromReturn("fchownat-dirfd", rc);
|
|
},
|
|
[](const std::string& path, uid_t owner, gid_t group) -> PosixError {
|
|
ASSIGN_OR_RETURN_ERRNO(auto dirfd, Open(std::string(Dirname(path)),
|
|
O_DIRECTORY | O_PATH));
|
|
int rc = fchownat(dirfd.get(), std::string(Basename(path)).c_str(),
|
|
owner, group, 0);
|
|
MaybeSave();
|
|
return errorFromReturn("fchownat-opathdirfd", rc);
|
|
}));
|
|
|
|
} // namespace
|
|
|
|
} // namespace testing
|
|
} // namespace gvisor
|