[ros-diffs] arty: 21367 - /trunk/documentation/audit/drivers/filesystems/vfat.txt

Author: arty
Date: Thu Mar 23 00:13:49 2006
New Revision: 21367

URL: http://svn.reactos.ru/svn/reactos?rev=21367&view=rev
Log:
Updated slightly.  IRP use here is clearly a bug.

Modified:
    trunk/documentation/audit/drivers/filesystems/vfat.txt

Modified: trunk/documentation/audit/drivers/filesystems/vfat.txt
URL:
http://svn.reactos.ru/svn/reactos/trunk/documentation/audit/drivers/filesys…
==============================================================================
--- trunk/documentation/audit/drivers/filesystems/vfat.txt (original)
+++ trunk/documentation/audit/drivers/filesystems/vfat.txt Thu Mar 23 00:13:49 2006
@@ -10,6 +10,7 @@
 virtually every operating system implements them.
 
 blockdev.c:
+Result: Mostly Obvious
 
 VfatReadWritePartialCompletion:
 Result: Obvious
@@ -39,7 +40,7 @@
 [1]
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/Kernel_r/h…
 
 VfatReadDiskPartial:
-Result: Obvious/Suspicious
+Result: Obvious
 
 This function uses VfatReadWritePartialCompletion to wait for all fired IRPs
 to return.  It only waits for completion if Wait is set.  If Wait is not set,
@@ -47,7 +48,7 @@
 
 blockdev.c:132     -- Get the address of the MDL provided by the user [2]
 blockdev.c:134-141 -- Allocate a sub-IRP and handle error
-blockdev.c:141     -- Suspicious.  Can somebody tell me if this is OK?
+blockdev.c:141     -- This has been identified as a bug.  We'll fix it.
 blockdev.c:142     -- Irp->Tail.Overlay is a general information structure
 blockdev.c:144-155 -- Set up IRP stack location
 blockdev.c:155-164 -- Allocate MDL for callee to use, handle error
@@ -74,3 +75,12 @@
 @blockdev.c:334     -- Propogate returned data length [3]
 
 [3]
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dndevice/h…
+
+cleanup.c:
+Result: 
+
+VfatCleanupFile:
+Result:
+
+This function returns resources after a file handle is closed.
+