From: pebenito@ieee.org (Chris PeBenito) Date: Sun, 10 Dec 2017 14:55:55 -0500 Subject: [refpolicy] [PATCH 1/1] label systemd-shutdown so shutdown works In-Reply-To: References: Message-ID: <77aec8b8-f1c0-ec8c-2127-bbb29876833d@ieee.org> To: refpolicy@oss.tresys.com List-Id: refpolicy.oss.tresys.com On 12/08/2017 05:30 PM, David Sugar via refpolicy wrote: > I am seeing (on RHEL 7.4 w/systemd) that halting the system doesn't work. It took me a long time (and a lot of help from Steve L.) to figure out what was going on. It turns out in refpolicy the default label for /usr/lib/systemd/systemd-shutdown is bin_t. But when systemd tried to execve systemd-shutdown it fails because init_t isn't allowed file entrypoint for bin_t. When I labeled systemd-shutdown as init_exec_t shutting down the system works. > > I was seeing the following log (from systemd) when I enabled systemd debug logging (which was very useful). > > [ 59.745037] systemd[1]: Starting Final Step. > [ 59.746112] systemd[1]: Starting Power-Off... > [ 59.776320] systemd[1]: Shutting down. > [ 59.783559] systemd[1]: Failed to execute shutdown binary, freezing: Operation not permitted > > At this point everything locks up instead of actually halting the system. > > This is a patch to change the label for systemd-shutdown which solves the problem. I'm happy to go through and make a distinct type of systemd-shutdown if someone doesn't think it is a good idea to share the type with systemd. But based on what is going on, this might be reasonable. > > Signed-off-by: Dave Sugar > --- > policy/modules/system/init.fc | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/policy/modules/system/init.fc b/policy/modules/system/init.fc > index 34bb6a7c..548a8631 100644 > --- a/policy/modules/system/init.fc > +++ b/policy/modules/system/init.fc > @@ -29,6 +29,7 @@ ifdef(`distro_gentoo',` > /usr/bin/upstart -- gen_context(system_u:object_r:init_exec_t,s0) > > /usr/lib/systemd/systemd -- gen_context(system_u:object_r:init_exec_t,s0) > +/usr/lib/systemd/systemd-shutdown -- gen_context(system_u:object_r:init_exec_t,s0) > /usr/lib/systemd/system-preset(/.*)? gen_context(system_u:object_r:systemd_unit_t,s0) > /usr/lib/systemd/user-preset(/.*)? gen_context(system_u:object_r:systemd_unit_t,s0) > /usr/lib/systemd/ntp-units\.d -d gen_context(system_u:object_r:systemd_unit_t,s0) Merged. -- Chris PeBenito