diff options
author | Harshavardhana Ranganath <harsha@gluster.com> | 2009-12-17 15:41:21 +0000 |
---|---|---|
committer | Vijay Bellur <vijay@dev.gluster.com> | 2009-12-18 08:41:29 -0800 |
commit | ff5170faa131b201bf8170601dbeae18b400542f (patch) | |
tree | d8aa89f66b642036d5ce6ad62a0bdb566e61ac16 /xlators/performance | |
parent | cfe095763c4ec59026467e3b76a92c61c18f0792 (diff) |
Added null checks in "fini"
Signed-off-by: Harshavardhana <harsha@gluster.com>
Signed-off-by: Vijay Bellur <vijay@dev.gluster.com>
BUG: 364 (Segfault in io-cache)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=364
Diffstat (limited to 'xlators/performance')
-rw-r--r-- | xlators/performance/io-cache/src/io-cache.c | 3 | ||||
-rw-r--r-- | xlators/performance/read-ahead/src/read-ahead.c | 3 |
2 files changed, 6 insertions, 0 deletions
diff --git a/xlators/performance/io-cache/src/io-cache.c b/xlators/performance/io-cache/src/io-cache.c index 60dda3e35a4..f01d8e3bb81 100644 --- a/xlators/performance/io-cache/src/io-cache.c +++ b/xlators/performance/io-cache/src/io-cache.c @@ -1433,6 +1433,9 @@ fini (xlator_t *this) { ioc_table_t *table = this->private; + if (table == NULL) + return; + pthread_mutex_destroy (&table->table_lock); FREE (table); diff --git a/xlators/performance/read-ahead/src/read-ahead.c b/xlators/performance/read-ahead/src/read-ahead.c index da9715c8413..8e140d82147 100644 --- a/xlators/performance/read-ahead/src/read-ahead.c +++ b/xlators/performance/read-ahead/src/read-ahead.c @@ -921,6 +921,9 @@ fini (xlator_t *this) { ra_conf_t *conf = this->private; + if (conf == NULL) + return; + pthread_mutex_destroy (&conf->conf_lock); FREE (conf); |