diff options
author | Stephen Gallagher <sgallagh@redhat.com> | 2012-02-27 12:00:52 -0500 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-02-27 12:19:32 -0500 |
commit | 89325d4a1c92616be2a388e9dff7c5122d782786 (patch) | |
tree | 613a1132013963af51d23bbd2ca413fa617b8e82 /src/providers | |
parent | 71ba8086905fdfdc0550e7ef8ecf5e4ae83681a7 (diff) | |
download | sssd-89325d4a1c92616be2a388e9dff7c5122d782786.tar.gz sssd-89325d4a1c92616be2a388e9dff7c5122d782786.tar.bz2 sssd-89325d4a1c92616be2a388e9dff7c5122d782786.zip |
PAM: Don't send PAM_SYSTEM_INFO message if module unset
We now have a session module that is only available for the IPA
provider. We should not be logging noisily that other providers
do not have the session provider configured.
https://fedorahosted.org/sssd/ticket/1211
Diffstat (limited to 'src/providers')
-rw-r--r-- | src/providers/data_provider_be.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/providers/data_provider_be.c b/src/providers/data_provider_be.c index 7e27ceac..2b76eb3c 100644 --- a/src/providers/data_provider_be.c +++ b/src/providers/data_provider_be.c @@ -595,16 +595,12 @@ static int be_pam_handler(DBusMessage *message, struct sbus_connection *conn) goto done; } - /* return an error if corresponding backend target is not configured */ + /* return PAM_MODULE_UNKNOWN if corresponding backend target is not + * configured + */ if (!becli->bectx->bet_info[target].bet_ops) { DEBUG(7, ("Undefined backend target.\n")); pd->pam_status = PAM_MODULE_UNKNOWN; - ret = pam_add_response(pd, SSS_PAM_SYSTEM_INFO, - sizeof(MSG_TARGET_NO_CONFIGURED), - (const uint8_t *) MSG_TARGET_NO_CONFIGURED); - if (ret != EOK) { - DEBUG(1, ("pam_add_response failed.\n")); - } goto done; } |