Changeset 5707 in vbox for trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp

Timestamp:

Nov 12, 2007 2:35:27 PM (17 years ago)

Author:

vboxsync

Message:

KeWaitForSingleObject must use UserMode or else a kill signal won't abort the wait.

File:

• trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp (modified) (3 diffs)

trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp

@@ -339 +339 @@
 {
     PVBOXGUESTDEVEXT pDevExt = (PVBOXGUESTDEVEXT)pvData;
-    LARGE_INTEGER timeout;
+    PLARGE_INTEGER pTimeout;
 
     dprintf(("VBoxHGCMCallback\n"));
@@ -346 +346 @@
      * call; introduce a timeout to make sure we don't wait indefinitely.
      */
-    timeout.QuadPart  = 250;
-    timeout.QuadPart *= -10000;     /* relative in 100ns units */
+    pTimeout = (PLARGE_INTEGER)VbglPhysHeapAlloc(sizeof(LARGE_INTEGER));
+    Assert(pTimeout);
+    if (!pTimeout)
+        return;
+        
+    pTimeout->QuadPart  = 250;
+    pTimeout->QuadPart *= -10000;     /* relative in 100ns units */
+
 
     while ((pHeader->fu32Flags & VBOX_HGCM_REQ_DONE) == 0)
     {
+        /* Specifying UserMode so killing the user process will abort the wait. */ 
         NTSTATUS rc = KeWaitForSingleObject (&pDevExt->keventNotification, Executive,
-                                             KernelMode, TRUE, &timeout
+                                             UserMode, TRUE, pTimeout
                                             );
         dprintf(("VBoxHGCMCallback: Wait returned %d fu32Flags=%x\n", rc, pHeader->fu32Flags));
@@ -367 +374 @@
         dprintf(("VBoxHGCMCallback: fu32Flags = %08X\n", pHeader->fu32Flags));
     }
-    
+    VbglPhysHeapFree(pTimeout);    
     return;
 }