diff options
author | Michael Steinert <[email protected]> | 2023-09-21 08:11:19 -0500 |
---|---|---|
committer | Michael Steinert <[email protected]> | 2023-09-21 08:23:39 -0500 |
commit | df9e70a6d733f806a827e0a465ab33e7881b1e8d (patch) | |
tree | 432dfcf85b3451d0645d5c51355930ec305707e7 /transaction.c | |
parent | d9b3ff918fc347ecb86bedd0aa32b91430b743c9 (diff) |
Format transaction.c with clang-format
To improve readability and encourage consistency in PRs.
Diffstat (limited to 'transaction.c')
-rw-r--r-- | transaction.c | 37 |
1 files changed, 18 insertions, 19 deletions
diff --git a/transaction.c b/transaction.c index df25cf6..dcc32eb 100644 --- a/transaction.c +++ b/transaction.c @@ -9,29 +9,23 @@ #define PAM_CONST const #endif -int cb_pam_conv( - int num_msg, - PAM_CONST struct pam_message **msg, - struct pam_response **resp, - void *appdata_ptr) +int cb_pam_conv(int num_msg, PAM_CONST struct pam_message **msg, struct pam_response **resp, void *appdata_ptr) { *resp = calloc(num_msg, sizeof **resp); - if (num_msg <= 0 || num_msg > PAM_MAX_NUM_MSG) { + if (num_msg <= 0 || num_msg > PAM_MAX_NUM_MSG) return PAM_CONV_ERR; - } - if (!*resp) { + + if (!*resp) return PAM_BUF_ERR; - } + for (size_t i = 0; i < num_msg; ++i) { - struct cbPAMConv_return result = cbPAMConv( - msg[i]->msg_style, - (char *)msg[i]->msg, - (uintptr_t)appdata_ptr); - if (result.r1 != PAM_SUCCESS) { + struct cbPAMConv_return result = cbPAMConv(msg[i]->msg_style, (char *)msg[i]->msg, (uintptr_t)appdata_ptr); + if (result.r1 != PAM_SUCCESS) goto error; - } + (*resp)[i].resp = result.r0; } + return PAM_SUCCESS; error: for (size_t i = 0; i < num_msg; ++i) { @@ -40,6 +34,7 @@ error: free((*resp)[i].resp); } } + memset(*resp, 0, num_msg * sizeof *resp); free(*resp); *resp = NULL; @@ -52,11 +47,15 @@ void init_pam_conv(struct pam_conv *conv, uintptr_t appdata) conv->appdata_ptr = (void *)appdata; } -// pam_start_confdir is a recent PAM api to declare a confdir (mostly for tests) -// weaken the linking dependency to detect if it’s present. -int pam_start_confdir(const char *service_name, const char *user, const struct pam_conv *pam_conversation, const char *confdir, pam_handle_t **pamh) __attribute__ ((weak)); -int check_pam_start_confdir(void) { +// pam_start_confdir is a recent PAM api to declare a confdir (mostly for +// tests) weaken the linking dependency to detect if it’s present. +int pam_start_confdir(const char *service_name, const char *user, const struct pam_conv *pam_conversation, + const char *confdir, pam_handle_t **pamh) __attribute__((weak)); + +int check_pam_start_confdir(void) +{ if (pam_start_confdir == NULL) return 1; + return 0; } |