Changeset 51947 in vbox

Timestamp:

Jul 9, 2014 8:55:34 AM (10 years ago)

Author:

vboxsync

Message:

SUPDrv-win.cpp: Drop the DOS/Windows symlinks.

File:

• trunk/src/VBox/HostDrivers/Support/win/SUPDrv-win.cpp (modified) (3 diffs)

trunk/src/VBox/HostDrivers/Support/win/SUPDrv-win.cpp

@@ -61 +61 @@
 #define SUPDRV_NT_POOL_TAG  'xoBV'
 
-/** Win32 device name for system access. */
-#define DEVICE_NAME_SYS         "\\\\.\\VBoxDrv"
 /** NT device name for system access. */
 #define DEVICE_NAME_NT_SYS      L"\\Device\\VBoxDrv"
-/** Win Symlink name for system access. */
-#define DEVICE_NAME_DOS_SYS     L"\\DosDevices\\VBoxDrv"
-
-/** Win32 device name for user access. */
-#define DEVICE_NAME_USR         "\\\\.\\VBoxDrvU"
 /** NT device name for user access. */
 #define DEVICE_NAME_NT_USR      L"\\Device\\VBoxDrvU"
-/** Win Symlink name for user access. */
-#define DEVICE_NAME_DOS_USR     L"\\DosDevices\\VBoxDrvU"
-
 #ifdef VBOX_WITH_HARDENING
-/** Win32 device name for hardened stub access. */
-# define DEVICE_NAME_STUB       "\\\\.\\VBoxDrvStub"
 /** NT device name for hardened stub access. */
 # define DEVICE_NAME_NT_STUB    L"\\Device\\VBoxDrvStub"
-///** Win Symlink name for hardened stub access. */
-//# define DEVICE_NAME_DOS_STUB   L"\\DosDevices\\VBoxDrvStub"
 
 /** Macro for checking for deflecting calls to the stub device. */
@@ -370 +356 @@
     if (NT_SUCCESS(rcNt))
     {
-        UNICODE_STRING DosNameSys;
-        RtlInitUnicodeString(&DosNameSys, DEVICE_NAME_DOS_SYS);
-        rcNt = IoCreateSymbolicLink(&DosNameSys, &DevName);
+        /*
+         * User device.
+         */
+        RtlInitUnicodeString(&DevName, DEVICE_NAME_NT_USR);
+        rcNt = IoCreateDevice(pDrvObj, sizeof(SUPDRVDEVEXTUSR), &DevName, FILE_DEVICE_UNKNOWN, 0, FALSE, &g_pDevObjUsr);
         if (NT_SUCCESS(rcNt))
         {
+            PSUPDRVDEVEXTUSR pDevExtUsr = (PSUPDRVDEVEXTUSR)g_pDevObjUsr->DeviceExtension;
+            pDevExtUsr->pMainDrvExt = (PSUPDRVDEVEXT)g_pDevObjSys->DeviceExtension;
+            pDevExtUsr->u32Cookie   = SUPDRVDEVEXTUSR_COOKIE;
+
+#ifdef VBOX_WITH_HARDENING
             /*
-             * User device.
+             * Hardened stub device.
              */
-            RtlInitUnicodeString(&DevName, DEVICE_NAME_NT_USR);
-            rcNt = IoCreateDevice(pDrvObj, sizeof(SUPDRVDEVEXTUSR), &DevName, FILE_DEVICE_UNKNOWN, 0, FALSE, &g_pDevObjUsr);
+            RtlInitUnicodeString(&DevName, DEVICE_NAME_NT_STUB);
+            rcNt = IoCreateDevice(pDrvObj, sizeof(SUPDRVDEVEXTSTUB), &DevName, FILE_DEVICE_UNKNOWN, 0, FALSE, &g_pDevObjStub);
             if (NT_SUCCESS(rcNt))
             {
-                UNICODE_STRING DosNameUsr;
-                RtlInitUnicodeString(&DosNameUsr, DEVICE_NAME_DOS_USR);
-                rcNt = IoCreateSymbolicLink(&DosNameUsr, &DevName);
                 if (NT_SUCCESS(rcNt))
                 {
-                    PSUPDRVDEVEXTUSR pDevExtUsr = (PSUPDRVDEVEXTUSR)g_pDevObjUsr->DeviceExtension;
-                    pDevExtUsr->pMainDrvExt = (PSUPDRVDEVEXT)g_pDevObjSys->DeviceExtension;
-                    pDevExtUsr->u32Cookie   = SUPDRVDEVEXTUSR_COOKIE;
-
+                    PSUPDRVDEVEXTSTUB pDevExtStub = (PSUPDRVDEVEXTSTUB)g_pDevObjStub->DeviceExtension;
+                    pDevExtStub->Common.pMainDrvExt = (PSUPDRVDEVEXT)g_pDevObjSys->DeviceExtension;
+                    pDevExtStub->Common.u32Cookie   = SUPDRVDEVEXTSTUB_COOKIE;
+#endif
+
+                    /* Done. */
+                    return rcNt;
 #ifdef VBOX_WITH_HARDENING
-                    /*
-                     * Hardened stub device.
-                     */
-                    RtlInitUnicodeString(&DevName, DEVICE_NAME_NT_STUB);
-                    rcNt = IoCreateDevice(pDrvObj, sizeof(SUPDRVDEVEXTSTUB), &DevName, FILE_DEVICE_UNKNOWN, 0, FALSE, &g_pDevObjStub);
-                    if (NT_SUCCESS(rcNt))
-                    {
-                        //UNICODE_STRING DosNameStub;
-                        //RtlInitUnicodeString(&DosNameStub, DEVICE_NAME_DOS_STUB);
-                        //rcNt = IoCreateSymbolicLink(&DosNameStub, &DevName);
-                        if (NT_SUCCESS(rcNt))
-                        {
-                            PSUPDRVDEVEXTSTUB pDevExtStub = (PSUPDRVDEVEXTSTUB)g_pDevObjStub->DeviceExtension;
-                            pDevExtStub->Common.pMainDrvExt = (PSUPDRVDEVEXT)g_pDevObjSys->DeviceExtension;
-                            pDevExtStub->Common.u32Cookie   = SUPDRVDEVEXTSTUB_COOKIE;
+                }
+
+                /* Bail out. */
+                IoDeleteDevice(g_pDevObjStub);
+                g_pDevObjUsr = NULL;
+            }
 #endif
-
-                            /* Done. */
-                            return rcNt;
-#ifdef VBOX_WITH_HARDENING
-                        }
-
-                        /* Bail out. */
-                        IoDeleteDevice(g_pDevObjStub);
-                        g_pDevObjUsr = NULL;
-                    }
-                    IoDeleteSymbolicLink(&DosNameUsr);
-#endif
-                }
-                IoDeleteDevice(g_pDevObjUsr);
-                g_pDevObjUsr = NULL;
-            }
-            IoDeleteSymbolicLink(&DosNameSys);
+            IoDeleteDevice(g_pDevObjUsr);
+            g_pDevObjUsr = NULL;
         }
         IoDeleteDevice(g_pDevObjSys);
@@ -437 +406 @@
 static void vboxdrvNtDestroyDevices(void)
 {
-    UNICODE_STRING DosName;
-    RtlInitUnicodeString(&DosName, DEVICE_NAME_DOS_SYS);
-    NTSTATUS rcNt = IoDeleteSymbolicLink(&DosName);
-
-    RtlInitUnicodeString(&DosName, DEVICE_NAME_DOS_USR);
-    rcNt = IoDeleteSymbolicLink(&DosName);
-
-#ifdef VBOX_WITH_HARDENING
-    //RtlInitUnicodeString(&DosName, DEVICE_NAME_DOS_STUB);
-    //rcNt = IoDeleteSymbolicLink(&DosName);
-#endif
-
     if (g_pDevObjUsr)
     {