diff options
| author | Poornima <pgurusid@redhat.com> | 2014-01-30 05:36:42 +0000 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2014-02-10 17:05:20 -0800 |
| commit | fcfd2b973cdb9d798861bca1d30a4a42b819b73f (patch) | |
| tree | 50b73147e4e1a3178f4935b8881c119e7958fed6 | |
| parent | 6b8ed063bb8a551856d0e659fa9fe4f5afa48c19 (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.c | 2 |
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 */ |