Changeset 81322 in vbox for trunk/src/VBox/GuestHost/SharedClipboard

Timestamp:

Oct 17, 2019 3:09:15 PM (6 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

134032

Message:

Shared Clipboard/Transfers: Added limiting of object and list handles (4096 by default).

File:

• trunk/src/VBox/GuestHost/SharedClipboard/clipboard-transfers.cpp (modified) (7 diffs)

trunk/src/VBox/GuestHost/SharedClipboard/clipboard-transfers.cpp

@@ -792 +792 @@
     AssertMsgReturn(pTransfer->pszPathRootAbs, ("Transfer has no root path set\n"), VERR_INVALID_PARAMETER);
     AssertMsgReturn(pOpenCreateParms->pszPath, ("No path in open/create params set\n"), VERR_INVALID_PARAMETER);
+
+    if (pTransfer->cObjHandles == pTransfer->cMaxObjHandles)
+        return VERR_SHCLPB_MAX_OBJECTS_REACHED;
 
     LogFlowFunc(("pszPath=%s, fCreate=0x%x\n", pOpenCreateParms->pszPath, pOpenCreateParms->fCreate));
@@ -834 +837 @@
 
                 RTListAppend(&pTransfer->lstObj, &pInfo->Node);
+                pTransfer->cObjHandles++;
+
+                LogFlowFunc(("cObjHandles=%RU32\n", pTransfer->cObjHandles));
 
                 *phObj = pInfo->hObj;
@@ -914 +920 @@
 
             RTListNodeRemove(&pInfo->Node);
+
+            Assert(pTransfer->cObjHandles);
+            pTransfer->cObjHandles--;
 
             ShClTransferObjHandleInfoDestroy(pInfo);
@@ -1260 +1269 @@
         pTransfer->State.enmStatus = SHCLTRANSFERSTATUS_INITIALIZED; /* Now we're ready to run. */
 
+        pTransfer->cMaxListHandles = _4K; /** @todo Make this dynamic. */
+        pTransfer->cMaxObjHandles  = _4K; /** @todo Ditto. */
+
         if (pTransfer->Callbacks.pfnTransferInitialize)
         {
@@ -1383 +1395 @@
 
     int rc;
+
+    if (pTransfer->cListHandles == pTransfer->cMaxListHandles)
+        return VERR_SHCLPB_MAX_LISTS_REACHED;
 
     if (pTransfer->State.enmSource == SHCLSOURCE_LOCAL)
@@ -1439 +1454 @@
                             {
                                 RTListAppend(&pTransfer->lstList, &pInfo->Node);
-
-                                LogFlowFunc(("pszPathLocalAbs=%s, hList=%RU64\n", pInfo->pszPathLocalAbs, pInfo->hList));
+                                pTransfer->cListHandles++;
+
+                                LogFlowFunc(("pszPathLocalAbs=%s, hList=%RU64, cListHandles=%RU32\n",
+                                             pInfo->pszPathLocalAbs, pInfo->hList, pTransfer->cListHandles));
 
                                 *phList = pInfo->hList;
@@ -1533 +1550 @@
 
             RTListNodeRemove(&pInfo->Node);
+
+            Assert(pTransfer->cListHandles);
+            pTransfer->cListHandles--;
 
             RTMemFree(pInfo);