Changeset 37384 in vbox for trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/xpdm

Timestamp:

Jun 8, 2011 3:09:14 PM (14 years ago)

Author:

vboxsync

Message:

1 million rectangles ought to be enough for anybody...

File:

• trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/xpdm/VBoxMPDriver.cpp (modified) (8 diffs)

trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/xpdm/VBoxMPDriver.cpp

@@ -26 +26 @@
 
 /* Resource list */
-VIDEO_ACCESS_RANGE  VBoxLegacyVGAResourceList[] = 
+VIDEO_ACCESS_RANGE  VBoxLegacyVGAResourceList[] =
 {
     { 0x000003B0, 0x00000000, 0x0000000C, 1, 1, 1, 0 }, /* VGA regs (0x3B0-0x3BB) */
@@ -106 +106 @@
             ULONG vendorId = 0x80EE;
             ULONG deviceId = 0xBEEF;
-            rc = VideoPortGetAccessRanges(pExt, 0, NULL, RT_ELEMENTS(tmpRanges), tmpRanges, 
+            rc = VideoPortGetAccessRanges(pExt, 0, NULL, RT_ELEMENTS(tmpRanges), tmpRanges,
                                           &vendorId, &deviceId, &slot);
         }
@@ -142 +142 @@
     }
 
-    /* @todo: pretend success to make the driver work. */
+    /** @todo pretend success to make the driver work. */
     rc = NO_ERROR;
 
@@ -281 +281 @@
         }
 
-        /* Returns count of supported video modes and structure size in bytes, 
+        /* Returns count of supported video modes and structure size in bytes,
          * used to allocate buffer for the following IOCTL_VIDEO_QUERY_AVAIL_MODES call.
          */
@@ -350 +350 @@
             STARTIO_IN(VIDEO_POINTER_POSITION, pPos);
 
-            /*@todo: set pointer position*/
+            /** @todo set pointer position*/
             bResult = VBoxMPEnablePointer(pExt, TRUE, pStatus);
             break;
@@ -439 +439 @@
             uint32_t cRects = RequestPacket->InputBufferLength/sizeof(RTRECT);
             /*Sanity check*/
-            if (RequestPacket->InputBufferLength != cRects*sizeof(RTRECT))
+            if (   cRects > _1M
+                || RequestPacket->InputBufferLength != cRects * sizeof(RTRECT))
             {
                 pStatus->Status = ERROR_INSUFFICIENT_BUFFER;
@@ -487 +488 @@
         case IOCTL_VIDEO_HGSMI_HANDLER_DISABLE:
         {
-            /* @todo: not implemented */
+            /** @todo not implemented */
             break;
         }
@@ -519 +520 @@
 }
 
-/* Called to set out hardware into desired power state, not supported at the moment. 
+/* Called to set out hardware into desired power state, not supported at the moment.
  * Required to return NO_ERROR always.
  */