nihilist/tor
1
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/core/tor.git synced 2024-11-24 04:13:28 +01:00
Code Issues Packages Projects Releases Wiki Activity

config.c: Turn options_transition_allowed() into a proper callback

Browse Source
This commit is contained in:
Nick Mathewson 2019-10-24 09:16:50 -04:00
parent bd891f517f
commit d17aa09308
1 changed files with 11 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/app/config/config.c
View File

@ -823,8 +823,8 @@ static const config_deprecation_t option_deprecation_notes_[] = {
static char *get_windows_conf_root(void);
#endif
static int options_act_reversible(const or_options_t *old_options, char **msg);
static int options_transition_allowed(const or_options_t *old,
const or_options_t *new,
static int options_check_transition_cb(const void *old,
const void *new,
char **msg);
static int options_transition_affects_workers(
const or_options_t *old_options, const or_options_t *new_options);
@ -877,6 +877,7 @@ static const config_format_t options_format = {
.deprecations = option_deprecation_notes_,
.vars = option_vars_,
.legacy_validate_fn = options_validate_cb,
.check_transition_fn = options_check_transition_cb,
.clear_fn = options_clear_cb,
.config_suite_offset = offsetof(or_options_t, subconfigs_),
};
@ -3240,11 +3241,6 @@ options_validate_and_set(const or_options_t *old_options,
rv = SETOPT_ERR_PARSE;
goto err;
}
if (options_transition_allowed(old_options, new_options, msg_out) < 0) {
rv = SETOPT_ERR_TRANSITION;
goto err;
}
in_option_validation = 0;
if (set_options(new_options, msg_out)) {
@ -4821,11 +4817,14 @@ opt_streq(const char *s1, const char *s2)
/** Check if any of the previous options have changed but aren't allowed to. */
static int
options_transition_allowed(const or_options_t *old,
const or_options_t *new_val,
options_check_transition_cb(const void *old_,
const void *new_val_,
char **msg)
{
if (!old)
const or_options_t *old = old_;
const or_options_t *new_val = new_val_;
if (BUG(!old))
return 0;
#define BAD_CHANGE_TO(opt, how) do { \