From b336e8c74ef996045694ec9c60054729def721c4 Mon Sep 17 00:00:00 2001 From: Kevin Butler Date: Mon, 2 Sep 2013 19:25:08 +0100 Subject: No longer writing control ports to file if updating reversible options fail. Fixes #5605. --- changes/bug5605 | 2 ++ src/or/config.c | 6 +++--- 2 files changed, 5 insertions(+), 3 deletions(-) create mode 100644 changes/bug5605 diff --git a/changes/bug5605 b/changes/bug5605 new file mode 100644 index 000000000..ae66a6e96 --- /dev/null +++ b/changes/bug5605 @@ -0,0 +1,2 @@ +o Minor Bugfixes: + - No longer writing control ports to file if updating reversible options fail. Fixes #5605. \ No newline at end of file diff --git a/src/or/config.c b/src/or/config.c index 657bc6039..cb3dc57d5 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -1115,9 +1115,6 @@ options_act_reversible(const or_options_t *old_options, char **msg) /* No need to roll back, since you can't change the value. */ } - /* Write control ports to disk as appropriate */ - control_ports_write_to_file(); - if (directory_caches_v2_dir_info(options)) { char *fn = NULL; tor_asprintf(&fn, "%s"PATH_SEPARATOR"cached-status", @@ -1315,6 +1312,9 @@ options_act(const or_options_t *old_options) } } + /* Write control ports to disk as appropriate */ + control_ports_write_to_file(); + if (running_tor && !have_lockfile()) { if (try_locking(options, 1) < 0) return -1; -- cgit v1.2.3