In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
nvme-tcp: reject a read that transferred too few bytes<\/p>\n
nvme_tcp_recv_data() completes a request once the current C2HData PDU
\nhas been consumed. Nothing compares the total bytes received against
\nthe length the command asked for: struct nvme_tcp_request has no
\nreceive-side counter, queue->data_remaining is per queue, and
\nblk_mq_end_request() completes for blk_rq_bytes(rq) unconditionally
\nwith no residual concept anywhere above.<\/p>\n
A controller can therefore answer a 4096-byte read with 512 bytes and
\nhave it reported as a complete read; user space then gets 4096 bytes of
\nwhich 3584 are whatever was already in the page. I reproduced that with
\na test target.<\/p>\n
Count the bytes received and refuse to complete a successful read whose
\ncount does not match, at the two NVME_TCP_F_DATA_SUCCESS paths and in
\nnvme_tcp_process_nvme_cqe(). The success test shifts req->status right
\nby one, because the driver keeps the wire value there and shifts it on
\ncompletion, so the check must see what the completion path will see.
\nOnly REQ_OP_READ is checked, because there the length comes from the
\nsectors the request covers; a passthrough command is built by its
\nsubmitter, which picks both command and buffer, so the kernel has
\nnothing to compare against.<\/p>\n","protected":false},"excerpt":{"rendered":"
In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
nvme-tcp: reject a read that transferred too few bytes<\/p>\n
nvme_tcp_recv_data() completes a request once the current C2HData PDU
\nhas b<\/p>\n","protected":false},"template":"","meta":[],"categories":[],"tags":[],"class_list":["post-21402","vuln","type-vuln","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln\/21402","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln"}],"about":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/types\/vuln"}],"wp:attachment":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=21402"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=21402"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=21402"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}