chiark / gitweb /
Revert "selinux: split up mac_selinux_have() from mac_selinux_use()"
authorGary Tierney <gary.tierney@gmx.com>
Tue, 2 May 2017 16:42:19 +0000 (17:42 +0100)
committerSven Eden <yamakuzure@gmx.net>
Tue, 25 Jul 2017 07:46:52 +0000 (09:46 +0200)
commite666102220558dbb43210176129b3ea8245c578c
treefedc9056d8431171f29b5ec35295f389b0d30234
parentd5aacbb6077b4e45fd36fbff6843595aa64d2288
Revert "selinux: split up mac_selinux_have() from mac_selinux_use()"

This reverts commit 6355e75610a8d47fc3ba5ab8bd442172a2cfe574.

The previously mentioned commit inadvertently broke a lot of SELinux related
functionality for both unprivileged users and elogind instances running as
MANAGER_USER.  In particular, setting the correct SELinux context after a User=
directive is used would fail to work since we attempt to set the security
context after changing UID.  Additionally, it causes activated socket units to
be mislabeled for elogind --user processes since setsockcreatecon() would never
be called.

Reverting this fixes the issues with labeling outlined above, and reinstates
SELinux access checks on unprivileged user services.
src/basic/selinux-util.c
src/basic/selinux-util.h
src/libelogind/sd-bus/bus-socket.c
src/test/test-selinux.c