[PATCH] nvmet-tcp: Fix the H2C expected PDU len calculation
Sagi Grimberg
sagi at grimberg.me
Thu Jan 4 05:10:15 PST 2024
> The nvmet_tcp_handle_h2c_data_pdu() function should take into
> consideration the possibility that the header digest and/or the data
> digests are enabled when calculating the expected PDU length before
> comparing it to the value stored in cmd->pdu_len.
>
> Fixes: efa56305908b ("nvmet-tcp: Fix a kernel panic when host sends an invalid H2C PDU length")
> Signed-off-by: Maurizio Lombardi <mlombard at redhat.com>
> ---
> drivers/nvme/target/tcp.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/nvme/target/tcp.c b/drivers/nvme/target/tcp.c
> index 3569c1255c5e..717ea1e5f4c3 100644
> --- a/drivers/nvme/target/tcp.c
> +++ b/drivers/nvme/target/tcp.c
> @@ -979,7 +979,7 @@ static int nvmet_tcp_handle_h2c_data_pdu(struct nvmet_tcp_queue *queue)
> {
> struct nvme_tcp_data_pdu *data = &queue->pdu.data;
> struct nvmet_tcp_cmd *cmd;
> - unsigned int plen;
> + unsigned int expected_len;
>
> if (likely(queue->nr_cmds)) {
> if (unlikely(data->ttag >= queue->nr_cmds)) {
> @@ -999,9 +999,12 @@ static int nvmet_tcp_handle_h2c_data_pdu(struct nvmet_tcp_queue *queue)
> goto err_proto;
> }
>
> - plen = le32_to_cpu(data->hdr.plen);
> + expected_len = le32_to_cpu(data->hdr.plen);
> + expected_len -= nvmet_tcp_hdgst_len(queue);
> + expected_len -= nvmet_tcp_ddgst_len(queue);
> + expected_len -= sizeof(*data);
I think we should call it exp_data_len.
Why not do:
exp_data_len = le32_to_cpu(data->hdr.plen) -
nvmet_tcp_hdgst_len(queue) -
nvmet_tcp_ddgst_len(queue) -
sizeof(*data);
> cmd->pdu_len = le32_to_cpu(data->data_length);
> - if (unlikely(cmd->pdu_len != (plen - sizeof(*data)) ||
> + if (unlikely(cmd->pdu_len != expected_len ||
> cmd->pdu_len == 0 ||
> cmd->pdu_len > NVMET_TCP_MAXH2CDATA)) {
> pr_err("H2CData PDU len %u is invalid\n", cmd->pdu_len);
More information about the Linux-nvme
mailing list