Commit b31d31b2 authored by Ben Walker's avatar Ben Walker Committed by Jim Harris
Browse files

vhost: Check for null pointers in vhost_blk 



This fixes two flagged Klocwork issues.

Change-Id: I98ac136995ebcdc89aa94c76fec095573e102674
Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-on: https://review.gerrithub.io/408237


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent 3ac5d089

lib/vhost/vhost_blk.c

+12 −2
Original line number Diff line number Diff line
@@ -598,7 +598,12 @@ static void 
spdk_vhost_blk_dump_info_json(struct spdk_vhost_dev *vdev, struct spdk_json_write_ctx *w)

{

	struct spdk_bdev *bdev = spdk_vhost_blk_get_dev(vdev);

	struct spdk_vhost_blk_dev *bvdev = to_blk_dev(vdev);

	struct spdk_vhost_blk_dev *bvdev;



	bvdev = to_blk_dev(vdev);

	if (bvdev == NULL) {

		return;

	}



	assert(bvdev != NULL);

	spdk_json_write_name(w, "block");
@@ -620,7 +625,12 @@ spdk_vhost_blk_dump_info_json(struct spdk_vhost_dev *vdev, struct spdk_json_writ 
static void

spdk_vhost_blk_write_config_json(struct spdk_vhost_dev *vdev, struct spdk_json_write_ctx *w)

{

	struct spdk_vhost_blk_dev *bvdev = to_blk_dev(vdev);

	struct spdk_vhost_blk_dev *bvdev;



	bvdev = to_blk_dev(vdev);

	if (bvdev == NULL) {

		return;

	}



	if (!bvdev->bdev) {

		return;