Commit 86428134 authored by Ziye Yang's avatar Ziye Yang Committed by Daniel Verkamp
Browse files

lib/iscsi: add a check in spdk_iscsi_conn_sock_cb 



Since we have two pollers for each iSCSI polling group,
so it could be possible that the nop_poller to set
iSCSI connection in existed or existing state.
So add a check here, which means the incoming data
is too late.

Change-Id: I94e5090b005c35c9eb5d7eedbd7acdae67327f94
Signed-off-by: default avatarZiye Yang <optimistyzy@gmail.com>
Reviewed-on: https://review.gerrithub.io/402039


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent 068b6fb3

lib/iscsi/conn.c

+5 −0
Original line number Diff line number Diff line
@@ -1150,6 +1150,11 @@ spdk_iscsi_conn_sock_cb(void *arg, struct spdk_sock_group *group, struct spdk_so 


	assert(conn != NULL);



	if ((conn->state == ISCSI_CONN_STATE_EXITED) ||

	    (conn->state == ISCSI_CONN_STATE_EXITING)) {

		return;

	}



	/* Handle incoming PDUs */

	rc = spdk_iscsi_conn_handle_incoming_pdus(conn);

	if (rc < 0) {