summaryrefslogtreecommitdiffstats
path: root/libglusterfs
diff options
context:
space:
mode:
authorAmar Tumballi <amar@gluster.com>2010-09-28 05:37:02 +0000
committerVijay Bellur <vijay@dev.gluster.com>2010-09-28 09:04:51 -0700
commit387f4d385e1d2c29af54c4f87eef7a07acfcfdf9 (patch)
tree22a63e2bac709e11329e76b445b1bd8739a12581 /libglusterfs
parent5ea64d983b31c6882d4582af71a339b6a4fcccf2 (diff)
more proper error returns in case of graph topology validation
Signed-off-by: Amar Tumballi <amar@gluster.com> Signed-off-by: Vijay Bellur <vijay@dev.gluster.com> BUG: 971 (dynamic volume management) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=971
Diffstat (limited to 'libglusterfs')
-rw-r--r--libglusterfs/src/graph.c5
-rw-r--r--libglusterfs/src/xlator.c9
2 files changed, 5 insertions, 9 deletions
diff --git a/libglusterfs/src/graph.c b/libglusterfs/src/graph.c
index c44fa44ad..0c63c194d 100644
--- a/libglusterfs/src/graph.c
+++ b/libglusterfs/src/graph.c
@@ -501,7 +501,6 @@ glusterfs_graph_reconfigure (glusterfs_graph_t *oldgraph,
{
xlator_t *old_xl = NULL;
xlator_t *new_xl = NULL;
- int ret = 0;
GF_ASSERT (oldgraph);
GF_ASSERT (newgraph);
@@ -509,9 +508,7 @@ glusterfs_graph_reconfigure (glusterfs_graph_t *oldgraph,
old_xl = oldgraph->first;
new_xl = newgraph->first;
- ret = xlator_tree_reconfigure (old_xl, new_xl);
-
- return ret;
+ return xlator_tree_reconfigure (old_xl, new_xl);
}
int
diff --git a/libglusterfs/src/xlator.c b/libglusterfs/src/xlator.c
index ef0808d5d..a09dd2e1e 100644
--- a/libglusterfs/src/xlator.c
+++ b/libglusterfs/src/xlator.c
@@ -916,7 +916,7 @@ xlator_fini_rec (xlator_t *xl)
}
}
-static void
+static int
xlator_reconfigure_rec (xlator_t *old_xl, xlator_t *new_xl)
{
xlator_list_t *trav1 = NULL;
@@ -924,7 +924,7 @@ xlator_reconfigure_rec (xlator_t *old_xl, xlator_t *new_xl)
if (old_xl == NULL || new_xl == NULL) {
gf_log ("xlator", GF_LOG_DEBUG, "invalid argument");
- return;
+ return -1;
}
trav1 = old_xl->children;
@@ -944,6 +944,7 @@ xlator_reconfigure_rec (xlator_t *old_xl, xlator_t *new_xl)
else
gf_log (old_xl->name, GF_LOG_DEBUG, "No reconfigure() found");
+ return 0;
}
int
@@ -1023,9 +1024,7 @@ xlator_tree_reconfigure (xlator_t *old_xl, xlator_t *new_xl)
old_top = old_xl;
new_top = new_xl;
- xlator_reconfigure_rec (old_top, new_top);
-
- return 0;
+ return xlator_reconfigure_rec (old_top, new_top);
}