Skip to content

Commit

Permalink
PAM: Don't send PAM_SYSTEM_INFO message if module unset
Browse files Browse the repository at this point in the history
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
  • Loading branch information
sgallagher committed Feb 27, 2012
1 parent 71ba808 commit 89325d4
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions src/providers/data_provider_be.c
Expand Up @@ -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;
}

Expand Down

0 comments on commit 89325d4

Please sign in to comment.