Commit 1dc9e702 authored by Darek Stojaczyk's avatar Darek Stojaczyk Committed by Jim Harris
Browse files

vhost/rpc: remove unnecessary if in the add_vhost_scsi_lun RPC 



We explicitly checked for one of the strings in the
parsed RPC request even though it's required for the
entire request to parse successfully. The extra check
is now removed.

Change-Id: I19c446786e4ac88b88f14e18dc5258f31b1a87f1
Signed-off-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/c/443317


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarPawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 244a619f

lib/vhost/vhost_rpc.c

+0 −6
Original line number Diff line number Diff line
@@ -137,12 +137,6 @@ spdk_rpc_add_vhost_scsi_lun(struct spdk_jsonrpc_request *request, 
		goto invalid;

	}



	if (req.ctrlr == NULL) {

		SPDK_ERRLOG("No controller name\n");

		rc = -EINVAL;

		goto invalid;

	}



	spdk_vhost_lock();

	vdev = spdk_vhost_dev_find(req.ctrlr);

	if (vdev == NULL) {