From d97adfda2868aeb9e62df96216203e8120a95d6d Mon Sep 17 00:00:00 2001 From: Kevin Cadieux Date: Fri, 19 Mar 2021 13:54:05 -0700 Subject: [PATCH] memleaktest with MSVC's AddressSanitizer Disabling memleaktest under MSVC because leak detection is not a supported feature with MSVC's AddressSanitizer. Leaving ASan enabled in this case causes a test failure because the test suite is expecting the leak to be detected. CLA: trivial Reviewed-by: Shane Lontis Reviewed-by: Tomas Mraz (Merged from https://github.com/openssl/openssl/pull/14581) --- test/memleaktest.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/memleaktest.c b/test/memleaktest.c index 1e07723a6f1..b48dbced0d4 100644 --- a/test/memleaktest.c +++ b/test/memleaktest.c @@ -20,7 +20,10 @@ # endif #endif /* If __SANITIZE_ADDRESS__ isn't defined, define it to be false */ -#ifndef __SANITIZE_ADDRESS__ +/* Leak detection is not yet supported with MSVC on Windows, so */ +/* set __SANITIZE_ADDRESS__ to false in this case as well. */ +#if !defined(__SANITIZE_ADDRESS__) || defined(_MSC_VER) +# undef __SANITIZE_ADDRESS__ # define __SANITIZE_ADDRESS__ 0 #endif -- 2.47.2