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

Timestamp:

Jul 18, 2023 8:06:24 AM (17 months ago)

Author:

vboxsync

Message:

Shared Clipboard: Made the Main host service dispatcher a bit more flexible so that it returns VERR_NOT_SUPPORTED if certain commands are not being handled. See comment(s) for details. bugref:9437

Location:

trunk/src/VBox/HostServices/SharedClipboard

Files:

• VBoxSharedClipboardSvc-internal.h (modified) (1 diff)
• VBoxSharedClipboardSvc.cpp (modified) (3 diffs)

trunk/src/VBox/HostServices/SharedClipboard/VBoxSharedClipboardSvc-internal.h

@@ -265 +265 @@
 typedef struct _SHCLEXTSTATE
 {
-    /** Pointer to the actual service extension handle. */
+    /** Pointer to the actual service extension handle.
+     *
+     * Must return VERR_NOT_SUPPORTED if the extension didn't handle the requested function.
+     * This will invoke the regular backend then.
+     */
     PFNHGCMSVCEXT  pfnExtension;
     /** Opaque pointer to extension-provided data. Don't touch. */

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

@@ -1617 +1617 @@
                 SHCLEXTPARMS parms;
                 RT_ZERO(parms);
+
                 parms.u.ReportFormats.uFormats = fFormats;
 
-                g_ExtState.pfnExtension(g_ExtState.pvExtension, VBOX_CLIPBOARD_EXT_FN_FORMAT_ANNOUNCE, &parms, sizeof(parms));
+                rc = g_ExtState.pfnExtension(g_ExtState.pvExtension, VBOX_CLIPBOARD_EXT_FN_FORMAT_ANNOUNCE, &parms, sizeof(parms));
             }
             else
+                rc = VERR_NOT_SUPPORTED;
+
+            /* Let the backend implementation know if the extension above didn't handle the format(s). */
+            if (rc == VERR_NOT_SUPPORTED)
             {
 #ifdef LOG_ENABLED
@@ -1783 +1788 @@
     }
     else
+        rc = VERR_NOT_SUPPORTED;
+
+    /* Try reading from the backend if the extension above didn't handle the read. */
+    if (rc == VERR_NOT_SUPPORTED)
     {
         rc = ShClBackendReadData(&g_ShClBackend, pClient, &cmdCtx, uFormat, pvData, cbData, &cbActual);
@@ -1949 +1958 @@
         SHCLEXTPARMS parms;
         RT_ZERO(parms);
+
         parms.u.ReadWriteData.uFormat = uFormat;
         parms.u.ReadWriteData.pvData  = pvData;
         parms.u.ReadWriteData.cbData  = cbData;
 
-        g_ExtState.pfnExtension(g_ExtState.pvExtension, VBOX_CLIPBOARD_EXT_FN_DATA_WRITE, &parms, sizeof(parms));
-        rc = VINF_SUCCESS;
+        rc = g_ExtState.pfnExtension(g_ExtState.pvExtension, VBOX_CLIPBOARD_EXT_FN_DATA_WRITE, &parms, sizeof(parms));
     }
     else
-    {
-        /* Let the backend implementation know. */
+        rc = VERR_NOT_SUPPORTED;
+
+    /* Let the backend implementation know if the extension above didn't handle the write. */
+    if (rc == VERR_NOT_SUPPORTED)
+    {
         rc = ShClBackendWriteData(&g_ShClBackend, pClient, &cmdCtx, uFormat, pvData, cbData);
         if (RT_FAILURE(rc))