From 74c48c65169c9283d8633a78cafb57303c81ccfc Mon Sep 17 00:00:00 2001 From: Andrew Nelless Date: Fri, 24 Jun 2016 13:10:17 +0100 Subject: [PATCH] #4978 Removed getOSName impl relying on deprecated GetVersionEx --- src/lib/arch/win32/ArchSystemWindows.cpp | 66 +++++++----------------- 1 file changed, 18 insertions(+), 48 deletions(-) diff --git a/src/lib/arch/win32/ArchSystemWindows.cpp b/src/lib/arch/win32/ArchSystemWindows.cpp index 5540e40c..0fe1e805 100644 --- a/src/lib/arch/win32/ArchSystemWindows.cpp +++ b/src/lib/arch/win32/ArchSystemWindows.cpp @@ -49,56 +49,26 @@ ArchSystemWindows::~ArchSystemWindows() std::string ArchSystemWindows::getOSName() const { -#if WINVER >= _WIN32_WINNT_WIN2K - OSVERSIONINFOEX info; -#else - OSVERSIONINFO info; -#endif + std::string osName ("Microsoft Windows "); + static const TCHAR* const windowsVersionKeyNames[] = { + _T("SOFTWARE"), + _T("Microsoft"), + _T("Windows NT"), + _T("CurrentVersion"), + NULL + }; - info.dwOSVersionInfoSize = sizeof(info); - if (GetVersionEx((OSVERSIONINFO*) &info)) { - switch (info.dwPlatformId) { - case VER_PLATFORM_WIN32_NT: - #if WINVER >= _WIN32_WINNT_WIN2K - if (info.dwMajorVersion == 6) { - if (info.dwMinorVersion == 0) { - if (info.wProductType == VER_NT_WORKSTATION) { - return "Microsoft Windows Vista"; - } - else { - return "Microsoft Windows Server 2008"; - } - } - else if (info.dwMinorVersion == 1) { - if (info.wProductType == VER_NT_WORKSTATION) { - return "Microsoft Windows 7"; - } - else { - return "Microsoft Windows Server 2008 R2"; - } - } - } - #endif - - if (info.dwMajorVersion == 5 && info.dwMinorVersion == 2) { - return "Microsoft Windows Server 2003"; - } - if (info.dwMajorVersion == 5 && info.dwMinorVersion == 1) { - return "Microsoft Windows XP"; - } - if (info.dwMajorVersion == 5 && info.dwMinorVersion == 0) { - return "Microsoft Windows Server 2000"; - } - char buffer[100]; - sprintf(buffer, "Microsoft Windows %d.%d", - info.dwMajorVersion, info.dwMinorVersion); - return buffer; - - default: - break; - } + HKEY key = ArchMiscWindows::openKey(HKEY_LOCAL_MACHINE, windowsVersionKeyNames); + if (key == NULL) { + return osName; } - return "Microsoft Windows "; + + std::string productName = ArchMiscWindows::readValueString(key, "ProductName"); + if (osName.empty()) { + return osName; + } + + return "Microsoft " + productName; } std::string