Changeset 7812 in vbox for trunk/src/VBox/HostServices/SharedFolders/service.cpp

Timestamp:

Apr 8, 2008 3:44:09 PM (17 years ago)

Author:

vboxsync

Message:

HGCM: Add user data pointer to HGCM service function table - fixed a parameter mismatch

File:

• trunk/src/VBox/HostServices/SharedFolders/service.cpp (modified) (3 diffs)

trunk/src/VBox/HostServices/SharedFolders/service.cpp

@@ -1224 +1224 @@
 }
 
-extern "C" DECLCALLBACK(DECLEXPORT(int)) VBoxHGCMSvcLoad (void *, VBOXHGCMSVCFNTABLE *ptable)
+extern "C" DECLCALLBACK(DECLEXPORT(int)) VBoxHGCMSvcLoad (VBOXHGCMSVCFNTABLE *ptable)
 {
     int rc = VINF_SUCCESS;
@@ -1230 +1230 @@
     Log(("VBoxHGCMSvcLoad: ptable = %p\n", ptable));
 
-    if (!ptable)
+    if (!VALID_PTR(ptable))
     {
+        LogRelFunc(("Bad value of ptable (%p) in shared folders service\n", ptable));
         rc = VERR_INVALID_PARAMETER;
     }
@@ -1241 +1242 @@
             ||  ptable->u32Version != VBOX_HGCM_SVC_VERSION)
         {
-            rc = VERR_INVALID_PARAMETER;
+            LogRelFunc(("version mismatch loading shared folders service: ptable->cbSize = %d, should be %d, ptable->u32Version = 0x%08X, should be 0x%08X\n", ptable->cbSize, sizeof (VBOXHGCMSVCFNTABLE), ptable->u32Version, VBOX_HGCM_SVC_VERSION));
+            rc = VERR_VERSION_MISMATCH;
         }
         else