diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2011-12-14 16:06:21 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-03-14 03:50:17 -0700 |
| commit | f9be1e7c665be0507e431cb42cd55032807bd3ea (patch) | |
| tree | bdef869fa8cb2b368a704953bda3880e9c97f7e1 | |
| parent | 7161d0995c937c1712d29a04356a11ca51e9ce07 (diff) | |
performance/read-ahead: cleanup xlator private data completely in fini.
Change-Id: I3bb143036557d7d3844dc825f6870140e414f85a
BUG: 767862
Signed-off-by: Raghavendra G <raghavendra@gluster.com>
Reviewed-on: http://review.gluster.com/792
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>
| -rw-r--r-- | xlators/performance/read-ahead/src/read-ahead.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/xlators/performance/read-ahead/src/read-ahead.c b/xlators/performance/read-ahead/src/read-ahead.c index 1a4dda97ac6..f7f16115b93 100644 --- a/xlators/performance/read-ahead/src/read-ahead.c +++ b/xlators/performance/read-ahead/src/read-ahead.c @@ -1096,11 +1096,14 @@ fini (xlator_t *this) goto out; } + this->private = NULL; + + GF_ASSERT ((conf->files.next == &conf->files) + && (conf->files.prev == &conf->files)); + pthread_mutex_destroy (&conf->conf_lock); GF_FREE (conf); - this->private = NULL; - out: return; } |