5 files changed, 22 insertions, 11 deletions
diff --git a/src/tests/main.cpp b/src/tests/main.cpp
index 5cf4e4fe3..0e0360783 100644
--- a/src/tests/main.cpp
+++ b/src/tests/main.cpp
@@ -60,7 +60,7 @@ int main(int argc, char* argv[])
       {
       const std::string arg_spec =
          "botan-test --verbose --help --data-dir= --pkcs11-lib= --provider= "
-         "--log-success --abort-on-first-fail --avoid-undefined "
+         "--log-success --abort-on-first-fail --no-avoid-undefined "
          "--run-long-tests --run-online-tests --test-runs=1 --drbg-seed= "
          "*suites";
 
@@ -85,7 +85,7 @@ int main(int argc, char* argv[])
          parser.flag_set("run-online-tests"),
          parser.flag_set("run-long-tests"),
          parser.flag_set("abort-on-first-fail"),
-         parser.flag_set("avoid-undefined"));
+         parser.flag_set("no-avoid-undefined"));
 
 #if defined(BOTAN_HAS_OPENSSL)
       if(opts.provider().empty() || opts.provider() == "openssl")
diff --git a/src/tests/test_dl_group.cpp b/src/tests/test_dl_group.cpp
index 10a7e3bc6..c9e654bd7 100644
--- a/src/tests/test_dl_group.cpp
+++ b/src/tests/test_dl_group.cpp
@@ -44,16 +44,19 @@ class DL_Group_Tests final : public Test
                             "DL_Group uninitialized",
                             []() { Botan::DL_Group dl; dl.get_p(); });
 
-         if(Test::avoid_undefined_behavior() == false)
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
+         if(Test::no_avoid_undefined_behavior())
             {
+#endif
             result.test_throws("Bad generator param",
                                "Invalid argument DL_Group unknown PrimeType",
                                []() {
                                auto invalid_type = static_cast<Botan::DL_Group::PrimeType>(9);
                                Botan::DL_Group dl(Test::rng(), invalid_type, 1024);
                                });
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
             }
-
+#endif
          return result;
          }
 
diff --git a/src/tests/test_ffi.cpp b/src/tests/test_ffi.cpp
index 58b9852a0..0d831f251 100644
--- a/src/tests/test_ffi.cpp
+++ b/src/tests/test_ffi.cpp
@@ -842,14 +842,18 @@ class FFI_Unit_Tests final : public Test
          // delete of null is ok/ignored
          TEST_FFI_RC(0, botan_hash_destroy, (nullptr));
 
-         if(Test::avoid_undefined_behavior() == false)
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
+         if(Test::no_avoid_undefined_behavior())
             {
+#endif
             // Confirm that botan_x_destroy checks the argument type
             botan_mp_t mp;
             botan_mp_init(&mp);
             TEST_FFI_RC(BOTAN_FFI_ERROR_INVALID_OBJECT, botan_hash_destroy, (reinterpret_cast<botan_hash_t>(mp)));
             TEST_FFI_RC(0, botan_mp_destroy, (mp));
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
             }
+#endif
 
          return result;
          }
diff --git a/src/tests/tests.h b/src/tests/tests.h
index 0c63797fa..56aa4b8c5 100644
--- a/src/tests/tests.h
+++ b/src/tests/tests.h
@@ -62,7 +62,7 @@ class Test_Options
                    bool run_online_tests,
                    bool run_long_tests,
                    bool abort_on_first_fail,
-                   bool avoid_undefined) :
+                   bool no_avoid_undefined) :
          m_requested_tests(requested_tests),
          m_data_dir(data_dir),
          m_pkcs11_lib(pkcs11_lib),
@@ -73,7 +73,7 @@ class Test_Options
          m_run_online_tests(run_online_tests),
          m_run_long_tests(run_long_tests),
          m_abort_on_first_fail(abort_on_first_fail),
-         m_avoid_undefined(avoid_undefined)
+         m_no_avoid_undefined(no_avoid_undefined)
          {}
 
       const std::vector<std::string>& requested_tests() const
@@ -97,7 +97,7 @@ class Test_Options
 
       bool abort_on_first_fail() const { return m_abort_on_first_fail; }
 
-      bool avoid_undefined_behavior() const { return m_avoid_undefined; }
+      bool no_avoid_undefined_behavior() const { return m_no_avoid_undefined; }
 
    private:
       std::vector<std::string> m_requested_tests;
@@ -110,7 +110,7 @@ class Test_Options
       bool m_run_online_tests;
       bool m_run_long_tests;
       bool m_abort_on_first_fail;
-      bool m_avoid_undefined;
+      bool m_no_avoid_undefined;
    };
 
 /*
@@ -466,7 +466,7 @@ class Test
 
       static void set_test_rng(std::unique_ptr<Botan::RandomNumberGenerator> rng);
 
-      static bool avoid_undefined_behavior() { return m_opts.avoid_undefined_behavior(); }
+      static bool no_avoid_undefined_behavior() { return m_opts.no_avoid_undefined_behavior(); }
       static bool log_success() { return m_opts.log_success(); }
       static bool run_online_tests() { return m_opts.run_online_tests(); }
       static bool run_long_tests() { return m_opts.run_long_tests(); }
diff --git a/src/tests/unit_ecdsa.cpp b/src/tests/unit_ecdsa.cpp
index dc2a43af5..f94f445ee 100644
--- a/src/tests/unit_ecdsa.cpp
+++ b/src/tests/unit_ecdsa.cpp
@@ -299,14 +299,18 @@ Test::Result test_encoding_options()
    result.test_eq("Hybrid point same size as uncompressed",
                   enc_uncompressed.size(), enc_hybrid.size());
 
-   if(Test::avoid_undefined_behavior() == false)
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
+   if(Test::no_avoid_undefined_behavior())
       {
+#endif
       auto invalid_format = static_cast<Botan::PointGFp::Compression_Type>(99);
 
       result.test_throws("Invalid point format throws",
                          "Invalid argument Invalid point encoding for EC_PublicKey",
                          [&] { key.set_point_encoding(invalid_format); });
+#if defined(BOTAN_HAS_SANITIZER_UNDEFINED)
       }
+#endif
 
    return result;
    }