[pschweitzer] 65144: [NTOSKRNL] Testman confirms that IopParseDevice() hack is no longer required. So, this is the end of this 8y old hack. Now ReactOS is able to properly use its storage stack (wh...

Author: pschweitzer
Date: Fri Oct 31 14:16:15 2014
New Revision: 65144

URL: http://svn.reactos.org/svn/reactos?rev=65144&view=rev
Log:
[NTOSKRNL]
Testman confirms that IopParseDevice() hack is no longer required.
So, this is the end of this 8y old hack.

Now ReactOS is able to properly use its storage stack (who said, amazing?)

Bisous IopParseDevice() hack :-)

Modified:
    trunk/reactos/ntoskrnl/io/iomgr/file.c

Modified: trunk/reactos/ntoskrnl/io/iomgr/file.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/file.c?r…
==============================================================================
--- trunk/reactos/ntoskrnl/io/iomgr/file.c	[iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/io/iomgr/file.c	[iso-8859-1] Fri Oct 31 14:16:15 2014
@@ -419,28 +419,6 @@
         DirectOpen = TRUE;
     }
 
-#if 0
-    /* FIXME: Small hack still exists, have to check why...
-     * This is triggered multiple times by usetup and then once per boot.
-     */
-    if (!(DirectOpen) &&
-        !(RemainingName->Length) &&
-        !(OpenPacket->RelatedFileObject) &&
-        ((wcsstr(CompleteName->Buffer, L"Harddisk")) ||
-         (wcsstr(CompleteName->Buffer, L"Floppy"))) &&
-        !(UseDummyFile))
-    {
-        DPRINT1("Using IopParseDevice() hack. Requested invalid attributes:
%lx\n",
-        DesiredAccess & ~(SYNCHRONIZE |
-                          FILE_READ_ATTRIBUTES |
-                          READ_CONTROL |
-                          ACCESS_SYSTEM_SECURITY |
-                          WRITE_OWNER |
-                          WRITE_DAC));
-        DirectOpen = TRUE;
-    }
-#endif
-
     /* Check if we have a related FO that wasn't a direct open */
     if ((OpenPacket->RelatedFileObject) &&
         !(OpenPacket->RelatedFileObject->Flags & FO_DIRECT_DEVICE_OPEN))