Commit 9e509a7d authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Tomasz Zawadzki
Browse files

nvmf/tcp: adjust recv state assertion in nvmf_tcp_sock_process() 



The PDU can be NULL in the QUIESCING and ERROR states too, e.g. after
receiving a H2CTermReq.

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


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent bd407444

lib/nvmf/tcp.c

+5 −1
Original line number Diff line number Diff line
@@ -2304,7 +2304,11 @@ nvmf_tcp_sock_process(struct spdk_nvmf_tcp_qpair *tqpair) 
		SPDK_DEBUGLOG(nvmf_tcp, "tqpair(%p) recv pdu entering state %d\n", tqpair, prev_state);



		pdu = tqpair->pdu_in_progress;

		assert(pdu || tqpair->recv_state == NVME_TCP_PDU_RECV_STATE_AWAIT_PDU_READY);

		assert(pdu != NULL ||

		       tqpair->recv_state == NVME_TCP_PDU_RECV_STATE_AWAIT_PDU_READY ||

		       tqpair->recv_state == NVME_TCP_PDU_RECV_STATE_QUIESCING ||

		       tqpair->recv_state == NVME_TCP_PDU_RECV_STATE_ERROR);



		switch (tqpair->recv_state) {

		/* Wait for the common header  */

		case NVME_TCP_PDU_RECV_STATE_AWAIT_PDU_READY: