path: root/security/selinux/selinuxfs.c
diff options
authorStephen Smalley <>2018-04-09 14:36:05 -0400
committerLinus Torvalds <>2018-04-09 11:52:56 -0700
commitfd40ffc72e2f74c7db61e400903e7d50a88bc0b0 (patch)
treeb9c6028f1a132ec509c805eb63d8f16afc776e82 /security/selinux/selinuxfs.c
parentd8312a3f61024352f1c7cb967571fd53631b0d6c (diff)
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 <> Reported-by: Dmitry Vyukov <> Signed-off-by: Stephen Smalley <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'security/selinux/selinuxfs.c')
1 files changed, 1 insertions, 0 deletions
diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c
index 4be683eeba01..245160373dab 100644
--- a/security/selinux/selinuxfs.c
+++ b/security/selinux/selinuxfs.c
@@ -2061,6 +2061,7 @@ __initcall(init_sel_fs);
void exit_sel_fs(void)
sysfs_remove_mount_point(fs_kobj, "selinux");
+ dput(selinux_null.dentry);