diff options
| author | Will Deacon <[email protected]> | 2025-07-17 09:01:10 +0000 |
|---|---|---|
| committer | Michael S. Tsirkin <[email protected]> | 2025-08-01 13:11:09 +0000 |
| commit | 87dbae5e36613a6020f3d64a2eaeac0a1e0e6dc6 (patch) | |
| tree | 3a34dc96d0548bb5889b29c901d792f77afacae1 /drivers/pci/controller/dwc/pcie-qcom-common.c | |
| parent | vsock/virtio: Validate length in packet header before skb_put() (diff) | |
| download | kernel-87dbae5e36613a6020f3d64a2eaeac0a1e0e6dc6.tar.gz kernel-87dbae5e36613a6020f3d64a2eaeac0a1e0e6dc6.zip | |
vsock/virtio: Move length check to callers of virtio_vsock_skb_rx_put()
virtio_vsock_skb_rx_put() only calls skb_put() if the length in the
packet header is not zero even though skb_put() handles this case
gracefully.
Remove the functionally redundant check from virtio_vsock_skb_rx_put()
and, on the assumption that this is a worthwhile optimisation for
handling credit messages, augment the existing length checks in
virtio_transport_rx_work() to elide the call for zero-length payloads.
Since the callers all have the length, extend virtio_vsock_skb_rx_put()
to take it as an additional parameter rather than fish it back out of
the packet header.
Note that the vhost code already has similar logic in
vhost_vsock_alloc_skb().
Reviewed-by: Stefano Garzarella <[email protected]>
Signed-off-by: Will Deacon <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Michael S. Tsirkin <[email protected]>
Diffstat (limited to 'drivers/pci/controller/dwc/pcie-qcom-common.c')
0 files changed, 0 insertions, 0 deletions
