Commit 6bba6fe9 authored by Tomasz Zawadzki's avatar Tomasz Zawadzki Committed by Jim Harris
Browse files

lvol: check spdk_lvol for NULL before dereferencing it 



Change-Id: I0f01e2ccde06fa0a5a4ff31f106381bbf3ae6020
Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-on: https://review.gerrithub.io/433858


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent f425b985

lib/lvol/lvol.c

+4 −4
Original line number Diff line number Diff line
@@ -1425,8 +1425,7 @@ void 
spdk_lvol_inflate(struct spdk_lvol *lvol, spdk_lvol_op_complete cb_fn, void *cb_arg)

{

	struct spdk_lvol_req *req;

	struct spdk_blob *blob = lvol->blob;

	spdk_blob_id blob_id = spdk_blob_get_id(blob);

	spdk_blob_id blob_id;



	assert(cb_fn != NULL);
@@ -1453,6 +1452,7 @@ spdk_lvol_inflate(struct spdk_lvol *lvol, spdk_lvol_op_complete cb_fn, void *cb_ 
		return;

	}



	blob_id = spdk_blob_get_id(lvol->blob);

	spdk_bs_inflate_blob(lvol->lvol_store->blobstore, req->channel, blob_id, _spdk_lvol_inflate_cb,

			     req);

}
@@ -1461,8 +1461,7 @@ void 
spdk_lvol_decouple_parent(struct spdk_lvol *lvol, spdk_lvol_op_complete cb_fn, void *cb_arg)

{

	struct spdk_lvol_req *req;

	struct spdk_blob *blob = lvol->blob;

	spdk_blob_id blob_id = spdk_blob_get_id(blob);

	spdk_blob_id blob_id;



	assert(cb_fn != NULL);
@@ -1489,6 +1488,7 @@ spdk_lvol_decouple_parent(struct spdk_lvol *lvol, spdk_lvol_op_complete cb_fn, v 
		return;

	}



	blob_id = spdk_blob_get_id(lvol->blob);

	spdk_bs_blob_decouple_parent(lvol->lvol_store->blobstore, req->channel, blob_id,

				     _spdk_lvol_inflate_cb, req);

}