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
From 95e821dde3cd60fccc33f55c95951179cbe89cbf Mon Sep 17 00:00:00 2001 From: Stephen Gallagher <sgall...@redhat.com> Date: Mon, 27 Feb 2012 12:00:52 -0500 Subject: [PATCH] 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 files changed, 3 insertions(+), 7 deletions(-) diff --git a/src/providers/data_provider_be.c b/src/providers/data_provider_be.c index 7e27ceacdb961cb9c5fd4e6874e1fdd31ca6d556..2b76eb3cef191ba40ebdc65445d2930f168254b3 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; } -- 1.7.7.6
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
