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/filesyst...
==============================================================================
--- 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/hh...

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/ht...
+
+cleanup.c:
+Result: 
+
+VfatCleanupFile:
+Result:
+
+This function returns resources after a file handle is closed.
+