src/init.c | 20 +++----------------- src/load_policy.c | 15 ++++++++++----- 2 files changed, 13 insertions(+), 22 deletions(-) diff --git a/src/init.c b/src/init.c index 3db4de0..3c687a2 100644 --- a/src/init.c +++ b/src/init.c @@ -11,7 +11,6 @@ #include #include #include -#include #include "dso.h" #include "policy.h" @@ -57,20 +56,15 @@ static int verify_selinuxmnt(const char *mnt) int selinuxfs_exists(void) { - int exists = 0, mnt_rc = 0; + int exists = 0; FILE *fp = NULL; char *buf = NULL; size_t len; ssize_t num; - mnt_rc = mount("proc", "/proc", "proc", 0, 0); - fp = fopen("/proc/filesystems", "r"); - if (!fp) { - exists = 1; /* Fail as if it exists */ - goto out; - } - + if (!fp) + return 1; /* Fail as if it exists */ __fsetlocking(fp, FSETLOCKING_BYCALLER); num = getline(&buf, &len, fp); @@ -84,14 +78,6 @@ int selinuxfs_exists(void) free(buf); fclose(fp); - -out: -#ifndef MNT_DETACH -#define MNT_DETACH 2 -#endif - if (mnt_rc == 0) - umount2("/proc", MNT_DETACH); - return exists; } hidden_def(selinuxfs_exists) diff --git a/src/load_policy.c b/src/load_policy.c index 21ee58b..4f39fc7 100644 --- a/src/load_policy.c +++ b/src/load_policy.c @@ -17,6 +17,10 @@ #include "policy.h" #include +#ifndef MNT_DETACH +#define MNT_DETACH 2 +#endif + int security_load_policy(void *data, size_t len) { char path[PATH_MAX]; @@ -348,11 +352,6 @@ int selinux_init_load_policy(int *enforce) fclose(cfg); free(buf); } -#ifndef MNT_DETACH -#define MNT_DETACH 2 -#endif - if (rc == 0) - umount2("/proc", MNT_DETACH); /* * Determine the final desired mode. @@ -400,11 +399,17 @@ int selinux_init_load_policy(int *enforce) /* Only emit this error if selinux was not disabled */ fprintf(stderr, "Mount failed for selinuxfs on %s: %s\n", SELINUXMNT, strerror(errno)); } + + if (rc == 0) + umount2("/proc", MNT_DETACH); goto noload; } set_selinuxmnt(mntpoint); + if (rc == 0) + umount2("/proc", MNT_DETACH); + /* * Note: The following code depends on having selinuxfs * already mounted and selinuxmnt set above.