From e8be2f64d789cfea1f0ddfd77c4c70a225d7666b Mon Sep 17 00:00:00 2001 From: Ziye Yang Date: Mon, 12 Sep 2016 15:31:17 +0800 Subject: [PATCH] nvmf: fix the error handling in spdk_nvmf_session_connect We need to free the session resource, if there is error for creating a new session Change-Id: I7c4f3e779e0b30e213e02b8676d93bd2fe9bf851 Signed-off-by: Ziye Yang --- lib/nvmf/session.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/nvmf/session.c b/lib/nvmf/session.c index 9437fe240..f34bb8b07 100644 --- a/lib/nvmf/session.c +++ b/lib/nvmf/session.c @@ -264,6 +264,9 @@ spdk_nvmf_session_connect(struct spdk_nvmf_conn *conn, session->max_connections_allowed = g_nvmf_tgt.max_queues_per_session; if (conn->transport->session_init(session, conn)) { rsp->status.sc = SPDK_NVME_SC_INTERNAL_DEVICE_ERROR; + conn->transport->session_fini(session); + free(session); + subsystem->session = NULL; return; }