Merge pull request #2020 from tomjepp/master
[mono.git] / mono / metadata / environment.c
index 00ec7dc24b0308a7ff51c9bb3a03aca3df202a65..e0d1970f61279e87374354b4bc734610cd344c9d 100644 (file)
@@ -18,7 +18,7 @@
 #include <mono/utils/mono-compiler.h>
 #include <mono/io-layer/io-layer.h>
 
-extern MonoString* ves_icall_System_Environment_GetOSVersionString (void) MONO_INTERNAL;
+extern MonoString* ves_icall_System_Environment_GetOSVersionString (void);
 
 #if !defined(HOST_WIN32) && defined(HAVE_SYS_UTSNAME_H)
 #include <sys/utsname.h>
@@ -41,26 +41,23 @@ MonoString*
 ves_icall_System_Environment_GetOSVersionString (void)
 {
 #ifdef HOST_WIN32
-       OSVERSIONINFO verinfo;
+       OSVERSIONINFOEX verinfo;
 
-       MONO_ARCH_SAVE_REGS;
-
-       verinfo.dwOSVersionInfoSize = sizeof (OSVERSIONINFO);
-       if (GetVersionEx (&verinfo)) {
-               char version [64];
-               /* maximum string length is 35 bytes 
-                  3 x 10 bytes per number, 1 byte for 0, 3 x 1 byte for dots, 1 for NULL */
-               sprintf (version, "%ld.%ld.%ld.0", 
-                       verinfo.dwMajorVersion,
-                       verinfo.dwMinorVersion,
-                       verinfo.dwBuildNumber);
+       verinfo.dwOSVersionInfoSize = sizeof (OSVERSIONINFOEX);
+       if (GetVersionEx ((OSVERSIONINFO*)&verinfo)) {
+               char version [128];
+               /* maximum string length is 45 bytes
+                  4 x 10 bytes per number, 1 byte for 0, 3 x 1 byte for dots, 1 for NULL */
+               sprintf (version, "%ld.%ld.%ld.%d",
+                                verinfo.dwMajorVersion,
+                                verinfo.dwMinorVersion,
+                                verinfo.dwBuildNumber,
+                                verinfo.wServicePackMajor << 16);
                return mono_string_new (mono_domain_get (), version);
        }
 #elif defined(HAVE_SYS_UTSNAME_H)
        struct utsname name;
 
-       MONO_ARCH_SAVE_REGS;
-
        if (uname (&name) >= 0) {
                return mono_string_new (mono_domain_get (), name.release);
        }