diff options
| author | Vikas Gorur <vikas@gluster.com> | 2009-04-20 03:54:04 -0700 |
|---|---|---|
| committer | Anand V. Avati <avati@amp.gluster.com> | 2009-04-20 18:01:24 +0530 |
| commit | 82394d484803e02e28441bc0b988efaaff60dd94 (patch) | |
| tree | 87e5031e88f5de1abd3ea880b79aeb57ac0a545d /xlators/cluster/afr/src/afr-self-heal-metadata.c | |
| parent | fbefbfee743a01b5ad9977ad24de372258aeb65c (diff) | |
Check return value of afr_sh_select_source.
If select_source returns -1, abort self-heal.
Signed-off-by: Anand V. Avati <avati@amp.gluster.com>
Diffstat (limited to 'xlators/cluster/afr/src/afr-self-heal-metadata.c')
| -rw-r--r-- | xlators/cluster/afr/src/afr-self-heal-metadata.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-metadata.c b/xlators/cluster/afr/src/afr-self-heal-metadata.c index 8e832698f..6468d7fc8 100644 --- a/xlators/cluster/afr/src/afr-self-heal-metadata.c +++ b/xlators/cluster/afr/src/afr-self-heal-metadata.c @@ -568,6 +568,15 @@ afr_sh_metadata_fix (call_frame_t *frame, xlator_t *this) } source = afr_sh_select_source (sh->sources, priv->child_count); + + if (source == -1) { + gf_log (this->name, GF_LOG_DEBUG, + "No active sources found."); + + afr_sh_metadata_finish (frame, this); + return 0; + } + sh->source = source; /* detect changes not visible through pending flags -- JIC */ |