summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Zeleny <jzeleny@redhat.com>2012-05-15 10:49:14 -0400
committerStephen Gallagher <sgallagh@redhat.com>2012-05-22 13:23:08 -0400
commitd4080fe3e5e25562d2dee8b4ae72064d9d72c0bc (patch)
treed6cc6645962dc33848aae4be7905f5f77254740e
parentb49c47d573208f22a40214a728112949956acb57 (diff)
downloadsssd-d4080fe3e5e25562d2dee8b4ae72064d9d72c0bc.tar.gz
sssd-d4080fe3e5e25562d2dee8b4ae72064d9d72c0bc.tar.xz
sssd-d4080fe3e5e25562d2dee8b4ae72064d9d72c0bc.zip
Fixed issue in SELinux user maps
There was an issue when IPA provider didn't set PAM_SUCCESS when successfully finished loading SELinux user maps. This lead to the map not being read in the responder.
-rw-r--r--src/providers/ipa/ipa_session.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/providers/ipa/ipa_session.c b/src/providers/ipa/ipa_session.c
index e23b0120e..238acdde7 100644
--- a/src/providers/ipa/ipa_session.c
+++ b/src/providers/ipa/ipa_session.c
@@ -104,6 +104,7 @@ static void ipa_session_handler_done(struct tevent_req *req)
struct sysdb_attrs **maps;
bool in_transaction = false;
char *default_user;
+ struct pam_data *pd = talloc_get_type(breq->req_data, struct pam_data);
char *map_order;
ret = ipa_get_selinux_recv(req, breq, &map_count, &maps,
@@ -140,6 +141,7 @@ static void ipa_session_handler_done(struct tevent_req *req)
in_transaction = false;
+ pd->pam_status = PAM_SUCCESS;
breq->fn(breq, DP_ERR_OK, EOK, "Success");
return;