summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKrutika Dhananjay <kdhananj@redhat.com>2014-09-02 21:05:56 +0530
committerPranith Kumar Karampuri <pkarampu@redhat.com>2014-09-03 04:10:37 -0700
commit1f094f2519c163bd033ab535555448db34e9804d (patch)
tree911e272e02091d5af4e751513b4cc4e1bc40d43c
parente03559c20ab37f1a7db54a367258bb1cd005e50d (diff)
cluster/afr: Minor fixes to #8574
* Fixed a bug in afr_selfheal_name_type_mismatch_check() * Fixed indentation * Removed redundant 'continue' statements Change-Id: Ie58b5dec9085ce9fe46ae9f244ebae1b1cef7ac5 BUG: 1132469 Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com> Reviewed-on: http://review.gluster.org/8586 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Tested-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
-rw-r--r--xlators/cluster/afr/src/afr-self-heal-name.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-name.c b/xlators/cluster/afr/src/afr-self-heal-name.c
index 9e7bb3bffa4..f8b887e9a24 100644
--- a/xlators/cluster/afr/src/afr-self-heal-name.c
+++ b/xlators/cluster/afr/src/afr-self-heal-name.c
@@ -291,12 +291,11 @@ afr_selfheal_name_type_mismatch_check (xlator_t *this, struct afr_reply *replies
priv->children[i]->name,
replies[type_idx].poststat.ia_type,
priv->children[type_idx]->name);
- return -EIO;
+ return -EIO;
}
+ inode_type = replies[i].poststat.ia_type;
+ type_idx = i;
}
- inode_type = replies[i].poststat.ia_type;
- type_idx = i;
- continue;
}
return 0;
}
@@ -345,7 +344,6 @@ afr_selfheal_name_gfid_mismatch_check (xlator_t *this, struct afr_reply *replies
gfid = &replies[i].poststat.ia_gfid;
gfid_idx_iter = i;
- continue;
}
}