SELinux: reset the security_ops before flushing the avc cache
authorEric Paris <eparis@redhat.com>
Mon, 21 Sep 2009 01:23:01 +0000 (21:23 -0400)
committerJames Morris <jmorris@namei.org>
Wed, 30 Sep 2009 09:17:06 +0000 (19:17 +1000)
This patch resets the security_ops to the secondary_ops before it flushes
the avc.  It's still possible that a task on another processor could have
already passed the security_ops dereference and be executing an selinux hook
function which would add a new avc entry.  That entry would still not be
freed.  This should however help to reduce the number of needless avcs the
kernel has when selinux is disabled at run time.  There is no wasted
memory if selinux is disabled on the command line or not compiled.

Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
security/selinux/hooks.c

index bb230d5d7085a9612f915edbb50124a0077db4be..a985d0bc59bb8a1a77dcfbb74ccfc729e80c1acb 100644 (file)
@@ -5830,12 +5830,12 @@ int selinux_disable(void)
        selinux_disabled = 1;
        selinux_enabled = 0;
 
-       /* Try to destroy the avc node cache */
-       avc_disable();
-
        /* Reset security_ops to the secondary module, dummy or capability. */
        security_ops = secondary_ops;
 
+       /* Try to destroy the avc node cache */
+       avc_disable();
+
        /* Unregister netfilter hooks. */
        selinux_nf_ip_exit();