On 04/27/2010 07:18 AM, Sumit Bose wrote:
Hi,
the patch which removes the authentication tokens from the PAM stack
didn't return the right status code if a password change fails. To make
it easier to read I introduced a new variable.
bye,
Sumit
Nack.
Under SSS_PAM_AUTHENTICATE:
exp_data = malloc(sizeof(int));
if (exp_data == NULL) {
D(("malloc failed."));
ret = PAM_BUF_ERR;
break;
}
You're still using ret here, which means that instead of returning
PAM_BUF_ERR from the function, you're returning whatever came back from
send_and_receive() above.
--
Stephen Gallagher
RHCE 804006346421761
Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/