Commit 29773365 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Jim Harris
Browse files

nvmf: don't touch subsystem->flags.allow_any_host directly 



This flag is protected with a mutex, so a getter function should be used
to retrieve it.

Signed-off-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I84dc0b4f926c805bc3111d43b89056500367449d
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22897


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarBen Walker <ben@nvidia.com>
parent 35948d8f

lib/nvmf/nvmf_rpc.c

+1 −1
Original line number Diff line number Diff line
@@ -922,7 +922,7 @@ rpc_nvmf_subsystem_add_listener(struct spdk_jsonrpc_request *request, 
	ctx->op = NVMF_RPC_LISTEN_ADD;

	spdk_nvmf_listen_opts_init(&ctx->opts, sizeof(ctx->opts));

	ctx->opts.transport_specific = params;

	if (subsystem->flags.allow_any_host == 1 && ctx->listener_opts.secure_channel == true) {

	if (spdk_nvmf_subsystem_get_allow_any_host(subsystem) && ctx->listener_opts.secure_channel) {

		spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

						 "Cannot establish secure channel, when 'allow_any_host' is set");

		nvmf_rpc_listener_ctx_free(ctx);