[master] 8e0c6264a varnishtest: Rename the no_sanitizer feature

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Wed Jun 2 06:30:09 UTC 2021


commit 8e0c6264a27acc8715415c8af90c1bc0dbefde94
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Wed Jun 2 08:15:16 2021 +0200

    varnishtest: Rename the no_sanitizer feature

diff --git a/bin/varnishtest/tests/v00004.vtc b/bin/varnishtest/tests/v00004.vtc
index 2f2b75459..8760279e5 100644
--- a/bin/varnishtest/tests/v00004.vtc
+++ b/bin/varnishtest/tests/v00004.vtc
@@ -1,7 +1,7 @@
 varnishtest "test if our default paramers make sense ..."
 
 feature 64bit
-feature no_sanitizer
+feature !sanitizer
 
 # Skip this test in GCOV mode, 68xx bytes extra per level makes it fail
 feature cmd {test -z "$GCOVPROG"}
diff --git a/bin/varnishtest/tests/v00041.vtc b/bin/varnishtest/tests/v00041.vtc
index 6c83a9f4d..68ba1b0bd 100644
--- a/bin/varnishtest/tests/v00041.vtc
+++ b/bin/varnishtest/tests/v00041.vtc
@@ -1,6 +1,6 @@
 varnishtest "Test priv_task"
 
-feature no_sanitizer
+feature !sanitizer
 
 server s1 {
 	rxreq
diff --git a/bin/varnishtest/vtc_misc.c b/bin/varnishtest/vtc_misc.c
index b962bedc9..a46254246 100644
--- a/bin/varnishtest/vtc_misc.c
+++ b/bin/varnishtest/vtc_misc.c
@@ -420,10 +420,10 @@ ipvx_works(const char *target)
  *        recognized as a macro.
  * persistent_storage
  *        Varnish was built with the deprecated persistent storage.
- * no_sanitizer
- *        Varnish was not built with a sanitizer.
+ * sanitizer
+ *        Varnish was built with a sanitizer.
  *
- * A feature name can be prepended with an exclamation mark (!) to skip a
+ * A feature name can be prefixed with an exclamation mark (!) to skip a
  * test if a feature is present.
  *
  * Be careful with ignore_unknown_macro, because it may cause a test with a
@@ -438,9 +438,9 @@ static const unsigned with_persistent_storage = 0;
 #endif
 
 #if __SANITIZER
-static const unsigned no_sanitizer = 0;
+static const unsigned sanitizer = 1;
 #else
-static const unsigned no_sanitizer = 1;
+static const unsigned sanitizer = 0;
 #endif
 
 #ifdef SO_RCVTIMEO_WORKS
@@ -495,7 +495,7 @@ cmd_feature(CMD_ARGS)
 		FEATURE("user_vcache", getpwnam("vcache") != NULL);
 		FEATURE("group_varnish", getgrnam("varnish") != NULL);
 		FEATURE("persistent_storage", with_persistent_storage);
-		FEATURE("no_sanitizer", no_sanitizer);
+		FEATURE("sanitizer", sanitizer);
 		FEATURE("SO_RCVTIMEO_WORKS", so_rcvtimeo_works);
 
 		if (!strcmp(feat, "disable_aslr")) {


More information about the varnish-commit mailing list