Commit d40f805d authored by Ziye Yang's avatar Ziye Yang Committed by Darek Stojaczyk
Browse files

nvmf: fix the error path for shared data buffer free. 



Since we use aligned buffer, I think that the error handling
path here is not correct, the address is wrong.

Change-Id: I5bcb7f050199496423f861fd6aea65e0fe48c804
Signed-off-by: default avatarZiye Yang <ziye.yang@intel.com>
Reviewed-on: https://review.gerrithub.io/435992


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: default avatarSeth Howell <seth.howell5141@gmail.com>
parent d5256e15

lib/nvmf/rdma.c

+1 −1
Original line number Diff line number Diff line
@@ -1177,7 +1177,7 @@ spdk_nvmf_rdma_request_fill_iovs(struct spdk_nvmf_rdma_transport *rtransport, 
nomem:

	while (i) {

		i--;

		spdk_mempool_put(rtransport->data_buf_pool, rdma_req->req.iov[i].iov_base);

		spdk_mempool_put(rtransport->data_buf_pool, rdma_req->data.buffers[i]);

		rdma_req->req.iov[i].iov_base = NULL;

		rdma_req->req.iov[i].iov_len = 0;

lib/nvmf/tcp.c

+1 −1
Original line number Diff line number Diff line
@@ -2128,7 +2128,7 @@ spdk_nvmf_tcp_req_fill_iovs(struct spdk_nvmf_tcp_transport *ttransport, 
nomem:

	while (i) {

		i--;

		spdk_mempool_put(ttransport->data_buf_pool, tcp_req->req.iov[i].iov_base);

		spdk_mempool_put(ttransport->data_buf_pool, tcp_req->buffers[i]);

		tcp_req->req.iov[i].iov_base = NULL;

		tcp_req->req.iov[i].iov_len = 0;