diff options
| author | Michal Zidek <mzidek@redhat.com> | 2014-11-06 16:35:11 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-11-06 19:19:35 +0100 |
| commit | 013c01bd491b535e1705dbb3dbd8424cffc66b7a (patch) | |
| tree | 595d9590cda6f3f8b86d027e00185f562b481dc6 /src/providers | |
| parent | 0d0e787555efc1e6e4eb39a924bb9861bf2921d5 (diff) | |
| download | sssd-013c01bd491b535e1705dbb3dbd8424cffc66b7a.tar.gz sssd-013c01bd491b535e1705dbb3dbd8424cffc66b7a.tar.xz sssd-013c01bd491b535e1705dbb3dbd8424cffc66b7a.zip | |
selinux_child: Do not ignore return values.
Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ipa/selinux_child.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/providers/ipa/selinux_child.c b/src/providers/ipa/selinux_child.c index a624cfd30..a38ffcb26 100644 --- a/src/providers/ipa/selinux_child.c +++ b/src/providers/ipa/selinux_child.c @@ -241,8 +241,16 @@ int main(int argc, const char *argv[]) DEBUG(SSSDBG_TRACE_FUNC, "performing selinux operations\n"); ret = set_seuser(ibuf->username, ibuf->seuser, ibuf->mls_range); + if (ret != EOK) { + DEBUG(SSSDBG_CRIT_FAILURE, "Cannot set SELinux login context.\n"); + goto fail; + } ret = prepare_response(main_ctx, ret, &resp); + if (ret != EOK) { + DEBUG(SSSDBG_CRIT_FAILURE, "Failed to prepare response buffer.\n"); + goto fail; + } errno = 0; |