From 3c5a2e77138f9ba3f80cbefab69583bdf1a196b2 Mon Sep 17 00:00:00 2001 From: eliz Date: Sat, 18 Apr 2009 08:16:34 +0000 Subject: [PATCH] * 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. --- gdb/ChangeLog | 6 ++++++ gdb/go32-nat.c | 19 ++++++++++++++++--- 2 files changed, 22 insertions(+), 3 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f0cbe8ec26..d24dbb2106 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2009-04-18 Eli Zaretskii + + * 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 * charset.c (_initialize_charset): Add special case for "646". diff --git a/gdb/go32-nat.c b/gdb/go32-nat.c index f5dd8b74da..267df62b34 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); -- 2.11.0