Skip to content

Commit a92976b

Browse files
committed
mctpd: minor formatting fixes for consistency
Use reverse-christmas-tree for var declarations, and be consistent with the result variable usage. Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
1 parent 2950eda commit a92976b

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

src/mctpd.c

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -846,13 +846,13 @@ handle_control_get_version_support(struct ctx *ctx, int sd,
846846
const struct sockaddr_mctp_ext *addr,
847847
const uint8_t *buf, const size_t buf_size)
848848
{
849-
struct mctp_ctrl_cmd_get_mctp_ver_support *req = NULL;
850849
struct mctp_ctrl_resp_get_mctp_ver_support *resp = NULL;
850+
struct mctp_ctrl_cmd_get_mctp_ver_support *req = NULL;
851+
size_t resp_len, i, ver_count = 0, ver_bytes_count;
851852
uint32_t *versions = NULL;
852853
uint8_t *respbuf = NULL;
853-
size_t resp_len, i, ver_count = 0, ver_bytes_count;
854854
ssize_t ver_idx = -1;
855-
int status;
855+
int rc;
856856

857857
if (buf_size < sizeof(struct mctp_ctrl_cmd_get_mctp_ver_support)) {
858858
warnx("short Get Version Support message");
@@ -902,9 +902,10 @@ handle_control_get_version_support(struct ctx *ctx, int sd,
902902

903903
mctp_ctrl_msg_hdr_init_resp(&resp->ctrl_hdr, req->ctrl_hdr);
904904

905-
status = reply_message(ctx, sd, resp, resp_len, addr);
905+
rc = reply_message(ctx, sd, resp, resp_len, addr);
906906
free(respbuf);
907-
return status;
907+
908+
return rc;
908909
}
909910

910911
static int handle_control_get_endpoint_id(struct ctx *ctx, int sd,
@@ -959,11 +960,11 @@ static int handle_control_get_message_type_support(
959960
struct ctx *ctx, int sd, const struct sockaddr_mctp_ext *addr,
960961
const uint8_t *buf, const size_t buf_size)
961962
{
962-
struct mctp_ctrl_cmd_get_msg_type_support *req = NULL;
963963
struct mctp_ctrl_resp_get_msg_type_support *resp = NULL;
964+
struct mctp_ctrl_cmd_get_msg_type_support *req = NULL;
965+
size_t i, resp_len, type_count;
964966
uint8_t *resp_buf, *msg_types;
965-
size_t resp_len, type_count;
966-
size_t i;
967+
int rc;
967968

968969
if (buf_size < sizeof(*req)) {
969970
warnx("short Get Message Type Support message");
@@ -991,9 +992,10 @@ static int handle_control_get_message_type_support(
991992
msg_types[i] = ctx->supported_msg_types[i].msg_type;
992993
}
993994

994-
int result = reply_message(ctx, sd, resp, resp_len, addr);
995+
rc = reply_message(ctx, sd, resp, resp_len, addr);
995996
free(resp_buf);
996-
return result;
997+
998+
return rc;
997999
}
9981000

9991001
static int

0 commit comments

Comments
 (0)