summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2011-05-13 16:20:01 -0400
committerNick Mathewson <nickm@torproject.org>2011-05-13 16:20:01 -0400
commit09da83e1e8c1f1050fd65af86736abc1f4e5b530 (patch)
tree5f4ae565617321cb88a5c9a944e32a62b8d116d9
parent2253697a04c926edb1a5f18727a6cd24528730b1 (diff)
downloadtor-09da83e1e8c1f1050fd65af86736abc1f4e5b530.tar.gz
tor-09da83e1e8c1f1050fd65af86736abc1f4e5b530.zip
Don't clear out transient addressmap entries on HUP
If you really want to purge the client DNS cache, the TrackHostExits mappings, and the virtual address mappings, you should be using NEWNYM instead. Fixes bug 1345; bugfix on Tor 0.1.0.1-rc. Note that this needs more work: now that we aren't nuking the transient addressmap entries on HUP, we need to make sure that configuration changes to VirtualAddressMap and TrackHostExits actually have a reasonable effect.
-rw-r--r--changes/bug13454
-rw-r--r--src/or/main.c1
2 files changed, 4 insertions, 1 deletions
diff --git a/changes/bug1345 b/changes/bug1345
new file mode 100644
index 0000000000..6945b5fe17
--- /dev/null
+++ b/changes/bug1345
@@ -0,0 +1,4 @@
+ o Minor bugfixes:
+ - On SIGHUP, do not clear out all TrackExitHost mappings, client DNS
+ cache entries, and virtual address mappings: that's what NEWNYM is
+ for. Bugfix on Tor 0.1.0.1-rc; fixes bug 1345.
diff --git a/src/or/main.c b/src/or/main.c
index 15682d5400..d700f0e7a8 100644
--- a/src/or/main.c
+++ b/src/or/main.c
@@ -1417,7 +1417,6 @@ do_hup(void)
router_reset_warnings();
routerlist_reset_warnings();
- addressmap_clear_transient();
/* first, reload config variables, in case they've changed */
if (options->ReloadTorrcOnSIGHUP) {
/* no need to provide argc/v, they've been cached in init_from_config */