diff options
| author | Susant Palai <spalai@redhat.com> | 2015-05-05 18:11:52 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-05-05 23:07:08 -0700 |
| commit | 68bc3c88fc37befe9ef73c8c7f5a2620c94ff4ce (patch) | |
| tree | 0f6e2f8c5f85c00ec3f30149285d75525bdd9194 | |
| parent | ebf068c4ca4cfe7abcf56a816de7561130ffabf7 (diff) | |
dht/rebalance: Go to exit if defrag is NULL
Problem: In case a defrag is null, going to out section will crash rebalance.
Change-Id: I8b3ee1ad85dc23ef0e2f2dd6f912d07216bd619f
Signed-off-by: Susant Palai <spalai@redhat.com>
Reviewed-on: http://review.gluster.org/10582
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: N Balachandran <nbalacha@redhat.com>
| -rw-r--r-- | xlators/cluster/dht/src/dht-rebalance.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index 09cfe515b49..e97e3916fd7 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -2477,19 +2477,19 @@ gf_defrag_start_crawl (void *data) this = data; if (!this) - goto out; + goto exit; ctx = this->ctx; if (!ctx) - goto out; + goto exit; conf = this->private; if (!conf) - goto out; + goto exit; defrag = conf->defrag; if (!defrag) - goto out; + goto exit; gettimeofday (&defrag->start_time, NULL); dht_build_root_inode (this, &defrag->root_inode); @@ -2688,7 +2688,7 @@ out: if (dict) dict_unref(dict); - +exit: return ret; } |