Changeset 82534 in vbox for trunk/src/VBox/HostServices

Timestamp:

Dec 10, 2019 10:41:32 AM (5 years ago)

Author:

vboxsync

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

135398

Message:

VBoxSharedClipboardSvc: Fixed unresolved ShClEventIDGenerate in ShClSvcImplReadData. bugref:9437

Location:

trunk/src/VBox/HostServices/SharedClipboard

Files:

• VBoxSharedClipboardSvc-x11.cpp (modified) (4 diffs)
• VBoxSharedClipboardSvc.cpp (modified) (1 diff)

trunk/src/VBox/HostServices/SharedClipboard/VBoxSharedClipboardSvc-x11.cpp

@@ -189 +189 @@
     if (pReq)
     {
-        const SHCLEVENTID uEvent = ShClEventIDGenerate(&pClient->EventSrc);
-
         pReq->pv        = pData->pvData;
         pReq->cb        = pData->cbData;
         pReq->pcbActual = pcbActual;
-        pReq->uEvent    = uEvent;
-
-        rc = ShClEventRegister(&pClient->EventSrc, uEvent);
-        if (RT_SUCCESS(rc))
+        const SHCLEVENTID idEvent = ShClEventIdGenerateAndRegister(&pClient->EventSrc);
+        pReq->uEvent    = idEvent;
+        if (idEvent)
         {
             rc = ShClX11ReadDataFromX11(&pClient->State.pCtx->X11, pData->uFormat, pReq);
@@ -203 +200 @@
             {
                 PSHCLEVENTPAYLOAD pPayload;
-                rc = ShClEventWait(&pClient->EventSrc, uEvent, 30 * 1000, &pPayload);
+                rc = ShClEventWait(&pClient->EventSrc, idEvent, 30 * 1000, &pPayload);
                 if (RT_SUCCESS(rc))
                 {
@@ -213 +210 @@
             }
 
-            ShClEventUnregister(&pClient->EventSrc, uEvent);
+            ShClEventUnregister(&pClient->EventSrc, idEvent);
+        }
+        else
+        {
+            RTMemFree(pReq);
+            rc = VERR_GENERAL_FAILURE;
         }
     }
@@ -295 +297 @@
         }
 
+        RTCritSectEnter(&pCtx->pClient->CritSect);
         rc2 = ShClEventSignal(&pCtx->pClient->EventSrc, pReq->uEvent, pPayload);
         AssertRC(rc2);
+        RTCritSectLeave(&pCtx->pClient->CritSect);
     }
 

trunk/src/VBox/HostServices/SharedClipboard/VBoxSharedClipboardSvc.cpp

@@ -1236 +1236 @@
     if (RT_SUCCESS(rc))
     {
+        RTCritSectEnter(&pClient->CritSect);
         rc = ShClEventSignal(&pClient->EventSrc, uEvent, pPayload);
+        RTCritSectLeave(&pClient->CritSect);
         if (RT_FAILURE(rc))
             ShClPayloadFree(pPayload);