[SSSD] [sssd PR#457][comment] ipa: Removal of umask(0) in selinux_child

2018-01-15 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/457 Title: #457: ipa: Removal of umask(0) in selinux_child bachradsusi commented: """ The problem with the test case is that it uses old path. `/etc/selinux/targeted/modules/active` changed to `/etc/selinux/targeted/active/modules/` ``

[SSSD] [sssd PR#457][comment] ipa: Removal of umask(0) in selinux_child

2018-01-15 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/457 Title: #457: ipa: Removal of umask(0) in selinux_child bachradsusi commented: """ I need to do more tests but it looks like the issue reported in the https://bugzilla.redhat.com/show_bug.cgi?id=1186422 is not fixed yet. The tes

[SSSD] [sssd PR#457][comment] ipa: Removal of umask(0) in selinux_child

2018-01-15 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/457 Title: #457: ipa: Removal of umask(0) in selinux_child bachradsusi commented: """ I need to do more tests but it looks like the issue reported in the https://bugzilla.redhat.com/show_bug.cgi?id=1186422 was actually fixed. The te

[SSSD] [sssd PR#457][comment] ipa: Removal of umask(0) in selinux_child

2017-11-23 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/457 Title: #457: ipa: Removal of umask(0) in selinux_child bachradsusi commented: """ I guess it's related to the change of SElinux module store location in /var/lib/selinux which happened in Release 2015-02-02 aka libsemanage-2.4

[SSSD] [sssd PR#342][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-09-05 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/342 Title: #342: SELINUX: Use getseuserbyname to get IPA seuser bachradsusi commented: """ You are right. `db_seuser` and `db_mls_range` need to be free()'d """ See the full comment at https://github.com/SSSD/s

[SSSD] [sssd PR#342][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-09-05 Thread bachradsusi
URL: https://github.com/SSSD/sssd/pull/342 Title: #342: SELINUX: Use getseuserbyname to get IPA seuser bachradsusi commented: """ When you need to manage - add/remove - SELinux login mappings you need to use libsemanage as you do now. There's no such interface in lib