diff options
| author | Nick Mathewson <nickm@torproject.org> | 2019-09-10 10:29:34 -0400 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2019-09-11 09:42:19 -0400 |
| commit | 478141e617a333ac4d998e3747c073246e04b5ae (patch) | |
| tree | 4ca7717dde9277c56dad60ddcea355b5740b80b0 /src/app | |
| parent | e61bfd0bfd1acfdd4f5d5912c5cc2c0e8ec61b75 (diff) | |
| download | tor-478141e617a333ac4d998e3747c073246e04b5ae.tar.gz tor-478141e617a333ac4d998e3747c073246e04b5ae.zip | |
Document inconsistent usage of config_var_is_listable()
See also ticket 31654.
Diffstat (limited to 'src/app')
| -rw-r--r-- | src/app/config/config.c | 7 | ||||
| -rw-r--r-- | src/app/config/confparse.c | 6 |
2 files changed, 13 insertions, 0 deletions
diff --git a/src/app/config/config.c b/src/app/config/config.c index 55e82934ec..72a1190b5d 100644 --- a/src/app/config/config.c +++ b/src/app/config/config.c @@ -2666,6 +2666,9 @@ list_torrc_options(void) { smartlist_t *vars = config_mgr_list_vars(get_options_mgr()); SMARTLIST_FOREACH_BEGIN(vars, const config_var_t *, var) { + /* Possibly this should check listable, rather than (or in addition to) + * settable. See ticket 31654. + */ if (! config_var_is_settable(var)) { /* This variable cannot be set, or cannot be set by this name. */ continue; @@ -2680,6 +2683,8 @@ static void list_deprecated_options(void) { smartlist_t *deps = config_mgr_list_deprecated_vars(get_options_mgr()); + /* Possibly this should check whether the variables are listable, + * but currently it does not. See ticket 31654. */ SMARTLIST_FOREACH(deps, const char *, name, printf("%s\n", name)); smartlist_free(deps); @@ -8142,6 +8147,8 @@ getinfo_helper_config(control_connection_t *conn, } else if (!strcmp(question, "config/defaults")) { smartlist_t *sl = smartlist_new(); int dirauth_lines_seen = 0, fallback_lines_seen = 0; + /* Possibly this should check whether the variables are listable, + * but currently it does not. See ticket 31654. */ smartlist_t *vars = config_mgr_list_vars(get_options_mgr()); SMARTLIST_FOREACH_BEGIN(vars, const config_var_t *, var) { if (var->initvalue != NULL) { diff --git a/src/app/config/confparse.c b/src/app/config/confparse.c index 930484620b..9f7fdc5c35 100644 --- a/src/app/config/confparse.c +++ b/src/app/config/confparse.c @@ -592,6 +592,12 @@ config_var_needs_copy(const config_var_t *var) /** * Return true iff variable <b>var</b> should appear on list of variable * names given to the controller or the CLI. + * + * (Note that this option is imperfectly obeyed. The + * --list-torrc-options command looks at the "settable" flag, whereas + * "GETINFO config/defaults" and "list_deprecated_*()" do not filter + * their results. It would be good for consistency to try to converge + * these behaviors in the future.) **/ bool config_var_is_listable(const config_var_t *var) |