summaryrefslogtreecommitdiffstats
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:29 -0500
commit9978bffd3ba92869101272e8a59f39cd96914b7c (patch)
tree1825c8ff371c121260bb19bf8ea58aac312283af
parentbb5f8ee0621689beb4e6b0ad1a20710894a42b06 (diff)
downloadsssd-9978bffd3ba92869101272e8a59f39cd96914b7c.tar.gz
sssd-9978bffd3ba92869101272e8a59f39cd96914b7c.tar.xz
sssd-9978bffd3ba92869101272e8a59f39cd96914b7c.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
-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 7e27ceacd..2b76eb3ce 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;
}