diff options
author | Andreas Schneider <asn@samba.org> | 2020-03-23 17:47:16 +0100 |
---|---|---|
committer | Andreas Schneider <asn@samba.org> | 2020-03-24 08:38:02 +0100 |
commit | c559f8d98331ecfa7c32396556fac040c61b7e55 (patch) | |
tree | 02e292e0e4723204d282d5b0db0c56aa9f45045a | |
parent | 345596e3fdf6d0c94a36f36568766c0534ab082b (diff) | |
download | pam_wrapper-c559f8d98331ecfa7c32396556fac040c61b7e55.tar.gz pam_wrapper-c559f8d98331ecfa7c32396556fac040c61b7e55.tar.xz pam_wrapper-c559f8d98331ecfa7c32396556fac040c61b7e55.zip |
pwrap: Log the return code of pam_start()
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
-rw-r--r-- | src/pam_wrapper.c | 31 |
1 files changed, 18 insertions, 13 deletions
diff --git a/src/pam_wrapper.c b/src/pam_wrapper.c index 67a75b4..088754b 100644 --- a/src/pam_wrapper.c +++ b/src/pam_wrapper.c @@ -1222,6 +1222,8 @@ static int pwrap_pam_start(const char *service_name, const struct pam_conv *pam_conversation, pam_handle_t **pamh) { + int rc; + pwrap_init(); PWRAP_LOG(PWRAP_LOG_TRACE, @@ -1230,22 +1232,25 @@ static int pwrap_pam_start(const char *service_name, user); #if defined(HAVE_OPENPAM) - return pwrap_openpam_start(service_name, - user, - pam_conversation, - pamh); + rc = pwrap_openpam_start(service_name, + user, + pam_conversation, + pamh); #elif defined (HAVE_PAM_START_CONFDIR) - return libpam_pam_start_confdir(service_name, - user, - pam_conversation, - pwrap.config_dir, - pamh); + rc = libpam_pam_start_confdir(service_name, + user, + pam_conversation, + pwrap.config_dir, + pamh); #else - return libpam_pam_start(service_name, - user, - pam_conversation, - pamh); + rc = libpam_pam_start(service_name, + user, + pam_conversation, + pamh); #endif + PWRAP_LOG(PWRAP_LOG_TRACE, "pam_start rc=%d", rc); + + return rc; } |