diff options
| author | Krutika Dhananjay <kdhananj@redhat.com> | 2014-09-02 21:05:56 +0530 |
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2014-09-03 04:10:37 -0700 |
| commit | 1f094f2519c163bd033ab535555448db34e9804d (patch) | |
| tree | 911e272e02091d5af4e751513b4cc4e1bc40d43c | |
| parent | e03559c20ab37f1a7db54a367258bb1cd005e50d (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.c | 8 |
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; } } |