Changeset 69541 in vbox for trunk/src/VBox

Timestamp:

Nov 1, 2017 1:20:00 PM (7 years ago)

Author:

vboxsync

Message:

testdriver: temporary heap debug code for windows

Location:

trunk/src/VBox/ValidationKit/testdriver

Files:

• vbox.py (modified) (10 diffs)
• winbase.py (modified) (1 diff)

trunk/src/VBox/ValidationKit/testdriver/vbox.py

@@ -816 +816 @@
         self.fEnableDebugger          = True;
 
+        # TEMPORARY: For process heap checking on windows 2012 boxes.
+        self.fDoHeapChecks = False;
+        if 'COMPUTERNAME' in os.environ and utils.getHostOs() == 'windows':
+            self.fDoHeapChecks = os.environ['COMPUTERNAME'] in [ 'TESTBOXWIN5', 'WEI01-B6KC-4', 'TESTBOXPILE2' ];
+
         # Quietly detect build and validation kit.
         self._detectBuild(False);
@@ -842 +847 @@
         if self.oBuild is not None:
             self.oBuild.dump();
+
+
+    def checkProcessHeap(self):
+        """
+        TEMPORARY: Check the process heap on some Windows 2012 server machines to try catch heap corruption issue.
+        """
+        if self.fDoHeapChecks:
+            if sys.platform == 'win32':
+                from testdriver import winbase;
+                return winbase.checkProcessHeap();
+        return True;
+
 
     def _detectBuild(self, fQuiet = False):
@@ -1417 +1434 @@
         self.oVBox            = None;
         vboxcon.goHackModuleClass.oVBoxMgr = None; # VBoxConstantWrappingHack.
+        self.checkProcessHeap(); ## TEMPORARY
 
         try:
@@ -1439 +1457 @@
             reporter.logXcpt();
         self.fImportedVBoxApi = False;
+        self.checkProcessHeap(); ## TEMPORARY
 
         if self.sHost == 'win':
@@ -1460 +1479 @@
             except:
                 reporter.logXcpt();
+        self.checkProcessHeap(); ## TEMPORARY
 
         try:
@@ -1466 +1486 @@
         except:
             reporter.logXcpt();
+        self.checkProcessHeap(); ## TEMPORARY
         return True;
 
@@ -1833 +1854 @@
         Only Ctrl-C exception, no return.
         """
+        self.checkProcessHeap(); ## TEMPORARY
         try:
             self.oVBoxMgr.waitForEvents(cMsTimeout);
@@ -1839 +1861 @@
         except:
             pass;
+        self.checkProcessHeap(); ## TEMPORARY
         return None;
 
@@ -2121 +2144 @@
         if not self.importVBoxApi():
             return None;
+        self.checkProcessHeap(); ## TEMPORARY
 
         # create + register the VM
@@ -2231 +2255 @@
         reporter.log('created "%s" with name "%s"' % (oVM.id, sName));
         self.aoVMs.append(oVM);
+        self.checkProcessHeap(); ## TEMPORARY
         self.logVmInfo(oVM); # testing...
+        self.checkProcessHeap(); ## TEMPORARY
         return oVM;
     # pylint: enable=R0913,R0914,R0915

trunk/src/VBox/ValidationKit/testdriver/winbase.py

@@ -272 +272 @@
     return True;
 
+def checkProcessHeap():
+    """
+    Calls HeapValidate(GetProcessHeap(), 0, NULL);
+    """
+
+    # Get the process heap.
+    try:
+        hHeap = ctypes.windll.kernel32.GetProcessHeap();
+    except:
+        reporter.logXcpt();
+        return False;
+
+    # Check it.
+    try:
+        fIsOkay = ctypes.windll.kernel32.HeapValidate(hHeap, 0, None);
+    except:
+        reporter.logXcpt();
+        return False;
+
+    if fIsOkay == 0:
+        reporter.log('HeapValidate failed!');
+
+        # Try trigger a dump using c:\utils\procdump64.exe.
+        from common import utils;
+
+        iPid = os.getpid();
+        asArgs = [ 'e:\\utils\\procdump64.exe', '-ma', '%s' % (iPid,), 'c:\\CrashDumps\\python.exe-%u-heap.dmp' % (iPid,)];
+        if utils.getHostArch() != 'amd64':
+            asArgs[0] = 'c:\\utils\\procdump.exe'
+        reporter.log('Trying to dump this process using: %s' % (asArgs,));
+        utils.processCall(asArgs);
+
+        # Generate a crash exception.
+        ctypes.windll.msvcrt.strcpy(None, None, 1024);
+
+    return True;
+
+