Changeset 100651 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Jul 19, 2023 11:52:59 AM (21 months ago)

Author:

vboxsync

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

158488

Message:

Shared Clipboard: Also give the registered service extension the chance of calling an optional callback. Only used by VRDE so far (and probably never will be used by anything else, historical reasons). bugref:9437

File:

• trunk/src/VBox/Main/src-client/GuestShClPrivate.cpp (modified) (5 diffs)

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

@@ -60 +60 @@
 GuestShCl::GuestShCl(Console *pConsole)
     : m_pConsole(pConsole)
+    , m_pfnExtCallback(NULL)
 {
     LogFlowFuncEnter();
@@ -86 +87 @@
 
     RT_ZERO(m_SvcExtVRDP);
+
+    m_pfnExtCallback = NULL;
 }
 
@@ -122 +125 @@
 {
     AssertPtrReturn(pfnExtension, VERR_INVALID_POINTER);
+    /* pvExtension is optional. */
 
     lock();
 
+    LogFlowFunc(("m_pfnExtCallback=%p\n", this->m_pfnExtCallback));
+
     PSHCLSVCEXT pExt = &this->m_SvcExtVRDP; /* Currently we only have one extension only. */
 
     Assert(pExt->pfnExt == NULL);
 
-    pExt->pfnExt = pfnExtension;
-    pExt->pvExt  = pvExtension;
+    pExt->pfnExt         = pfnExtension;
+    pExt->pvExt          = pvExtension;
+    pExt->pfnExtCallback = this->m_pfnExtCallback; /* Assign callback function. Optional and can be NULL. */
+
+    if (pExt->pfnExtCallback)
+    {
+        /* Make sure to also give the extension the ability to use the callback. */
+        SHCLEXTPARMS parms;
+        RT_ZERO(parms);
+
+        parms.u.SetCallback.pfnCallback = pExt->pfnExtCallback;
+
+        /* ignore rc, callback is optional */ pExt->pfnExt(pExt->pvExt,
+                                                           VBOX_CLIPBOARD_EXT_FN_SET_CALLBACK, &parms, sizeof(parms));
+    }
 
     unlock();
@@ -153 +172 @@
     AssertReturnStmt(pExt->pfnExt == pfnExtension, unlock(), VERR_INVALID_PARAMETER);
     AssertPtr(pExt->pfnExt);
+
+    /* Unregister the callback (setting to NULL). */
+    SHCLEXTPARMS parms;
+    RT_ZERO(parms);
+
+    /* ignore rc, callback is optional */ pExt->pfnExt(pExt->pvExt,
+                                                       VBOX_CLIPBOARD_EXT_FN_SET_CALLBACK, &parms, sizeof(parms));
 
     RT_BZERO(pExt, sizeof(SHCLSVCEXT));
@@ -244 +270 @@
     switch (u32Function)
     {
+        case VBOX_CLIPBOARD_EXT_FN_SET_CALLBACK:
+            pThis->m_pfnExtCallback = pParms->u.SetCallback.pfnCallback;
+            vrc = VINF_SUCCESS;
+            break;
+
         case VBOX_CLIPBOARD_EXT_FN_ERROR:
         {