Commit 4c443d1b authored by Stephen Smalley's avatar Stephen Smalley Committed by Linus Torvalds

[PATCH] selinux: fix avc_alloc_node() oom with no policy loaded

This patch should fix the avc_alloc_node() oom condition that Andrew
reported when no policy is loaded in SELinux.

Prior to this patch, when no policy was loaded, the SELinux "security
server" (policy engine) was only returning allowed decisions for the
requested permissions for each access check.  This caused the cache to
thrash when trying to use SELinux for real work with no policy loaded
(typically, the no policy loaded state is only for bootstrapping to the
point where we can load an initial policy).

This patch changes the SELinux security server to return the complete
allowed access vector at once, and then to reset the cache after the
initial policy load to flush the initial cache state created during
bootstrapping.
Signed-off-by: default avatarStephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: default avatarJames Morris <jmorris@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent b8107480
...@@ -476,8 +476,8 @@ int security_compute_av(u32 ssid, ...@@ -476,8 +476,8 @@ int security_compute_av(u32 ssid,
int rc = 0; int rc = 0;
if (!ss_initialized) { if (!ss_initialized) {
avd->allowed = requested; avd->allowed = 0xffffffff;
avd->decided = requested; avd->decided = 0xffffffff;
avd->auditallow = 0; avd->auditallow = 0;
avd->auditdeny = 0xffffffff; avd->auditdeny = 0xffffffff;
avd->seqno = latest_granting; avd->seqno = latest_granting;
...@@ -1196,9 +1196,11 @@ int security_load_policy(void *data, size_t len) ...@@ -1196,9 +1196,11 @@ int security_load_policy(void *data, size_t len)
} }
policydb_loaded_version = policydb.policyvers; policydb_loaded_version = policydb.policyvers;
ss_initialized = 1; ss_initialized = 1;
seqno = ++latest_granting;
LOAD_UNLOCK; LOAD_UNLOCK;
selinux_complete_init(); selinux_complete_init();
avc_ss_reset(seqno);
selnl_notify_policyload(seqno);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment