selinux: fix missing dput() before selinuxfs unmount
authorStephen Smalley <sds@tycho.nsa.gov>
Mon, 9 Apr 2018 18:36:05 +0000 (14:36 -0400)
committerLinus Torvalds <torvalds@linux-foundation.org>
Mon, 9 Apr 2018 18:52:56 +0000 (11:52 -0700)
commitfd40ffc72e2f74c7db61e400903e7d50a88bc0b0
treeb9c6028f1a132ec509c805eb63d8f16afc776e82
parentd8312a3f61024352f1c7cb967571fd53631b0d6c
selinux: fix missing dput() before selinuxfs unmount

Commit 0619f0f5e36f ("selinux: wrap selinuxfs state") triggers a BUG
when SELinux is runtime-disabled (i.e. systemd or equivalent disables
SELinux before initial policy load via /sys/fs/selinux/disable based on
/etc/selinux/config SELINUX=disabled).

This does not manifest if SELinux is disabled via kernel command line
argument or if SELinux is enabled (permissive or enforcing).

Before:
  SELinux:  Disabled at runtime.
  BUG: Dentry 000000006d77e5c7{i=17,n=null}  still in use (1) [unmount of selinuxfs selinuxfs]

After:
  SELinux:  Disabled at runtime.

Fixes: 0619f0f5e36f ("selinux: wrap selinuxfs state")
Reported-by: Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
security/selinux/selinuxfs.c