From 9978bffd3ba92869101272e8a59f39cd96914b7c Mon Sep 17 00:00:00 2001 From: Stephen Gallagher Date: Mon, 27 Feb 2012 12:00:52 -0500 Subject: 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 --- src/providers/data_provider_be.c | 10 +++------- 1 file 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; } -- cgit