Changeset 35759 in vbox for trunk/src/VBox

Timestamp:

Jan 28, 2011 1:06:58 PM (14 years ago)

Author:

vboxsync

Message:

DevVGA: verify the size of the screenshot.

File:

• trunk/src/VBox/Devices/Graphics/DevVGA.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Graphics/DevVGA.cpp

@@ -4605 +4605 @@
     size_t cbRequired = pThis->last_scr_width * 4 * pThis->last_scr_height;
 
-    if (cbRequired)
+    /* The size can't be zero or greater than the size of the VRAM.
+     * Inconsistent VGA device state can cause the incorrect size values.
+     */
+    if (cbRequired && cbRequired <= pThis->vram_size)
     {
         uint8_t *pu8Data = (uint8_t *)RTMemAlloc(cbRequired);