From: Eli Zaretskii <eliz@gnu.org>
Date: Sat, 18 Apr 2009 08:16:34 +0000 (+0000)
Subject: 	* go32-nat.c (go32_sysinfo): Check if the call to
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d647eed6d56c7cf8cfa4fb35e610e7d7433760b8;p=binutils-gdb.git

	* go32-nat.c (go32_sysinfo): Check if the call to
	__dpmi_get_capabilities fills the buffer with information, and
	don't use the buffer if not.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index f0cbe8ec26f..d24dbb21063 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2009-04-18  Eli Zaretskii  <eliz@gnu.org>
+
+	* go32-nat.c (go32_sysinfo): Check if the call to
+	__dpmi_get_capabilities fills the buffer with information, and
+	don't use the buffer if not.
+
 2009-04-17  Tom Tromey  <tromey@redhat.com>
 
 	* charset.c (_initialize_charset): Add special case for "646".
diff --git a/gdb/go32-nat.c b/gdb/go32-nat.c
index f5dd8b74da0..267df62b343 100644
--- a/gdb/go32-nat.c
+++ b/gdb/go32-nat.c
@@ -983,6 +983,10 @@ print_mem (unsigned long datum, const char *header, int in_pages_p)
 static void
 go32_sysinfo (char *arg, int from_tty)
 {
+  static const char test_pattern[] =
+    "deadbeafdeadbeafdeadbeafdeadbeafdeadbeaf"
+    "deadbeafdeadbeafdeadbeafdeadbeafdeadbeaf"
+    "deadbeafdeadbeafdeadbeafdeadbeafdeadbeafdeadbeaf";
   struct utsname u;
   char cpuid_vendor[13];
   unsigned cpuid_max = 0, cpuid_eax, cpuid_ebx, cpuid_ecx, cpuid_edx;
@@ -990,8 +994,7 @@ go32_sysinfo (char *arg, int from_tty)
   unsigned advertized_dos_version = ((unsigned int)_osmajor << 8) | _osminor;
   int dpmi_flags;
   char dpmi_vendor_info[129];
-  int dpmi_vendor_available =
-    __dpmi_get_capabilities (&dpmi_flags, dpmi_vendor_info);
+  int dpmi_vendor_available;
   __dpmi_version_ret dpmi_version_data;
   long eflags;
   __dpmi_free_mem_info mem_info;
@@ -1218,7 +1221,15 @@ go32_sysinfo (char *arg, int from_tty)
   else if (true_dos_version == 0x532 && advertized_dos_version == 0x500)
     printf_filtered ("Windows Version................Windows NT family (W2K/XP/W2K3/Vista/W2K8)\n");
   puts_filtered ("\n");
-  if (dpmi_vendor_available == 0)
+  /* On some versions of Windows, __dpmi_get_capabilities returns
+     zero, but the buffer is not filled with info, so we fill the
+     buffer with a known pattern and test for it afterwards.  */
+  memcpy (dpmi_vendor_info, test_pattern, sizeof(dpmi_vendor_info));
+  dpmi_vendor_available =
+    __dpmi_get_capabilities (&dpmi_flags, dpmi_vendor_info);
+  if (dpmi_vendor_available == 0
+      && memcmp (dpmi_vendor_info, test_pattern,
+		 sizeof(dpmi_vendor_info)) != 0)
     {
       /* The DPMI spec says the vendor string should be ASCIIZ, but
 	 I don't trust the vendors to follow that...  */
@@ -1230,6 +1241,8 @@ go32_sysinfo (char *arg, int from_tty)
 		       (unsigned)dpmi_vendor_info[1],
 		       ((unsigned)dpmi_flags & 0x7f));
     }
+  else
+    printf_filtered ("DPMI Host......................(Info not available)\n");
   __dpmi_get_version (&dpmi_version_data);
   printf_filtered ("DPMI Version...................%d.%02d\n",
 		   dpmi_version_data.major, dpmi_version_data.minor);