diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2010-11-05 08:54:50 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-11-05 14:50:55 -0400 |
| commit | 6e3ab18c27e724db5fb7953daa11361227be22af (patch) | |
| tree | 84e211f05e8b0d86979fde60ff0fea52cb600e64 | |
| parent | 91565f10d6567abd250a6d0ad4f278d33152a38c (diff) | |
| download | sssd2-6e3ab18c27e724db5fb7953daa11361227be22af.tar.gz sssd2-6e3ab18c27e724db5fb7953daa11361227be22af.tar.xz sssd2-6e3ab18c27e724db5fb7953daa11361227be22af.zip | |
Properly check the return value from semanage_commit
semanage_commit() returns -1 on error, and can return a positive
value on success.
https://bugzilla.redhat.com/show_bug.cgi?id=649037
| -rw-r--r-- | src/tools/selinux.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/tools/selinux.c b/src/tools/selinux.c index 27cc5680..647fc17a 100644 --- a/src/tools/selinux.c +++ b/src/tools/selinux.c @@ -324,7 +324,7 @@ int set_seuser(const char *login_name, const char *seuser_name) } ret = semanage_commit(handle); - if (ret != 0) { + if (ret < 0) { DEBUG(1, ("Cannot commit SELinux transaction\n")); ret = EIO; goto done; @@ -394,7 +394,7 @@ int del_seuser(const char *login_name) } ret = semanage_commit(handle); - if (ret != 0) { + if (ret < 0) { DEBUG(1, ("Cannot commit SELinux transaction\n")); ret = EIO; goto done; |