Changeset 29743 in vbox for trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp

Timestamp:

May 21, 2010 4:00:02 PM (15 years ago)

Author:

vboxsync

Message:

HGCM: fixed memory leak (xTracker 4961).

File:

• trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp (modified) (1 diff)

trunk/src/VBox/Devices/VMMDev/VMMDevHGCM.cpp

@@ -1066 +1066 @@
         rc = pVMMDevState->pHGCMDrv->pfnCall (pVMMDevState->pHGCMDrv, pCmd, pHGCMCall->u32ClientID,
                                               pHGCMCall->u32Function, cParms, pCmd->paHostParms);
-    }
-    else
+
+        if (RT_FAILURE (rc))
+        {
+            Log(("vmmdevHGCMCall: pfnCall failed rc = %Rrc\n", rc));
+            vmmdevHGCMRemoveCommand (pVMMDevState, pCmd);
+        }
+    }
+
+    if (RT_FAILURE (rc))
     {
         if (pCmd->paLinPtrs)