Changeset 70219 in vbox for trunk/src/VBox/Additions

Timestamp:

Dec 19, 2017 11:18:18 AM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

119809

Message:

VBoxGuest-win.cpp: Cleaning up version code a little and made it report VBOXOSTYPE_WinNT3x when appropriate.

File:

• trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuest-win.cpp (modified) (7 diffs)

trunk/src/VBox/Additions/common/VBoxGuest/VBoxGuest-win.cpp

@@ -163 +163 @@
 {
     VGDRVNTVER_INVALID = 0,
-    VGDRVNTVER_WINNT31,
+    VGDRVNTVER_WINNT310,
     VGDRVNTVER_WINNT350,
     VGDRVNTVER_WINNT351,
@@ -285 +285 @@
     switch (ulMajorVer)
     {
-        case 10:
-            switch (ulMinorVer)
-            {
-                case 0:
-                    /* Windows 10 Preview builds starting with 9926. */
-                default:
-                    /* Also everything newer. */
-                    g_enmVGDrvNtVer = VGDRVNTVER_WIN10;
-                    break;
-            }
+        case 10: /* Windows 10 Preview builds starting with 9926. */
+            g_enmVGDrvNtVer = VGDRVNTVER_WIN10;
             break;
         case 6: /* Windows Vista or Windows 7 (based on minor ver) */
@@ -311 +303 @@
                     g_enmVGDrvNtVer = VGDRVNTVER_WIN81;
                     break;
-                case 4:
-                    /* Windows 10 Preview builds. */
+                case 4: /* Windows 10 Preview builds. */
                 default:
-                    /* Also everything newer. */
                     g_enmVGDrvNtVer = VGDRVNTVER_WIN10;
                     break;
@@ -343 +333 @@
                 g_enmVGDrvNtVer = VGDRVNTVER_WINNT350;
             else
-                g_enmVGDrvNtVer = VGDRVNTVER_WINNT31;
+                g_enmVGDrvNtVer = VGDRVNTVER_WINNT310;
             break;
         default:
+            /* Major versions above 6 gets classified as windows 10. */
             if (ulMajorVer > 6)
-            {
-                /* "Windows 10 mode" for Windows 8.1+. */
                 g_enmVGDrvNtVer = VGDRVNTVER_WIN10;
-            }
             else
             {
-                if (ulMajorVer < 4)
-                    LogRelFunc(("At least Windows NT4 required! (%u.%u)\n", ulMajorVer, ulMinorVer));
-                else
-                    LogRelFunc(("Unknown version %u.%u!\n", ulMajorVer, ulMinorVer));
+                LogRelFunc(("At least Windows NT 3.10 required! Found %u.%u!\n", ulMajorVer, ulMinorVer));
                 rcNt = STATUS_DRIVER_UNABLE_TO_LOAD;
             }
@@ -429 +414 @@
 
 /**
- * Translates NT version to VBox OS.
+ * Translates our internal NT version enum to VBox OS.
  *
  * @returns VBox OS type.
@@ -439 +424 @@
     switch (enmNtVer)
     {
-        case VGDRVNTVER_WINNT31:
-        case VGDRVNTVER_WINNT350:
-        case VGDRVNTVER_WINNT351:
-        case VGDRVNTVER_WINNT4:
-            enmOsType = VBOXOSTYPE_WinNT4;
-            break;
-
-        case VGDRVNTVER_WIN2K:
-            enmOsType = VBOXOSTYPE_Win2k;
-            break;
-
-        case VGDRVNTVER_WINXP:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_WinXP_x64;
-#else
-            enmOsType = VBOXOSTYPE_WinXP;
-#endif
-            break;
-
-        case VGDRVNTVER_WIN2K3:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_Win2k3_x64;
-#else
-            enmOsType = VBOXOSTYPE_Win2k3;
-#endif
-            break;
-
-        case VGDRVNTVER_WINVISTA:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_WinVista_x64;
-#else
-            enmOsType = VBOXOSTYPE_WinVista;
-#endif
-            break;
-
-        case VGDRVNTVER_WIN7:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_Win7_x64;
-#else
-            enmOsType = VBOXOSTYPE_Win7;
-#endif
-            break;
-
-        case VGDRVNTVER_WIN8:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_Win8_x64;
-#else
-            enmOsType = VBOXOSTYPE_Win8;
-#endif
-            break;
-
-        case VGDRVNTVER_WIN81:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_Win81_x64;
-#else
-            enmOsType = VBOXOSTYPE_Win81;
-#endif
-            break;
-
-        case VGDRVNTVER_WIN10:
-#if ARCH_BITS == 64
-            enmOsType = VBOXOSTYPE_Win10_x64;
-#else
-            enmOsType = VBOXOSTYPE_Win10;
-#endif
-            break;
+        case VGDRVNTVER_WINNT310:   enmOsType = VBOXOSTYPE_WinNT3x; break;
+        case VGDRVNTVER_WINNT350:   enmOsType = VBOXOSTYPE_WinNT3x; break;
+        case VGDRVNTVER_WINNT351:   enmOsType = VBOXOSTYPE_WinNT3x; break;
+        case VGDRVNTVER_WINNT4:     enmOsType = VBOXOSTYPE_WinNT4; break;
+        case VGDRVNTVER_WIN2K:      enmOsType = VBOXOSTYPE_Win2k; break;
+        case VGDRVNTVER_WINXP:      enmOsType = VBOXOSTYPE_WinXP; break;
+        case VGDRVNTVER_WIN2K3:     enmOsType = VBOXOSTYPE_Win2k3; break;
+        case VGDRVNTVER_WINVISTA:   enmOsType = VBOXOSTYPE_WinVista; break;
+        case VGDRVNTVER_WIN7:       enmOsType = VBOXOSTYPE_Win7; break;
+        case VGDRVNTVER_WIN8:       enmOsType = VBOXOSTYPE_Win8; break;
+        case VGDRVNTVER_WIN81:      enmOsType = VBOXOSTYPE_Win81; break;
+        case VGDRVNTVER_WIN10:      enmOsType = VBOXOSTYPE_Win10; break;
 
         default:
@@ -511 +442 @@
             break;
     }
+#if ARCH_BITS == 64
+    enmOsType = (VBOXOSTYPE)((int)enmOsType | VBOXOSTYPE_x64);
+#endif
     return enmOsType;
 }