Thread (51 messages) 51 messages, 6 authors, 2021-05-31

Re: [PATCH v7 05/12] virtio_scsi: Add validation for residual bytes from response

From: Jason Wang <jasowang@redhat.com>
Date: 2021-05-26 02:41:57
Also in: kvm, linux-fsdevel, linux-iommu, lkml, virtualization

在 2021/5/17 下午5:55, Xie Yongji 写道:
quoted hunk ↗ jump to hunk
This ensures that the residual bytes in response (might come
from an untrusted device) will not exceed the data buffer length.

Signed-off-by: Xie Yongji <redacted>
---
  drivers/scsi/virtio_scsi.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c
index efcaf0674c8d..ad7d8cecec32 100644
--- a/drivers/scsi/virtio_scsi.c
+++ b/drivers/scsi/virtio_scsi.c
@@ -97,7 +97,7 @@ static inline struct Scsi_Host *virtio_scsi_host(struct virtio_device *vdev)
  static void virtscsi_compute_resid(struct scsi_cmnd *sc, u32 resid)
  {
  	if (resid)
-		scsi_set_resid(sc, resid);
+		scsi_set_resid(sc, min(resid, scsi_bufflen(sc)));
  }
  
  /*

Acked-by: Jason Wang <jasowang@redhat.com>

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help