summaryrefslogtreecommitdiff
path: root/transaction.c
diff options
context:
space:
mode:
authorMichael Steinert <[email protected]>2023-09-21 08:11:19 -0500
committerMichael Steinert <[email protected]>2023-09-21 08:23:39 -0500
commitdf9e70a6d733f806a827e0a465ab33e7881b1e8d (patch)
tree432dfcf85b3451d0645d5c51355930ec305707e7 /transaction.c
parentd9b3ff918fc347ecb86bedd0aa32b91430b743c9 (diff)
Format transaction.c with clang-format
To improve readability and encourage consistency in PRs.
Diffstat (limited to 'transaction.c')
-rw-r--r--transaction.c37
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;
}