mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2025-02-06 10:32:10 +00:00
7b98502972
Set the SECBIT_NO_SETUID_FIXUP securebit so that we don't lose
capabilities when changing the thread uid/gid. This allows us to
simplify the create and mkdir functions so that all the checks we
currently carry out are only done once by the host kernel.
To ensure that the setuid and setgid bits still get dropped when a file
is modified by a process that doesn't hold CAP_FSETID, check for
WRITE_KILL_PRIV in the write flags and temporarily drop CAP_FSETID when
it is set.
BUG=none
TEST=Check that default posix acls, setgid bits, and file/directory
creation via membership of a supplementary group all work as
expected.
Change-Id: I420484e357a970e997cb3e968a433278e82d8ad4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2684067
Auto-Submit: Chirantan Ekbote <chirantan@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
Commit-Queue: Chirantan Ekbote <chirantan@chromium.org>
Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
53 lines
974 B
Text
53 lines
974 B
Text
# Copyright 2019 The Chromium OS 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 /usr/share/policy/crosvm/common_device.policy
|
|
|
|
copy_file_range: 1
|
|
fallocate: 1
|
|
fchdir: 1
|
|
fchmod: 1
|
|
fchmodat: 1
|
|
fchown: 1
|
|
fchownat: 1
|
|
fdatasync: 1
|
|
fgetxattr: 1
|
|
getxattr: 1
|
|
fsetxattr: 1
|
|
setxattr: 1
|
|
flistxattr: 1
|
|
listxattr: 1
|
|
fremovexattr: 1
|
|
removexattr: 1
|
|
fsync: 1
|
|
newfstatat: 1
|
|
fstatfs: 1
|
|
ftruncate: 1
|
|
getdents64: 1
|
|
getegid: 1
|
|
geteuid: 1
|
|
getrandom: 1
|
|
ioctl: arg1 == FS_IOC_FSGETXATTR || \
|
|
arg1 == FS_IOC_FSSETXATTR || \
|
|
arg1 == FS_IOC_GETFLAGS || \
|
|
arg1 == FS_IOC_SETFLAGS || \
|
|
arg1 == FS_IOC_GET_ENCRYPTION_POLICY_EX
|
|
linkat: 1
|
|
lseek: 1
|
|
mkdirat: 1
|
|
mknodat: 1
|
|
openat: 1
|
|
preadv: 1
|
|
pwritev: 1
|
|
readlinkat: 1
|
|
renameat2: 1
|
|
setresgid: 1
|
|
setresuid: 1
|
|
symlinkat: 1
|
|
umask: 1
|
|
unlinkat: 1
|
|
utimensat: 1
|
|
prctl: arg0 == PR_SET_SECUREBITS || arg0 == PR_GET_SECUREBITS
|
|
capget: 1
|
|
capset: 1
|