Changeset 100412 in vbox for trunk/src/VBox/GuestHost

Timestamp:

Jul 6, 2023 2:08:30 PM (20 months ago)

Author:

vboxsync

Message:

Shared Clipboard: Windows host service now uses the same way as the Linux host service wrt transfer handling. ​​​bugref:9437

File:

• trunk/src/VBox/GuestHost/SharedClipboard/ClipboardDataObjectImpl-win.cpp (modified) (12 diffs)

trunk/src/VBox/GuestHost/SharedClipboard/ClipboardDataObjectImpl-win.cpp

@@ -188 +188 @@
     AssertReturnVoid(m_lRefCount == 0);
 
+    /* Make sure to release the transfer. */
+    setTransferLocked(NULL);
+
     int rc = RTCritSectDelete(&m_CritSect);
     AssertRC(rc);
@@ -310 +313 @@
     GlobalFree(hGlobal);
     return VERR_ACCESS_DENIED;
+}
+
+inline int SharedClipboardWinDataObject::lock(void)
+{
+    int rc = RTCritSectEnter(&m_CritSect);
+    AssertRCReturn(rc, rc);
+
+    return rc;
+}
+
+inline int SharedClipboardWinDataObject::unlock(void)
+{
+    int rc = RTCritSectLeave(&m_CritSect);
+    AssertRCReturn(rc, rc);
+
+    return rc;
 }
 
@@ -675 +694 @@
     AssertPtrReturn(pMedium, DV_E_FORMATETC);
 
-    int rc2 = RTCritSectEnter(&m_CritSect);
-    AssertRCReturn(rc2, E_UNEXPECTED);
+    lock();
 
     LogFlowFunc(("lIndex=%RI32, enmStatus=%#x\n", pFormatEtc->lindex, m_enmStatus));
@@ -702 +720 @@
 
                 /* Leave lock while requesting + waiting. */
-                rc2 = RTCritSectLeave(&m_CritSect);
-                AssertRCBreak(rc);
+                unlock();
 
                 /* Start the transfer. */
@@ -715 +732 @@
                  *       Windows Explorer unresponsive (i.e. "ghost window") when waiting for too long. */
                 rc = RTSemEventWait(m_EventStatusChanged, RT_MS_10SEC);
-                AssertRCBreak(rc);
 
                 /* Re-acquire lock. */
-                rc = RTCritSectEnter(&m_CritSect);
-                AssertRCBreak(rc);
+                lock();
+
+                if (RT_FAILURE(rc))
+                {
+                    LogRel(("Shared Clipboard: Waiting for IDataObject status status failed, rc=%Rrc\n", rc));
+                    break;
+                }
 
                 if (m_enmStatus != Running)
@@ -757 +778 @@
 
                             /* Leave lock while waiting. */
-                            rc = RTCritSectLeave(&m_CritSect);
-                            AssertRCReturn(rc, E_UNEXPECTED);
+                            unlock();
 
                             /* Don't block for too long here, as this also will screw other apps running on the OS. */
@@ -765 +785 @@
 
                             /* Re-acquire lock. */
-                            rc = RTCritSectEnter(&m_CritSect);
-                            AssertRCReturn(rc, E_UNEXPECTED);
+                            lock();
                         }
                     }
@@ -847 +866 @@
     }
 
-    rc2 = RTCritSectLeave(&m_CritSect);
-    AssertRCReturn(rc2, E_UNEXPECTED);
+    unlock();
 
     LogFlowFunc(("hr=%Rhrc\n", hr));
@@ -1012 +1030 @@
 
 /**
- * Assigns a transfer object for the data object.
+ * Assigns a transfer object for the data object, internal version.
  *
  * @returns VBox status code.
  * @param   pTransfer           Transfer to assign.
- *                              Must be in started state.
- */
-int SharedClipboardWinDataObject::SetTransfer(PSHCLTRANSFER pTransfer)
-{
-    AssertReturn(m_pTransfer == NULL, VERR_WRONG_ORDER); /* Transfer already set? */
-
-    int rc = RTCritSectEnter(&m_CritSect);
-    if (RT_SUCCESS(rc))
-    {
+ *                              Must be in INITIALIZED state.
+ *                              When set to NULL, the transfer will be released from the object.
+ */
+int SharedClipboardWinDataObject::setTransferLocked(PSHCLTRANSFER pTransfer)
+{
+    LogFlowFunc(("pTransfer=%p\n", pTransfer));
+
+    int rc = VINF_SUCCESS;
+
+    if (pTransfer) /* Set */
+    {
+        Assert(m_pTransfer == NULL); /* Transfer already set? */
+
         if (m_enmStatus == Initialized)
         {
@@ -1039 +1061 @@
         else
             AssertFailedStmt(rc = VERR_WRONG_ORDER);
-
-        RTCritSectLeave(&m_CritSect);
-    }
+    }
+    else /* Unset */
+    {
+        if (m_pTransfer)
+        {
+            ShClTransferRelease(m_pTransfer);
+            m_pTransfer = NULL;
+        }
+    }
+
+    return rc;
+}
+
+/**
+ * Assigns a transfer object for the data object.
+ *
+ * @returns VBox status code.
+ * @param   pTransfer           Transfer to assign.
+ *                              Must be in INITIALIZED state.
+ *                              When set to NULL, the transfer will be released from the object.
+ */
+int SharedClipboardWinDataObject::SetTransfer(PSHCLTRANSFER pTransfer)
+{
+    lock();
+
+    int rc = setTransferLocked(pTransfer);
+
+    unlock();
 
     return rc;
@@ -1057 +1104 @@
 int SharedClipboardWinDataObject::SetStatus(Status enmStatus, int rcSts /* = VINF_SUCCESS */)
 {
-    int rc = RTCritSectEnter(&m_CritSect);
-    if (RT_SUCCESS(rc))
-    {
-        rc = setStatusLocked(enmStatus, rcSts);
-
-        RTCritSectLeave(&m_CritSect);
-    }
-
+    lock();
+
+    int rc = setStatusLocked(enmStatus, rcSts);
+
+    unlock();
     return rc;
 }
@@ -1141 +1185 @@
     RT_NOREF(rc);
 
-    int rc2 = RTCritSectEnter(&m_CritSect);
-    if (RT_SUCCESS(rc2))
-    {
-        LogFlowFunc(("enmStatus=%#x (current is: %#x)\n", enmStatus, m_enmStatus));
-
-        switch (enmStatus)
-        {
-            case Completed:
-            {
-                LogFlowFunc(("m_uObjIdx=%RU32 (total: %zu)\n", m_uObjIdx, m_lstEntries.size()));
-
-                const bool fComplete = m_uObjIdx == m_lstEntries.size() - 1 /* Object index is zero-based */;
-                if (fComplete)
-                    m_enmStatus = Completed;
-                break;
-            }
-
-            default:
-            {
-                m_enmStatus = enmStatus;
-                break;
-            }
-        }
-
-        if (RT_FAILURE(rc))
-            LogRel(("Shared Clipboard: Data object received error %Rrc (status %#x)\n", rc, enmStatus));
-
-        if (m_EventStatusChanged != NIL_RTSEMEVENT)
-            rc2 = RTSemEventSignal(m_EventStatusChanged);
-
-        RTCritSectLeave(&m_CritSect);
-    }
+    LogFlowFunc(("enmStatus=%#x (current is: %#x)\n", enmStatus, m_enmStatus));
+
+    int rc2 = VINF_SUCCESS;
+
+    switch (enmStatus)
+    {
+        case Completed:
+        {
+            LogFlowFunc(("m_uObjIdx=%RU32 (total: %zu)\n", m_uObjIdx, m_lstEntries.size()));
+
+            const bool fComplete = m_uObjIdx == m_lstEntries.size() - 1 /* Object index is zero-based */;
+            if (fComplete)
+                m_enmStatus = Completed;
+            break;
+        }
+
+        default:
+        {
+            m_enmStatus = enmStatus;
+            break;
+        }
+    }
+
+    if (RT_FAILURE(rc))
+        LogRel(("Shared Clipboard: Data object received error %Rrc (status %#x)\n", rc, enmStatus));
+
+    if (m_EventStatusChanged != NIL_RTSEMEVENT)
+        rc2 = RTSemEventSignal(m_EventStatusChanged);
 
     return rc2;