summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPoornima <pgurusid@redhat.com>2014-01-30 05:36:42 +0000
committerVijay Bellur <vbellur@redhat.com>2014-02-10 17:05:20 -0800
commitfcfd2b973cdb9d798861bca1d30a4a42b819b73f (patch)
tree50b73147e4e1a3178f4935b8881c119e7958fed6
parent6b8ed063bb8a551856d0e659fa9fe4f5afa48c19 (diff)
performance/io-cache: Fix for the bugs reported by coverity
Change-Id: I24c10d874511a2f24dda2fb84d31f5074da1616f BUG: 789278 Signed-off-by: Poornima <pgurusid@redhat.com> Reviewed-on: http://review.gluster.org/6869 Reviewed-by: Raghavendra G <rgowdapp@redhat.com> Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r--xlators/performance/io-cache/src/page.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/xlators/performance/io-cache/src/page.c b/xlators/performance/io-cache/src/page.c
index 94b8f229bc0..a21b318169c 100644
--- a/xlators/performance/io-cache/src/page.c
+++ b/xlators/performance/io-cache/src/page.c
@@ -315,6 +315,7 @@ __ioc_wait_on_page (ioc_page_t *page, call_frame_t *frame, off_t offset,
local->op_errno = ENOMEM;
gf_log (frame->this->name, GF_LOG_WARNING,
"asked to wait on a NULL page");
+ goto out;
}
waitq = GF_CALLOC (1, sizeof (*waitq), gf_ioc_mt_ioc_waitq_t);
@@ -476,6 +477,7 @@ ioc_fault_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
iobref_unref (page->iobref);
GF_FREE (page->vector);
page->vector = NULL;
+ page->iobref = NULL;
}
/* keep a copy of the page for our cache */