From b40893d1bb4c626cb89cd86a1ec301459be5f991 Mon Sep 17 00:00:00 2001 From: "Kaleb S. KEITHLEY" Date: Mon, 22 Aug 2016 13:22:03 -0400 Subject: glusterd (mgmt): fix unused variable warnings/errors http://review.gluster.org/14085 fixes a/the "leak" - via the generated rpc/xdr headers - of pragmas that mask these warnings. However 14085 won't pass the smoke test until all the warnings are fixed. Change-Id: Ia98f2934b179145398895600a3f34dbff841c3bd BUG: 1369124 Signed-off-by: Kaleb S. KEITHLEY Reviewed-on: http://review.gluster.org/15268 CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System Smoke: Gluster Build System Reviewed-by: Atin Mukherjee --- xlators/mgmt/glusterd/src/glusterd-mgmt.c | 65 ++++++++++++------------------- 1 file changed, 24 insertions(+), 41 deletions(-) (limited to 'xlators/mgmt/glusterd/src/glusterd-mgmt.c') diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt.c b/xlators/mgmt/glusterd/src/glusterd-mgmt.c index 092283a7daf..6cc7b84b8b2 100644 --- a/xlators/mgmt/glusterd/src/glusterd-mgmt.c +++ b/xlators/mgmt/glusterd/src/glusterd-mgmt.c @@ -37,11 +37,9 @@ gd_mgmt_v3_collate_errors (struct syncargs *args, int op_ret, int op_errno, char *peer_str = NULL; char err_str[PATH_MAX] = "Please check log file for details."; char op_err[PATH_MAX] = ""; - int32_t len = -1; xlator_t *this = NULL; int is_operrstr_blk = 0; char *err_string = NULL; - char *cli_err_str = NULL; glusterd_peerinfo_t *peerinfo = NULL; this = THIS; @@ -68,68 +66,59 @@ gd_mgmt_v3_collate_errors (struct syncargs *args, int op_ret, int op_errno, switch (op_code) { case GLUSTERD_MGMT_V3_LOCK: { - len = snprintf (op_err, sizeof(op_err), - "Locking failed " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Locking failed on %s. %s", + peer_str, err_string); break; } case GLUSTERD_MGMT_V3_PRE_VALIDATE: { - len = snprintf (op_err, sizeof(op_err), - "Pre Validation failed " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Pre Validation failed on %s. %s", + peer_str, err_string); break; } case GLUSTERD_MGMT_V3_BRICK_OP: { - len = snprintf (op_err, sizeof(op_err), - "Brick ops failed " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Brick ops failed on %s. %s", + peer_str, err_string); break; } case GLUSTERD_MGMT_V3_COMMIT: { - len = snprintf (op_err, sizeof(op_err), - "Commit failed" - " on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Commit failed on %s. %s", + peer_str, err_string); break; } case GLUSTERD_MGMT_V3_POST_VALIDATE: { - len = snprintf (op_err, sizeof(op_err), - "Post Validation failed " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Post Validation failed on %s. %s", + peer_str, err_string); break; } case GLUSTERD_MGMT_V3_UNLOCK: { - len = snprintf (op_err, sizeof(op_err), - "Unlocking failed " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Unlocking failed on %s. %s", + peer_str, err_string); break; } default : - len = snprintf (op_err, sizeof(op_err), - "Unknown error! " - "on %s. %s", peer_str, - err_string); + snprintf (op_err, sizeof(op_err), + "Unknown error! on %s. %s", + peer_str, err_string); } if (args->errstr) { - len = snprintf (err_str, sizeof(err_str), - "%s\n%s", args->errstr, - op_err); + snprintf (err_str, sizeof(err_str), + "%s\n%s", args->errstr, op_err); GF_FREE (args->errstr); args->errstr = NULL; } else - len = snprintf (err_str, sizeof(err_str), - "%s", op_err); + snprintf (err_str, sizeof(err_str), "%s", op_err); gf_msg (this->name, GF_LOG_ERROR, 0, GD_MSG_MGMTV3_OP_FAIL, "%s", op_err); @@ -428,7 +417,6 @@ gd_mgmt_v3_lock_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_lock_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; @@ -700,7 +688,6 @@ gd_mgmt_v3_pre_validate_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_pre_val_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; @@ -1021,7 +1008,6 @@ gd_mgmt_v3_brick_op_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_brick_op_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; @@ -1242,7 +1228,6 @@ gd_mgmt_v3_commit_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_commit_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; @@ -1504,7 +1489,6 @@ gd_mgmt_v3_post_validate_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_post_val_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; @@ -1726,7 +1710,6 @@ gd_mgmt_v3_unlock_cbk_fn (struct rpc_req *req, struct iovec *iov, { int32_t ret = -1; struct syncargs *args = NULL; - glusterd_peerinfo_t *peerinfo = NULL; gd1_mgmt_v3_unlock_rsp rsp = {{0},}; call_frame_t *frame = NULL; int32_t op_ret = -1; -- cgit