fedora-selinux / selinux-policy

selinux-policy for Fedora is a large patch off the mainline
GNU General Public License v2.0
156 stars 157 forks source link

Dontaudit systemd-coredump sys_admin capability #2151

Closed zpytela closed 3 weeks ago

zpytela commented 4 weeks ago

Systemd uses the "trusted.delegate" and "user.delegate" extended attributes to indicate coredumps can be forwarded. The capability check is raised on the cg_is_delegated() library call when forward_coredump_to_container() and can_forward_coredump() is called, because the SYS_ADMIN capability is needed to access attributes in the "trusted" namespace, but preferred way actually is to use the attribute in the "user" namespace.

cgroup_delegate_xattr_apply() /* Indicate on the cgroup whether delegation is on, via an xattr. This is best-effort, as old kernels

Systemd changes with 255

packit-as-a-service[bot] commented 4 weeks ago

Cockpit tests failed for commit da3fa03541abcad0268ed83ab89690c9e8e2212a. @martinpitt, @jelly, @mvollmer please check.

martinpitt commented 4 weeks ago

@zpytela something in rawhide broke ABRT and changed tuned. These are high on our list to investigate. Unfortunately there are a lot of other OS regressions on that list too, so it'll take a bit.. So please ignore here, the test looks fine.

packit-as-a-service[bot] commented 4 weeks ago

Cockpit tests failed for commit 218a4ef9b193d7ef422aa896968cbe45af903b64. @martinpitt, @jelly, @mvollmer please check.

martinpitt commented 3 weeks ago

@zpytela We reported the ABRT regression and naughtied it, so tests should go green again. I retried this one and all other recent ones.

martinpitt commented 3 weeks ago

Green again.

zpytela commented 3 weeks ago

Thanks Martin for your detailed updates.