diff options
| author | Karel Zak | 2008-07-01 14:24:58 +0200 |
|---|---|---|
| committer | Karel Zak | 2008-07-01 14:32:53 +0200 |
| commit | 4ba66edf82e03109bd9110682179254d823c58a3 (patch) | |
| tree | b0a86c16fc0ac848949f603fcc75e96d994dafca /login-utils/chsh.c | |
| parent | mount: warn on "file_t" selinux context (diff) | |
| download | kernel-qcow2-util-linux-4ba66edf82e03109bd9110682179254d823c58a3.tar.gz kernel-qcow2-util-linux-4ba66edf82e03109bd9110682179254d823c58a3.tar.xz kernel-qcow2-util-linux-4ba66edf82e03109bd9110682179254d823c58a3.zip | |
selinux: is_selinux_enabled() returns 0, 1 and -1
Unfortunately, the current libselinux implementation of
is_selinux_enabled() returns -1 on error. This behavior is
undocumented.
The proper solution is to use "if (is_selinux_enabled() > 0)".
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'login-utils/chsh.c')
| -rw-r--r-- | login-utils/chsh.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/login-utils/chsh.c b/login-utils/chsh.c index d4cc52670..08b1e0d28 100644 --- a/login-utils/chsh.c +++ b/login-utils/chsh.c @@ -134,7 +134,7 @@ main (int argc, char *argv[]) { } #ifdef HAVE_LIBSELINUX - if (is_selinux_enabled()) { + if (is_selinux_enabled() > 0) { if(uid == 0) { if (checkAccess(pw->pw_name,PASSWD__CHSH)!=0) { security_context_t user_context; |