IopCreateFile(): Fix semantics of volume mounting (Vpb vs. Device etc.)

[reactos.git] / ntoskrnl / io / create.c

diff --git a/ntoskrnl/io/create.c b/ntoskrnl/io/create.c
index f1254dd..cf0553d 100644 (file)
--- a/ntoskrnl/io/create.c
+++ b/ntoskrnl/io/create.c
@@ -114,6 +114,7 @@ IopCreateFile(PVOID                 ObjectBody,
       FileObject->Flags = FileObject->Flags | FO_DIRECT_DEVICE_OPEN;
       FileObject->FileName.Buffer = 0;
       FileObject->FileName.Length = FileObject->FileName.MaximumLength = 0;
+      FileObject->Vpb = DeviceObject->Vpb;
     }
   else
     {
@@ -126,6 +127,7 @@ IopCreateFile(PVOID                 ObjectBody,
          && (DeviceObject->DeviceType != FILE_DEVICE_MAILSLOT))
        {
          CPRINT("Device was wrong type\n");
+         KeBugCheck(0);
          return(STATUS_UNSUCCESSFUL);
        }
 
@@ -144,12 +146,20 @@ IopCreateFile(PVOID                       ObjectBody,
                         Status);
                  return(Status);
                }
+             if (!DeviceObject->Vpb)
+               KeBugCheck(0);
+             if (DeviceObject->Vpb->DeviceObject->Vpb)
+               KeBugCheck(0);
+             DeviceObject->Vpb->DeviceObject->Vpb=DeviceObject->Vpb;
+             DeviceObject->Vpb->Flags |= VPB_MOUNTED;
            }
+
          DeviceObject = DeviceObject->Vpb->DeviceObject;
          DPRINT("FsDeviceObject %lx\n", DeviceObject);
        }
       RtlCreateUnicodeString(&(FileObject->FileName),
                             RemainingPath);
+      FileObject->Vpb = NULL;
     }
 
   DPRINT("FileObject->FileName %wZ\n",
@@ -158,7 +168,6 @@ IopCreateFile(PVOID                 ObjectBody,
   DPRINT("FileObject %x DeviceObject %x\n",
         FileObject,
         DeviceObject);
-  FileObject->Vpb = DeviceObject->Vpb;
   FileObject->Type = InternalFileType;
   FileObject->RelatedFileObject = NULL;
 
@@ -367,10 +376,16 @@ IoCreateFile(OUT  PHANDLE                 FileHandle,
    if( CreateOptions & FILE_NO_INTERMEDIATE_BUFFERING )
      FileObject->Flags |= FO_NO_INTERMEDIATE_BUFFERING;
 
+   FileObject->FsContext=NULL;
+   FileObject->FsContext2=NULL;
 
    RtlZeroMemory(&AccessState, sizeof(AccessState));
-   AccessState.RemainingDesiredAccess=0;
-   AccessState.PreviouslyGrantedAccess=0;
+   AccessState.RemainingDesiredAccess=0;       /* FIXME: Meaning? */
+   AccessState.PreviouslyGrantedAccess=DesiredAccess;  /* FIXME: Meaning? */
+   /* 'OriginalDesiredAccess' is required during file create by ntfs.sys of NT-5.1sp1
+    * W32 undocumented.
+    */
+   AccessState.OriginalDesiredAccess=DesiredAccess;    /* FIXME: Meaning? */
 
    SecurityContext.SecurityQos = NULL; /* ?? */
    SecurityContext.AccessState = &AccessState;