Files
8b8ee76e29 Add missing Linux capability checks for SO_BINDTODEVICE, mknod, sched_setaffinity, and setpriority
## Summary

This patch adds capability and permission checks that the Linux kernel enforces but gVisor currently omits. Each fix was verified against native Linux behavior using `bazel test` on both native and `runsc_ptrace` platforms.

## Changes

### 1. `SO_BINDTODEVICE`: Add `CAP_NET_RAW` check
**File:** `pkg/sentry/socket/netstack/netstack.go`
**Linux reference:** `net/core/sock.c:sock_setsockopt()` checks `ns_capable(sock_net(sk)->user_ns, CAP_NET_RAW)`
**Evidence this is unintended:** gVisor's own test suite asserts `"CAP_NET_RAW is required to use SO_BINDTODEVICE"` (`test/syscalls/linux/socket_bind_to_device.cc:52`), and `SO_RCVBUFFORCE` in the same file already correctly checks `CAP_NET_ADMIN`.

### 2. `mknod(S_IFBLK/S_IFCHR)`: Add `CAP_MKNOD` check
**File:** `pkg/sentry/syscalls/linux/sys_file.go`
**Linux reference:** `fs/namei.c:vfs_mknod()` checks `capable(CAP_MKNOD)` for block/char device creation
**Evidence this is unintended:** `CAP_MKNOD` is defined (`pkg/abi/linux/capability.go:56`), parsed from OCI specs (`runsc/specutils/specutils.go:491`), and has strace formatting — but is never checked anywhere. Zero `HasCapability` calls for it exist in the codebase.

### 3. `sched_setaffinity`: Add UID match / `CAP_SYS_NICE` check
**File:** `pkg/sentry/syscalls/linux/sys_thread.go`
**Linux reference:** `kernel/sched/core.c:check_same_owner()` requires EUID match or `CAP_SYS_NICE`
**Impact:** Without this check, any unprivileged process could modify another process's CPU affinity mask.

### 4. `setpriority`: Add UID match / `CAP_SYS_NICE` check
**File:** `pkg/sentry/syscalls/linux/sys_thread.go`
**Linux reference:** `kernel/sys.c:set_one_prio()` requires UID match or `CAP_SYS_NICE`
**Impact:** Without this check, any unprivileged process could change another process's scheduling priority.

## Testing

Tests added in `test/syscalls/linux/capability_checks.cc`, verified on both native Linux and gVisor:

```
bazel test //test/syscalls:capability_checks_test_native        → 6/6 passed
bazel test //test/syscalls:capability_checks_test_runsc_ptrace  → 4 passed, 2 skipped
```

The 2 skipped tests are the mknod positive cases (creating device nodes with `CAP_MKNOD`), which are skipped on gVisor because the sandbox does not permit device node creation regardless of capabilities.

| Test | What it verifies |
|------|-----------------|
| `SoBindToDeviceCapTest.RequiresCapNetRaw` | `EPERM` without `CAP_NET_RAW` |
| `MknodCapTest.CharDevRequiresCapMknod` | `EPERM` for `S_IFCHR` without `CAP_MKNOD` (native only) |
| `MknodCapTest.BlockDevRequiresCapMknod` | `EPERM` for `S_IFBLK` without `CAP_MKNOD` (native only) |
| `MknodCapTest.FifoDoesNotRequireCapMknod` | `S_IFIFO` succeeds without `CAP_MKNOD` |
| `SchedSetaffinityCapTest.OtherUidRequiresCapSysNice` | `EPERM` without UID match or `CAP_SYS_NICE` |
| `SetpriorityCapTest.OtherUidRequiresCapSysNice` | `EPERM` without UID match or `CAP_SYS_NICE` |

COPYBARA_INTEGRATE_REVIEW=https://github.com/google/gvisor/pull/12872 from petrmarinec:fix/missing-capability-checks 70b4e0fa308ba182e7916112e90aed40cdea75f3
PiperOrigin-RevId: 903343461

Co-authored-by: Hanzo Dev <[email protected]>
2026-04-21 11:43:53 -07:00

182 lines
6.4 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 <sys/socket.h>
#include <sys/stat.h>
#include <sys/sysmacros.h>
#include <sys/types.h>
#include <sys/un.h>
#include <unistd.h>
#include "gtest/gtest.h"
#include "test/util/file_descriptor.h"
#include "test/util/linux_capability_util.h"
#include "test/util/mount_util.h"
#include "test/util/temp_path.h"
#include "test/util/test_util.h"
namespace gvisor {
namespace testing {
namespace {
TEST(MknodTest, RegularFile) {
const std::string node0 = NewTempAbsPath();
EXPECT_THAT(mknod(node0.c_str(), S_IFREG, 0), SyscallSucceeds());
const std::string node1 = NewTempAbsPath();
EXPECT_THAT(mknod(node1.c_str(), 0, 0), SyscallSucceeds());
}
TEST(MknodTest, RegularFilePermissions) {
const std::string node = NewTempAbsPath();
mode_t newUmask = 0077;
umask(newUmask);
// Attempt to open file with mode 0777. Not specifying file type should create
// a regular file.
mode_t perms = S_IRWXU | S_IRWXG | S_IRWXO;
EXPECT_THAT(mknod(node.c_str(), perms, 0), SyscallSucceeds());
// In the absence of a default ACL, the permissions of the created node are
// (mode & ~umask). -- mknod(2)
mode_t wantPerms = perms & ~newUmask;
struct stat st;
ASSERT_THAT(stat(node.c_str(), &st), SyscallSucceeds());
ASSERT_EQ(st.st_mode & 0777, wantPerms);
// "Zero file type is equivalent to type S_IFREG." - mknod(2)
ASSERT_EQ(st.st_mode & S_IFMT, S_IFREG);
}
TEST(MknodTest, MknodAtFIFO) {
const TempPath dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
const std::string fifo_relpath = NewTempRelPath();
const std::string fifo = JoinPath(dir.path(), fifo_relpath);
const FileDescriptor dirfd =
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path().c_str(), O_RDONLY));
ASSERT_THAT(mknodat(dirfd.get(), fifo_relpath.c_str(), S_IFIFO | S_IRUSR, 0),
SyscallSucceeds());
struct stat st;
ASSERT_THAT(stat(fifo.c_str(), &st), SyscallSucceeds());
EXPECT_TRUE(S_ISFIFO(st.st_mode));
}
#ifndef __Fuchsia__ // Fuchsia doesn't support caps.
// Test that mknod(S_IFCHR) requires CAP_MKNOD.
TEST(MknodTest, CharacterDeviceRequiresCapMknod) {
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_MKNOD)));
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SYS_ADMIN)));
const TempPath dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
const auto mount =
ASSERT_NO_ERRNO_AND_VALUE(Mount("", dir.path(), "tmpfs", 0, "", 0));
const std::string path = dir.path() + "/device";
// With CAP_MKNOD: should succeed.
ASSERT_THAT(mknod(path.c_str(), S_IFCHR | 0666, makedev(1, 3)),
SyscallSucceeds());
ASSERT_THAT(unlink(path.c_str()), SyscallSucceeds());
// Drop CAP_MKNOD: should fail with EPERM.
AutoCapability cap(CAP_MKNOD, false);
EXPECT_THAT(mknod(path.c_str(), S_IFCHR | 0666, makedev(1, 3)),
SyscallFailsWithErrno(EPERM));
}
#endif // __Fuchsia__
#ifndef __Fuchsia__ // Fuchsia doesn't support caps.
// Test that mknod(S_IFBLK) requires CAP_MKNOD.
TEST(MknodTest, BlockDeviceRequiresCapMknod) {
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_MKNOD)));
SKIP_IF(!ASSERT_NO_ERRNO_AND_VALUE(HaveCapability(CAP_SYS_ADMIN)));
const TempPath dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
const auto mount =
ASSERT_NO_ERRNO_AND_VALUE(Mount("", dir.path(), "tmpfs", 0, "", 0));
const std::string path = dir.path() + "/device";
// With CAP_MKNOD: should succeed.
ASSERT_THAT(mknod(path.c_str(), S_IFBLK | 0666, makedev(7, 0)),
SyscallSucceeds());
ASSERT_THAT(unlink(path.c_str()), SyscallSucceeds());
// Drop CAP_MKNOD: should fail with EPERM.
AutoCapability cap(CAP_MKNOD, false);
EXPECT_THAT(mknod(path.c_str(), S_IFBLK | 0666, makedev(7, 0)),
SyscallFailsWithErrno(EPERM));
}
#endif // __Fuchsia__
#ifndef __Fuchsia__ // Fuchsia doesn't support caps.
// Test that mknod(S_IFIFO) does NOT require CAP_MKNOD.
TEST(MknodTest, FifoDoesNotRequireCapMknod) {
const std::string path = NewTempAbsPath();
AutoCapability cap(CAP_MKNOD, false);
EXPECT_THAT(mknod(path.c_str(), S_IFIFO | 0666, 0), SyscallSucceeds());
EXPECT_THAT(unlink(path.c_str()), SyscallSucceeds());
}
#endif // __Fuchsia__
TEST(MknodTest, MknodOnExistingPathFails) {
const TempPath file = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateFile());
const TempPath slink = ASSERT_NO_ERRNO_AND_VALUE(
TempPath::CreateSymlinkTo(GetAbsoluteTestTmpdir(), file.path()));
EXPECT_THAT(mknod(file.path().c_str(), S_IFREG, 0),
SyscallFailsWithErrno(EEXIST));
EXPECT_THAT(mknod(file.path().c_str(), S_IFIFO, 0),
SyscallFailsWithErrno(EEXIST));
EXPECT_THAT(mknod(slink.path().c_str(), S_IFREG, 0),
SyscallFailsWithErrno(EEXIST));
EXPECT_THAT(mknod(slink.path().c_str(), S_IFIFO, 0),
SyscallFailsWithErrno(EEXIST));
}
TEST(MknodTest, Socket) {
ASSERT_THAT(chdir(GetAbsoluteTestTmpdir().c_str()), SyscallSucceeds());
auto filename = NewTempRelPath();
ASSERT_THAT(mknod(filename.c_str(), S_IFSOCK | S_IRUSR | S_IWUSR, 0),
SyscallSucceeds());
int sk;
ASSERT_THAT(sk = socket(AF_UNIX, SOCK_SEQPACKET, 0), SyscallSucceeds());
FileDescriptor fd(sk);
struct sockaddr_un addr = {.sun_family = AF_UNIX};
absl::SNPrintF(addr.sun_path, sizeof(addr.sun_path), "%s", filename.c_str());
ASSERT_THAT(connect(sk, (struct sockaddr *)&addr, sizeof(addr)),
SyscallFailsWithErrno(ECONNREFUSED));
ASSERT_THAT(unlink(filename.c_str()), SyscallSucceeds());
}
TEST(MknodTest, MknodAtEmptyPath) {
auto dir = ASSERT_NO_ERRNO_AND_VALUE(TempPath::CreateDir());
auto fd =
ASSERT_NO_ERRNO_AND_VALUE(Open(dir.path(), O_RDONLY | O_DIRECTORY, 0666));
EXPECT_THAT(mknodat(fd.get(), "", S_IFREG | 0777, 0),
SyscallFailsWithErrno(ENOENT));
}
} // namespace
} // namespace testing
} // namespace gvisor