Fix Windows check

cisco · Nov 2, 2023 · 5d5b52a · 5d5b52a
1 parent b043ae1
commit 5d5b52a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/bytes/test/bytes.cpp b/lib/bytes/test/bytes.cpp
@@ -10,8 +10,9 @@ using namespace std::literals::string_literals;
 // To check that memory is safely zeroized on destroy, we have to deliberately
 // do a use-after-free.  This will be caught by the sanitizers, so we only do it
 // when sanitizers are not enabled.  This test is also disabled on Windows
-// because the use-after-free causes Windows CI runs to fail.
-#if !defined(SANITIZERS) || defined(WINDOWS)
+// because it appears to cause Windows CI runs to fail.  (In addition, Windows
+// appears to overwrite freed buffers with 0xCD, so this test is unnecessary.)
+#if !defined(SANITIZERS) && !defined(WINDOWS)
 TEST_CASE("Zeroization")
 {
   const auto size = size_t(1024);