summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarshavardhana Ranganath <harsha@gluster.com>2009-12-17 15:41:21 +0000
committerVijay Bellur <vijay@dev.gluster.com>2009-12-18 08:41:29 -0800
commitff5170faa131b201bf8170601dbeae18b400542f (patch)
treed8aa89f66b642036d5ce6ad62a0bdb566e61ac16
parentcfe095763c4ec59026467e3b76a92c61c18f0792 (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
-rw-r--r--xlators/performance/io-cache/src/io-cache.c3
-rw-r--r--xlators/performance/read-ahead/src/read-ahead.c3
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);