Add a missing 'continue' in the code for the PAM_SUCCESS case.
Add more debugging log messages. Sponsored by: DARPA, NAI Labs git-svn-id: svn+ssh://svn.openpam.org/svn/openpam/trunk@56 185d5e19-27fe-0310-9dcf-9bff6b9f3609
This commit is contained in:
parent
39a1582d7e
commit
104cfd8443
|
@ -119,6 +119,7 @@ openpam_dispatch(pam_handle_t *pamh,
|
||||||
if (chain->flag == PAM_SUFFICIENT &&
|
if (chain->flag == PAM_SUFFICIENT &&
|
||||||
primitive != PAM_SM_SETCRED)
|
primitive != PAM_SM_SETCRED)
|
||||||
break;
|
break;
|
||||||
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
_openpam_check_error_code(primitive, r);
|
_openpam_check_error_code(primitive, r);
|
||||||
|
@ -131,6 +132,7 @@ openpam_dispatch(pam_handle_t *pamh,
|
||||||
if (err == 0)
|
if (err == 0)
|
||||||
err = r;
|
err = r;
|
||||||
if (chain->flag == PAM_REQUIRED && !fail) {
|
if (chain->flag == PAM_REQUIRED && !fail) {
|
||||||
|
openpam_log(PAM_LOG_DEBUG, "required module failed");
|
||||||
fail = 1;
|
fail = 1;
|
||||||
err = r;
|
err = r;
|
||||||
}
|
}
|
||||||
|
@ -140,12 +142,16 @@ openpam_dispatch(pam_handle_t *pamh,
|
||||||
* immediately.
|
* immediately.
|
||||||
*/
|
*/
|
||||||
if (chain->flag == PAM_REQUISITE) {
|
if (chain->flag == PAM_REQUISITE) {
|
||||||
|
openpam_log(PAM_LOG_DEBUG, "requisite module failed");
|
||||||
fail = 1;
|
fail = 1;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return (fail ? err : PAM_SUCCESS);
|
if (!fail)
|
||||||
|
err = PAM_SUCCESS;
|
||||||
|
openpam_log(PAM_LOG_DEBUG, "returning: %s", pam_strerror(pamh, err));
|
||||||
|
return (err);
|
||||||
}
|
}
|
||||||
|
|
||||||
#if !defined(OPENPAM_RELAX_CHECKS)
|
#if !defined(OPENPAM_RELAX_CHECKS)
|
||||||
|
|
Loading…
Reference in New Issue