From 0e651882a9aadcb35ea764ea65c05c3661175e00 Mon Sep 17 00:00:00 2001 From: Raghavendra Bhat Date: Mon, 17 Jan 2011 00:44:39 +0000 Subject: remove op_errstr from glusterd_handle_rpc_msg as op_errstr is not used in that function Signed-off-by: Raghavendra Bhat Signed-off-by: Anand V. Avati BUG: 1598 ([glusterfs-3.1.0qa22]: crash in glusterd) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1598 --- xlators/mgmt/glusterd/src/glusterd3_1-mops.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'xlators') diff --git a/xlators/mgmt/glusterd/src/glusterd3_1-mops.c b/xlators/mgmt/glusterd/src/glusterd3_1-mops.c index b5ac57194f1..09b6b175444 100644 --- a/xlators/mgmt/glusterd/src/glusterd3_1-mops.c +++ b/xlators/mgmt/glusterd/src/glusterd3_1-mops.c @@ -1342,7 +1342,6 @@ glusterd_handle_rpc_msg (rpcsvc_request_t *req) { int ret = -1; gf_boolean_t is_cli_req = _gf_false; - char *op_errstr = NULL; int lock_fail = 0; GF_ASSERT (req); @@ -1490,10 +1489,9 @@ out: request, which causes double replies */ if (!lock_fail) (void) glusterd_opinfo_unlock (); - ret = glusterd_op_send_cli_response (req->procnum, ret, 0, req, NULL, op_errstr); + ret = glusterd_op_send_cli_response (req->procnum, ret, 0, req, NULL, "operation failed"); } - if (op_errstr && (strcmp (op_errstr, ""))) - GF_FREE (op_errstr); + if (ret) gf_log ("", GF_LOG_WARNING, "Returning %d", ret); -- cgit