From d2bd17e5a53d0ffa375df1a5ad957556be2f2b83 Mon Sep 17 00:00:00 2001 From: Pranith Kumar K Date: Wed, 6 Jan 2016 14:30:08 +0530 Subject: dict: Don't expose get_new_dict/dict_destroy get_new_dict/dict_destroy is causing confusion where, dict_new/dict_destroy or get_new_dict/dict_unref are used instead of dict_new/dict_unref. Change-Id: I4cc69f5b6711d720823395e20fd624a0c6c1168c BUG: 1296043 Signed-off-by: Pranith Kumar K Reviewed-on: http://review.gluster.org/13183 Smoke: Gluster Build System CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Jeff Darcy Reviewed-by: Krutika Dhananjay --- xlators/mgmt/glusterd/src/glusterd-geo-rep.c | 4 ++-- xlators/mgmt/glusterd/src/glusterd-op-sm.c | 4 ++-- xlators/mgmt/glusterd/src/glusterd-volgen.c | 12 ++++++------ 3 files changed, 10 insertions(+), 10 deletions(-) (limited to 'xlators/mgmt') diff --git a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c index 55e249643c0..67f27712f86 100644 --- a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c +++ b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c @@ -1982,7 +1982,7 @@ is_geo_rep_active (glusterd_volinfo_t *volinfo, char *slave, ret = 0; out: if (confd) - dict_destroy (confd); + dict_unref (confd); return ret; } @@ -2621,7 +2621,7 @@ fetch_data: } if (confd) - dict_destroy (confd); + dict_unref (confd); gf_msg_debug (this->name, 0, "Returning %d ", ret); return ret; diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c index e9f261c2fb3..b06a5978540 100644 --- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c +++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c @@ -4047,7 +4047,7 @@ glusterd_op_build_payload (dict_t **req, char **op_errstr, dict_t *op_ctx) if (ret) goto out; } - dict_destroy (req_dict); + dict_unref (req_dict); req_dict = dict_ref (dict); } break; @@ -4073,7 +4073,7 @@ glusterd_op_build_payload (dict_t **req, char **op_errstr, dict_t *op_ctx) goto out; } - dict_destroy (req_dict); + dict_unref (req_dict); req_dict = dict_ref (dict); } break; diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index 6a755486d7d..9eefd2a8491 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -93,7 +93,7 @@ xlator_instantiate_va (const char *type, const char *format, va_list arg) ret = xlator_set_type_virtual (xl, type); if (ret) goto error; - xl->options = get_new_dict(); + xl->options = dict_new (); if (!xl->options) goto error; xl->name = volname; @@ -1065,7 +1065,7 @@ build_graph_generic (volgen_graph_t *graph, glusterd_volinfo_t *volinfo, int ret = 0; if (mod_dict) { - set_dict = dict_copy (volinfo->dict, NULL); + set_dict = dict_copy_with_ref (volinfo->dict, NULL); if (!set_dict) return -1; dict_copy (mod_dict, set_dict); @@ -1079,7 +1079,7 @@ build_graph_generic (volgen_graph_t *graph, glusterd_volinfo_t *volinfo, ret = volgen_graph_set_options (graph, set_dict); if (mod_dict) - dict_destroy (set_dict); + dict_unref (set_dict); return ret; } @@ -4798,7 +4798,7 @@ build_rebalance_volfile (glusterd_volinfo_t *volinfo, char *filepath, return 0; } - set_dict = dict_copy (volinfo->dict, NULL); + set_dict = dict_copy_with_ref (volinfo->dict, NULL); if (!set_dict) return -1; @@ -4840,7 +4840,7 @@ build_rebalance_volfile (glusterd_volinfo_t *volinfo, char *filepath, out: volgen_graph_free (&graph); - dict_destroy (set_dict); + dict_unref (set_dict); return ret; } @@ -5106,7 +5106,7 @@ build_nfs_graph (volgen_graph_t *graph, dict_t *mod_dict) out: gf_msg_debug ("glusterd", 0, "Returning %d", ret); - dict_destroy (set_dict); + dict_unref (set_dict); return ret; } -- cgit