From ff5170faa131b201bf8170601dbeae18b400542f Mon Sep 17 00:00:00 2001 From: Harshavardhana Ranganath Date: Thu, 17 Dec 2009 15:41:21 +0000 Subject: Added null checks in "fini" Signed-off-by: Harshavardhana Signed-off-by: Vijay Bellur BUG: 364 (Segfault in io-cache) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=364 --- xlators/performance/io-cache/src/io-cache.c | 3 +++ xlators/performance/read-ahead/src/read-ahead.c | 3 +++ 2 files changed, 6 insertions(+) (limited to 'xlators') diff --git a/xlators/performance/io-cache/src/io-cache.c b/xlators/performance/io-cache/src/io-cache.c index 60dda3e35..f01d8e3bb 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 da9715c84..8e140d821 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); -- cgit