summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt/glusterd/src/glusterd-handler.c
diff options
context:
space:
mode:
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-handler.c119
1 files changed, 95 insertions, 24 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index f1a9ef5ef5c..c87ba07aa95 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -459,6 +459,8 @@ glusterd_handle_stage_op (rpcsvc_request_t *req)
ret = glusterd_op_sm_inject_event (GD_OP_EVENT_STAGE_OP, ctx);
out:
+ if (stage_req.buf.buf_val)
+ free (stage_req.buf.buf_val);//malloced by xdr
return ret;
}
@@ -506,6 +508,8 @@ glusterd_handle_commit_op (rpcsvc_request_t *req)
ret = glusterd_op_sm_inject_event (GD_OP_EVENT_COMMIT_OP, ctx);
out:
+ if (commit_req.buf.buf_val)
+ free (commit_req.buf.buf_val);//malloced by xdr
return ret;
}
@@ -551,6 +555,8 @@ glusterd_handle_cli_probe (rpcsvc_request_t *req)
gf_cmd_log ("peer probe","on host %s:%d %s",cli_req.hostname, cli_req.port,
(ret) ? "FAILED" : "SUCCESS");
out:
+ if (cli_req.hostname)
+ free (cli_req.hostname);//its malloced by xdr
return ret;
}
@@ -576,6 +582,8 @@ glusterd_handle_cli_deprobe (rpcsvc_request_t *req)
gf_cmd_log ("peer deprobe", "on host %s:%d %s", cli_req.hostname,
cli_req.port, (ret) ? "FAILED" : "SUCCESS");
out:
+ if (cli_req.hostname)
+ free (cli_req.hostname);//malloced by xdr
return ret;
}
@@ -608,6 +616,8 @@ glusterd_handle_cli_list_friends (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.dict.dict_val;
}
}
@@ -988,7 +998,8 @@ out:
ret = glusterd_submit_reply (req, &rsp, NULL, 0, NULL,
gf_xdr_serialize_cli_defrag_vol_rsp);
-
+ if (cli_req.volname)
+ free (cli_req.volname);//malloced by xdr
return ret;
}
@@ -1021,6 +1032,8 @@ glusterd_handle_cli_get_volume (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.dict.dict_val;
}
}
@@ -1087,6 +1100,8 @@ glusterd_handle_create_volume (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.bricks.bricks_val;
}
}
@@ -1198,6 +1213,8 @@ brick_validation:
((ret || err_ret) != 0) ? "FAILED": "SUCCESS");
out:
+ if ((err_ret || ret) && dict)
+ dict_unref (dict);
if (err_ret) {
rsp.op_ret = -1;
rsp.op_errno = 0;
@@ -1216,6 +1233,8 @@ out:
GF_FREE(free_ptr);
if (brickinfo)
glusterd_brickinfo_delete (brickinfo);
+ if (cli_req.volname)
+ free (cli_req.volname); // its a malloced by xdr
return ret;
}
@@ -1224,7 +1243,6 @@ glusterd_handle_cli_start_volume (rpcsvc_request_t *req)
{
int32_t ret = -1;
gf1_cli_start_vol_req cli_req = {0,};
- //dict_t *dict = NULL;
int32_t flags = 0;
GF_ASSERT (req);
@@ -1244,7 +1262,8 @@ glusterd_handle_cli_start_volume (rpcsvc_request_t *req)
((ret == 0) ? "SUCCESS": "FAILED"));
out:
-
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1272,6 +1291,8 @@ glusterd_handle_cli_stop_volume (rpcsvc_request_t *req)
((ret)?"FAILED":"SUCCESS"));
out:
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1300,6 +1321,8 @@ glusterd_handle_cli_delete_volume (rpcsvc_request_t *req)
((ret) ? "FAILED" : "SUCCESS"));
out:
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1327,6 +1350,7 @@ glusterd_handle_add_brick (rpcsvc_request_t *req)
glusterd_volinfo_t *tmpvolinfo = NULL;
glusterd_conf_t *priv = NULL;
xlator_t *this = NULL;
+ char *free_ptr = NULL;
this = THIS;
priv = this->private;
@@ -1357,6 +1381,8 @@ glusterd_handle_add_brick (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.bricks.bricks_val;
}
}
@@ -1416,10 +1442,17 @@ brick_val:
if (bricks)
brick_list = gf_strdup (bricks);
+ if (!brick_list) {
+ gf_log ("", GF_LOG_ERROR, "Out of memory");
+ ret = -1;
+ goto out;
+ } else {
+ free_ptr = brick_list;
+ }
gf_cmd_log ("Volume add-brick", "volname: %s type %s count:%d bricks:%s"
,volname, ((volinfo->type == 0)? "DEFAULT" : ((volinfo->type
- == 1)? "STRIPE": "REPLICATE")), brick_count, brick_list);
+ == 1)? "STRIPE": "REPLICATE")), brick_count, brick_list);
while ( i < brick_count) {
i++;
@@ -1438,7 +1471,7 @@ brick_val:
snprintf(err_str, 1048, "Host %s not a friend",
brickinfo->hostname);
gf_log ("glusterd", GF_LOG_ERROR, "%s", err_str);
- err_ret = 1;
+ err_ret = 1;
goto out;
}
if ((!peerinfo->connected) &&
@@ -1475,6 +1508,8 @@ brick_validation:
((ret || err_ret) != 0)? "FAILED" : "SUCCESS");
out:
+ if ((err_ret || ret) && dict)
+ dict_unref (dict);
if (err_ret) {
rsp.op_ret = -1;
rsp.op_errno = 0;
@@ -1491,6 +1526,10 @@ out:
}
if (brickinfo)
glusterd_brickinfo_delete (brickinfo);
+ if (free_ptr)
+ GF_FREE (free_ptr);
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1529,6 +1568,8 @@ glusterd_handle_replace_brick (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.bricks.bricks_val;
}
}
@@ -1584,6 +1625,10 @@ glusterd_handle_replace_brick (rpcsvc_request_t *req)
(ret) ? "FAILED" : "SUCCESS");
out:
+ if (ret && dict)
+ dict_unref (dict);
+ if (cli_req.volname)
+ free (cli_req.volname);//malloced by xdr
return ret;
}
@@ -1633,6 +1678,8 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = cli_req.bricks.bricks_val;
}
}
@@ -1653,7 +1700,7 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
ret = glusterd_volinfo_find (cli_req.volname, &volinfo);
if (ret) {
- snprintf (err_str, 2048, "volname %s not found",
+ snprintf (err_str, 2048, "volname %s not found",
cli_req.volname);
gf_log ("", GF_LOG_ERROR, "%s", err_str);
err_ret = 1;
@@ -1670,7 +1717,7 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
err_ret = 1;
ret = -1;
goto out;
- }
+ }
}
@@ -1694,10 +1741,10 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
gf_log ("", GF_LOG_DEBUG, "Remove brick count %d brick: %s",
i, brick);
- ret = glusterd_brickinfo_get(brick, volinfo, &brickinfo);
+ ret = glusterd_brickinfo_get(brick, volinfo, &brickinfo);
if (ret) {
snprintf(err_str, 2048," Incorrect brick %s for volname"
- " %s", brick, cli_req.volname);
+ " %s", brick, cli_req.volname);
gf_log ("", GF_LOG_ERROR, "%s", err_str);
err_ret = 1;
goto out;
@@ -1732,16 +1779,16 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
" for replica");
gf_log ("",GF_LOG_ERROR,
"%s", err_str);
- err_ret = 1;
+ err_ret = 1;
goto out;
}
}
break;
- }
+ }
pos++;
}
}
- gf_cmd_log ("Volume remove-brick","volname: %s count:%d bricks:%s",
+ gf_cmd_log ("Volume remove-brick","volname: %s count:%d bricks:%s",
cli_req.volname, count, brick_list);
ret = glusterd_remove_brick (req, dict);
@@ -1750,6 +1797,8 @@ glusterd_handle_remove_brick (rpcsvc_request_t *req)
(ret) ? "FAILED" : "SUCCESS");
out:
+ if ((ret || err_ret) && dict)
+ dict_unref (dict);
if (err_ret) {
rsp.op_ret = -1;
rsp.op_errno = 0;
@@ -1763,12 +1812,14 @@ out:
"opinfo failed");
ret = 0; //sent error to cli, prevent second reply
-
+
}
if (brick_list)
GF_FREE (brick_list);
if (err_str)
GF_FREE (err_str);
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1794,19 +1845,21 @@ glusterd_handle_log_filename (rpcsvc_request_t *req)
if (!dict)
goto out;
- ret = dict_set_str (dict, "volname", cli_req.volname);
+ ret = dict_set_dynmstr (dict, "volname", cli_req.volname);
if (ret)
goto out;
- ret = dict_set_str (dict, "brick", cli_req.brick);
+ ret = dict_set_dynmstr (dict, "brick", cli_req.brick);
if (ret)
goto out;
- ret = dict_set_str (dict, "path", cli_req.path);
+ ret = dict_set_dynmstr (dict, "path", cli_req.path);
if (ret)
goto out;
ret = glusterd_log_filename (req, dict);
out:
+ if (ret && dict)
+ dict_unref (dict);
return ret;
}
@@ -1866,6 +1919,10 @@ out:
ret = glusterd_submit_reply (req, &rsp, NULL, 0, NULL,
gf_xdr_serialize_cli_log_locate_rsp);
+ if (cli_req.brick)
+ free (cli_req.brick); //its malloced by xdr
+ if (cli_req.volname)
+ free (cli_req.volname); //its malloced by xdr
return ret;
}
@@ -1891,11 +1948,11 @@ glusterd_handle_log_rotate (rpcsvc_request_t *req)
if (!dict)
goto out;
- ret = dict_set_str (dict, "volname", cli_req.volname);
+ ret = dict_set_dynmstr (dict, "volname", cli_req.volname);
if (ret)
goto out;
- ret = dict_set_str (dict, "brick", cli_req.brick);
+ ret = dict_set_dynmstr (dict, "brick", cli_req.brick);
if (ret)
goto out;
@@ -1906,6 +1963,8 @@ glusterd_handle_log_rotate (rpcsvc_request_t *req)
ret = glusterd_log_rotate (req, dict);
out:
+ if (ret && dict)
+ dict_unref (dict);
return ret;
}
@@ -2135,12 +2194,18 @@ glusterd_handle_incoming_friend_req (rpcsvc_request_t *req)
if (ret)
goto out;
+ else
+ dict->extra_stdfree = friend_req.vols.vols_val;
ret = glusterd_handle_friend_req (req, friend_req.uuid,
friend_req.hostname, friend_req.port,
dict);
out:
+ if (ret && dict)
+ dict_unref (dict);
+ if (friend_req.hostname)
+ free (friend_req.hostname);//malloced by xdr
return ret;
}
@@ -2167,7 +2232,10 @@ glusterd_handle_incoming_unfriend_req (rpcsvc_request_t *req)
friend_req.hostname, friend_req.port);
out:
-
+ if (friend_req.hostname)
+ free (friend_req.hostname);//malloced by xdr
+ if (friend_req.vols.vols_val)
+ free (friend_req.vols.vols_val);//malloced by xdr
return ret;
}
@@ -2219,6 +2287,8 @@ glusterd_handle_friend_update (rpcsvc_request_t *req)
"failed to "
"unserialize req-buffer to dictionary");
goto out;
+ } else {
+ dict->extra_stdfree = friend_req.friends.friends_val;
}
}
@@ -2278,13 +2348,10 @@ glusterd_handle_probe_query (rpcsvc_request_t *req)
glusterd_conf_t *conf = NULL;
gd1_mgmt_probe_req probe_req = {{0},};
gd1_mgmt_probe_rsp rsp = {{0},};
- char hostname[1024] = {0};
glusterd_peer_hostname_t *name = NULL;
GF_ASSERT (req);
- probe_req.hostname = hostname;
-
if (!gd_xdr_to_mgmt_probe_req (req->msg[0], &probe_req)) {
//failed to decode msg;
req->rpc_err = GARBAGE_ARGS;
@@ -2319,6 +2386,8 @@ glusterd_handle_probe_query (rpcsvc_request_t *req)
"Responded to %s, ret: %d", probe_req.hostname, ret);
out:
+ if (probe_req.hostname)
+ free (probe_req.hostname);//malloced by xdr
return ret;
}
@@ -2378,7 +2447,7 @@ glusterd_friend_add (const char *hoststr, int port,
goto out;
}
- ret = dict_set_str (options, "remote-host", hostname);
+ ret = dict_set_dynstr (options, "remote-host", hostname);
if (ret)
goto out;
@@ -2767,7 +2836,7 @@ glusterd_stop_volume (rpcsvc_request_t *req, char *volname, int flags)
if (!dup_volname)
goto out;
- ret = dict_set_str (ctx, "volname", dup_volname);
+ ret = dict_set_dynstr (ctx, "volname", dup_volname);
if (ret)
goto out;
@@ -2784,6 +2853,8 @@ glusterd_stop_volume (rpcsvc_request_t *req, char *volname, int flags)
ret = glusterd_op_txn_begin ();
out:
+ if (ret && ctx)
+ dict_unref (ctx);
return ret;
}