Changeset 35760 in vbox

Timestamp:

Jan 28, 2011 1:09:10 PM (14 years ago)

Author:

vboxsync

Message:

Main/DisplayImpl: free the screenshot buffer on error for secondary displays.

File:

• trunk/src/VBox/Main/src-client/DisplayImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-client/DisplayImpl.cpp

@@ -2246 +2246 @@
                     *pu32Height = height;
                 }
+                else
+                {
+                    RTMemFree(pu8Data);
+                }
             }
         }
@@ -2315 +2319 @@
         }
 
-        /* This can be called from any thread. */
-        pDrv->pUpPort->pfnFreeScreenshot (pDrv->pUpPort, pu8Data);
+        if (aScreenId == VBOX_VIDEO_PRIMARY_SCREEN)
+        {
+            /* This can be called from any thread. */
+            pDrv->pUpPort->pfnFreeScreenshot (pDrv->pUpPort, pu8Data);
+        }
+        else
+        {
+            RTMemFree(pu8Data);
+        }
     }