commenting out an assert that could be triggered on Win7 (#57524)
authorVladimir Sadov <vsadov@microsoft.com>
Tue, 17 Aug 2021 17:21:27 +0000 (10:21 -0700)
committerGitHub <noreply@github.com>
Tue, 17 Aug 2021 17:21:27 +0000 (10:21 -0700)
src/coreclr/vm/threadsuspend.cpp

index 8359706..57318e2 100644 (file)
@@ -1985,7 +1985,9 @@ CONTEXT* AllocateOSContextHelper(BYTE** contextBuffer)
         pfnInitializeContext2(NULL, context, NULL, &contextSize, xStateCompactionMask) :
         InitializeContext(NULL, context, NULL, &contextSize);
 
-    _ASSERTE(!success && GetLastError() == ERROR_INSUFFICIENT_BUFFER);
+    // The following assert is valid, but gets triggered in some Win7 runs with no impact on functionality.
+    // commenting this out to reduce noise, as long as Win7 is supported.
+    // _ASSERTE(!success && GetLastError() == ERROR_INSUFFICIENT_BUFFER);
 
     // So now allocate a buffer of that size and call InitializeContext again
     BYTE* buffer = new (nothrow)BYTE[contextSize];