Changeset 64810 in vbox

Timestamp:

Dec 8, 2016 1:57:47 PM (8 years ago)

Author:

vboxsync

Message:

Devices/Virtio: vqueuePut - don't call PDMDevHlpPCIPhysWrite with zero
cbSegLen, it asserts that down the call chain.

File:

• trunk/src/VBox/Devices/VirtIO/Virtio.cpp (modified) (1 diff)

trunk/src/VBox/Devices/VirtIO/Virtio.cpp

@@ -230 +230 @@
         if (pElem->aSegsIn[i].pv)
         {
-            Log2(("%s vqueuePut: %s used_idx=%u seg=%u addr=%p pv=%p cb=%u acb=%u\n", INSTANCE(pState),
-                  QUEUENAME(pState, pQueue), pQueue->uNextUsedIndex, i, pElem->aSegsIn[i].addr, pElem->aSegsIn[i].pv, pElem->aSegsIn[i].cb, cbSegLen));
-            PDMDevHlpPCIPhysWrite(pState->CTX_SUFF(pDevIns), pElem->aSegsIn[i].addr + cbReserved,
-                                  pElem->aSegsIn[i].pv, cbSegLen);
+            if (cbSegLen > 0)
+            {
+                Log2(("%s vqueuePut: %s used_idx=%u seg=%u addr=%p pv=%p cb=%u acb=%u\n", INSTANCE(pState),
+                      QUEUENAME(pState, pQueue), pQueue->uNextUsedIndex, i, pElem->aSegsIn[i].addr, pElem->aSegsIn[i].pv, pElem->aSegsIn[i].cb, cbSegLen));
+                PDMDevHlpPCIPhysWrite(pState->CTX_SUFF(pDevIns), pElem->aSegsIn[i].addr + cbReserved,
+                                      pElem->aSegsIn[i].pv, cbSegLen);
+            }
             cbReserved = 0;
         }