Changeset 106241 in vbox

Timestamp:

Oct 8, 2024 3:20:56 PM (7 weeks ago)

Author:

vboxsync

Message:

DrvSCSI: When completing SCSI commands, copy only the amount of data provided by device rather than entire buffer size (see bugref:10774).

File:

• trunk/src/VBox/Devices/Storage/DrvSCSI.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/Storage/DrvSCSI.cpp

@@ -1037 +1037 @@
 
     /* Sync buffers. */
+    size_t cbCopyIn = RT_MIN(pReq->cbBuf, cbXfer);
     if (   RT_SUCCESS(rcReq)
-        && pReq->cbBuf
+        && cbCopyIn
         && (   pReq->enmXferDir == PDMMEDIAEXIOREQSCSITXDIR_UNKNOWN
             || pReq->enmXferDir == PDMMEDIAEXIOREQSCSITXDIR_FROM_DEVICE))
@@ -1045 +1046 @@
         RTSgBufInit(&SgBuf, &pReq->Seg, 1);
         int rcCopy = pThis->pDevMediaExPort->pfnIoReqCopyFromBuf(pThis->pDevMediaExPort, (PDMMEDIAEXIOREQ)pReq,
-                                                                 &pReq->abAlloc[0], 0, &SgBuf, pReq->cbBuf);
+                                                                 &pReq->abAlloc[0], 0, &SgBuf, cbCopyIn);
         if (RT_FAILURE(rcCopy))
             rcReq = rcCopy;