From cfd838c27cc88acd431549f2b8ed515c9ce3e55f Mon Sep 17 00:00:00 2001 From: Susant Palai Date: Mon, 29 Apr 2019 11:36:16 +0530 Subject: cloudsync/plugin: coverity fixes CID 1401087: Null pointer dereferences (REVERSE_INULL) CID 1401088: Null pointer dereferences (FORWARD_NULL) Change-Id: I71bf67af80e1b22bcd2eb997b01a1a5ef0b4d80b Updates: bz#789278 Signed-off-by: Susant Palai --- .../cloudsync/src/cloudsync-plugins/src/cvlt/src/libcvlt.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'xlators/features/cloudsync/src/cloudsync-plugins') diff --git a/xlators/features/cloudsync/src/cloudsync-plugins/src/cvlt/src/libcvlt.c b/xlators/features/cloudsync/src/cloudsync-plugins/src/cvlt/src/libcvlt.c index e827882f68d..5243c8bd492 100644 --- a/xlators/features/cloudsync/src/cloudsync-plugins/src/cvlt/src/libcvlt.c +++ b/xlators/features/cloudsync/src/cloudsync-plugins/src/cvlt/src/libcvlt.c @@ -264,10 +264,12 @@ cvlt_init_xlator(xlator_t *this, archive_t *arch, int num_req, int num_iatt) return ret; err: - cvlt_free_resources(arch); + if (arch) { + cvlt_free_resources(arch); - if (locked) { - UNLOCK(&(arch->lock)); + if (locked) { + UNLOCK(&(arch->lock)); + } } return ret; @@ -445,9 +447,7 @@ out: STACK_UNWIND_STRICT(readv, frame, op_ret, op_errno, &iov, 1, &postbuf, req->iobref, local->xattr_rsp); - if (req) { - cvlt_free_req(priv, req); - } + cvlt_free_req(priv, req); return; } -- cgit