Commit 253a8b1d authored by James Morris's avatar James Morris Committed by Linus Torvalds

[PATCH] selinuxfs cleanups: sel_make_bools

Remove the call to sel_make_bools() from sel_fill_super(), as policy needs to
be loaded before the boolean files can be created.  Policy will never be
loaded during sel_fill_super() as selinuxfs is kernel mounted during init and
the only means to load policy is via selinuxfs.

Also, the call to d_genocide() on the error path of sel_make_bools() is
incorrect and replaced with sel_remove_bools().
Signed-off-by: default avatarJames Morris <jmorris@namei.org>
Acked-by: default avatarStephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 161ce45a
...@@ -970,7 +970,7 @@ out: ...@@ -970,7 +970,7 @@ out:
return ret; return ret;
err: err:
kfree(values); kfree(values);
d_genocide(dir); sel_remove_bools(dir);
ret = -ENOMEM; ret = -ENOMEM;
goto out; goto out;
} }
...@@ -1226,9 +1226,6 @@ static int sel_fill_super(struct super_block * sb, void * data, int silent) ...@@ -1226,9 +1226,6 @@ static int sel_fill_super(struct super_block * sb, void * data, int silent)
goto err; goto err;
bool_dir = dentry; bool_dir = dentry;
ret = sel_make_bools();
if (ret)
goto err;
dentry = d_alloc_name(sb->s_root, NULL_FILE_NAME); dentry = d_alloc_name(sb->s_root, NULL_FILE_NAME);
if (!dentry) { if (!dentry) {
......
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