diff options
author | Nick Mathewson <nickm@torproject.org> | 2019-10-22 14:01:42 -0400 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2019-10-22 14:01:42 -0400 |
commit | 41e6e2f496dd89a710ba8641dafa1996de69adeb (patch) | |
tree | f04e33f4e10e972a304ffd76c4790a9572489aa5 /src/test/test_config.c | |
parent | 8a27023d211aa62a456237d41766b825fe91aa9b (diff) | |
download | tor-41e6e2f496dd89a710ba8641dafa1996de69adeb.tar.gz tor-41e6e2f496dd89a710ba8641dafa1996de69adeb.zip |
Use coccinelle to fix most options_validate() call sites.
There was one that it could not find because it was in a macro definition.
I used the following semantic patch:
@@
expression e1, e2, e3, e4, e5;
@@
options_validate(e1,
e2,
- e3,
- e4,
e5)
Diffstat (limited to 'src/test/test_config.c')
-rw-r--r-- | src/test/test_config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/test_config.c b/src/test/test_config.c index cbb84e4dcf..ea5f3f538d 100644 --- a/src/test/test_config.c +++ b/src/test/test_config.c @@ -913,7 +913,7 @@ test_config_fix_my_family(void *arg) options_init(defaults); options->MyFamily_lines = family; - options_validate(NULL, options, defaults, 0, &err) ; + options_validate(NULL, options, &err) ; if (err != NULL) { TT_FAIL(("options_validate failed: %s", err)); @@ -5649,7 +5649,7 @@ test_config_check_bridge_distribution_setting_not_a_bridge(void *arg) options->BridgeRelay = 0; options->BridgeDistribution = (char*)("https"); - ret = options_validate(old_options, options, default_options, 0, &message); + ret = options_validate(old_options, options, &message); tt_int_op(ret, OP_EQ, -1); tt_str_op(message, OP_EQ, "You set BridgeDistribution, but you " |