From 3b770d8b1bef5c63607f100345428cc8c8fb4b31 Mon Sep 17 00:00:00 2001 From: tavplubix Date: Sun, 8 Mar 2020 12:32:22 +0300 Subject: [PATCH] Fix stateless tests with msan (#9531) * try run tests * try fix build * try enable other libraries * suppress some msan warnings * Update msan_suppressions.txt * Update msan_suppressions.txt * use function names in suppressions list * update submodule --- cmake/sanitize.cmake | 21 --------------------- contrib/openssl | 2 +- dbms/tests/msan_suppressions.txt | 9 +++++++++ 3 files changed, 10 insertions(+), 22 deletions(-) diff --git a/cmake/sanitize.cmake b/cmake/sanitize.cmake index 13947425f7b..3d192f1fe76 100644 --- a/cmake/sanitize.cmake +++ b/cmake/sanitize.cmake @@ -35,27 +35,6 @@ if (SANITIZE) set (CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -static-libmsan") endif () - # Temporarily disable many external libraries that don't work under - # MemorySanitizer yet. - set (ENABLE_HDFS 0 CACHE BOOL "") - set (ENABLE_CAPNP 0 CACHE BOOL "") - set (ENABLE_RDKAFKA 0 CACHE BOOL "") - set (ENABLE_POCO_MONGODB 0 CACHE BOOL "") - set (ENABLE_POCO_NETSSL 0 CACHE BOOL "") - set (ENABLE_POCO_ODBC 0 CACHE BOOL "") - set (ENABLE_ODBC 0 CACHE BOOL "") - set (ENABLE_MYSQL 0 CACHE BOOL "") - set (ENABLE_EMBEDDED_COMPILER 0 CACHE BOOL "") - set (USE_INTERNAL_CAPNP_LIBRARY 0 CACHE BOOL "") - set (USE_SIMDJSON 0 CACHE BOOL "") - set (ENABLE_ORC 0 CACHE BOOL "") - set (ENABLE_PARQUET 0 CACHE BOOL "") - set (USE_CAPNP 0 CACHE BOOL "") - set (USE_INTERNAL_ORC_LIBRARY 0 CACHE BOOL "") - set (USE_ORC 0 CACHE BOOL "") - set (USE_AVRO 0 CACHE BOOL "") - set (ENABLE_SSL 0 CACHE BOOL "") - elseif (SANITIZE STREQUAL "thread") set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${SAN_FLAGS} -fsanitize=thread") set (CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${SAN_FLAGS} -fsanitize=thread") diff --git a/contrib/openssl b/contrib/openssl index debbae80cb4..07e96230645 160000 --- a/contrib/openssl +++ b/contrib/openssl @@ -1 +1 @@ -Subproject commit debbae80cb44de55fd8040fdfbe4b506601ff2a6 +Subproject commit 07e9623064508d15dd61367f960ebe7fc9aecd77 diff --git a/dbms/tests/msan_suppressions.txt b/dbms/tests/msan_suppressions.txt index b3db90b9123..0ceef2d834c 100644 --- a/dbms/tests/msan_suppressions.txt +++ b/dbms/tests/msan_suppressions.txt @@ -4,3 +4,12 @@ fun:__gxx_personality_* # We apply std::tolower to uninitialized padding, but don't use the result, so # it is OK. Reproduce with "select ngramDistanceCaseInsensitive(materialize(''), '')" fun:tolower + +# May be it's not OK, but suppress it to run other tests +# Some functions in OpenSSL: +fun:probable_prime +fun:BN_bin2bn +fun:BN_add_word +fun:bn_div_fixed_top +fun:bn_mul_words +fun:BN_cmp