Changeset 1842 in vbox for trunk/src/VBox/HostDrivers

Timestamp:

Mar 30, 2007 8:30:47 PM (18 years ago)

Author:

vboxsync

Message:

test the minimum requirement and no the required one.

File:

• trunk/src/VBox/HostDrivers/Support/SUPDRVShared.c (modified) (4 diffs)

trunk/src/VBox/HostDrivers/Support/SUPDRVShared.c

@@ -659 +659 @@
              */
             if (    pIn->u32MinVersion > SUPDRVIOC_VERSION
-                ||  (pIn->u32ReqVersion & 0xffff0000) != (SUPDRVIOC_VERSION & 0xffff0000))
+                ||  (pIn->u32MinVersion & 0xffff0000) != (SUPDRVIOC_VERSION & 0xffff0000))
             {
                 OSDBGPRINT(("SUP_IOCTL_COOKIE: Version mismatch. Requested: %#x  Min: %#x  Current: %#x\n", 
@@ -675 +675 @@
             /*
              * Fill in return data and be gone.
+             * N.B. The first one to change SUPDRVIOC_VERSION shall makes sure that 
+             *      u32SessionVersion <= u32ReqVersion!
              */
             /** @todo A more secure cookie negotiation? */
@@ -680 +682 @@
             pOut->u32SessionCookie  = pSession->u32Cookie;
             pOut->u32SessionVersion = SUPDRVIOC_VERSION;
-            pOut->u32DriverVersion  = SUPDRVIOC_VERSION;
+            pOut->u32DriverVersion  = SUPDRVIOC_VERSION; 
             pOut->pSession          = pSession;
             pOut->cFunctions        = sizeof(g_aFunctions) / sizeof(g_aFunctions[0]);
@@ -2861 +2863 @@
     /*
      * Call VMMR0Entry
-     *      We don't have to push the arguments here, but we have to
+     *      We don't have to push the arguments here, but we have top
      *      reserve some stack space for the interrupt forwarding.
      */