Changeset 86027 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Video

Timestamp:

Sep 3, 2020 5:45:50 PM (4 years ago)

Author:

vboxsync

Message:

WDDM: fix for VBoxVGA adapter

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPLegacy.cpp (modified) (1 diff)

trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPLegacy.cpp

@@ -649 +649 @@
     PVBOXWDDM_ALLOCATION pDstAlloc = pBlt->Blt.DstAlloc.pAlloc;
 //    PVBOXWDDM_ALLOCATION pSrcAlloc = pBlt->Blt.SrcAlloc.pAlloc;
-    BOOLEAN fVRAMUpdated = FALSE;
     {
         /* the allocations contain a real data in VRAM, do blitting */
         vboxVdmaGgDmaBlt(pDevExt, &pBlt->Blt);
-        fVRAMUpdated = TRUE;
-
-        VBOXWDDM_SOURCE *pSource = &pDevExt->aSources[pDstAlloc->AllocData.SurfDesc.VidPnSourceId];
-        Assert(pDstAlloc->AllocData.SurfDesc.VidPnSourceId < VBOX_VIDEO_MAX_SCREENS);
-        Assert(pSource->pPrimaryAllocation == pDstAlloc);
-        vboxVdmaBltDirtyRectsUpdate(pDevExt, pSource, pBlt->Blt.DstRects.UpdateRects.cRects, pBlt->Blt.DstRects.UpdateRects.aRects);
+
+        if (pDstAlloc->bAssigned && pDstAlloc->bVisible)
+        {
+            /* Only for visible framebuffer allocations. */
+            VBOXWDDM_SOURCE *pSource = &pDevExt->aSources[pDstAlloc->AllocData.SurfDesc.VidPnSourceId];
+            /* Assert but otherwise ignore wrong allocations. */
+            AssertReturn(pDstAlloc->AllocData.SurfDesc.VidPnSourceId < VBOX_VIDEO_MAX_SCREENS, STATUS_SUCCESS);
+            AssertReturn(pSource->pPrimaryAllocation == pDstAlloc, STATUS_SUCCESS);
+            vboxVdmaBltDirtyRectsUpdate(pDevExt, pSource, pBlt->Blt.DstRects.UpdateRects.cRects, pBlt->Blt.DstRects.UpdateRects.aRects);
+        }
     }
     return Status;