[vbox-dev] [PATCH] Fix for ticket #6453 diff and description

Dennis Lawler Dennis_Lawler at symantec.com
Wed Jun 23 20:23:22 PDT 2010


Fairly minimal patch to address ticket #6453, and some minor changes.  I
release this tiny bit of code under the MIT license.

 

Problem description and fix:

If verifier is enabled with "Force Pending IO requests", IoCallDriver
will return 0x103(STATUS_PENDING).  If IoCallDriver returns pending, we
wait for the completion event as we should - but we fail to copy the
status from the IOSB, so the packet is completed with STATUS_PENDING.
This will cause a verifier bugcheck, and an assertion on checked builds
of Windows.  The fix is to simply copy the status value from the IOSB to
the status variable after waiting for completion.

 

On a side note, if anyone has a few spare cycles to work on #6183, it
should be a pretty simple fix and would be greatly appreciated.

 

Additional changes: 

-          Optimization from DDK sample event-setting completion routine
added.  See WDK\src\general\toaster\wdm\filter\filter.c,
FilterStartCompletionRoutine or 
http://msdn.microsoft.com/en-us/library/ms810023.aspx.

-          Removed information and Irp->IoStatus.Status check as they
are unnecessary.

 

 

-Dennis Lawler

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.virtualbox.org/pipermail/vbox-dev/attachments/20100623/96ea428c/attachment-0001.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: VBoxMouse.patch
Type: application/octet-stream
Size: 1743 bytes
Desc: VBoxMouse.patch
Url : http://www.virtualbox.org/pipermail/vbox-dev/attachments/20100623/96ea428c/attachment-0001.obj 


More information about the vbox-dev mailing list