Commit e01a3f1c authored Jun 13, 2021 by Zhiqiang Liu Committed by Jim Harris Jun 15, 2021

nvmf: check return value of strdup in spdk_nvmf_subsystem_disconnect_host()



In spdk_nvmf_subsystem_disconnect_host(), we should check
whether strdup() return NULL.

Signed-off-by: Zhiqiang Liu <liuzhiqiang26@huawei.com>
Change-Id: I29cb6b2499ecd2a2367001c0d21ac95da4e10e20
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8304


Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>

parent 1e660daa

lib/nvmf/subsystem.c

+6 −1

Original line number	Diff line number	Diff line
		@@ -848,8 +848,13 @@ spdk_nvmf_subsystem_disconnect_host(struct spdk_nvmf_subsystem *subsystem,
		return -ENOMEM;
		}

		ctx->subsystem = subsystem;
		ctx->hostnqn = strdup(hostnqn);
		if (ctx->hostnqn == NULL) {
		free(ctx);
		return -ENOMEM;
		}

		ctx->subsystem = subsystem;
		ctx->cb_fn = cb_fn;
		ctx->cb_arg = cb_arg;