summaryrefslogtreecommitdiff
path: root/src/providers
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2012-02-27 12:00:52 -0500
committerStephen Gallagher <sgallagh@redhat.com>2012-02-27 12:19:32 -0500
commit89325d4a1c92616be2a388e9dff7c5122d782786 (patch)
tree613a1132013963af51d23bbd2ca413fa617b8e82 /src/providers
parent71ba8086905fdfdc0550e7ef8ecf5e4ae83681a7 (diff)
downloadsssd-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.c10
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;
}