summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2019-01-04 14:57:06 -0500
committerNick Mathewson <nickm@torproject.org>2019-01-04 14:57:06 -0500
commitfe3331b84554c6d6e2e633d77de02453b9a73f88 (patch)
tree270201a0d08528abe1f545953dae90da470d37b3
parentd47c9276de69ee206670844b4032965fbb497d3b (diff)
parentbf8bb9c526a7dc99ab56b6e4786737f4617bb4ed (diff)
downloadtor-fe3331b84554c6d6e2e633d77de02453b9a73f88.tar.gz
tor-fe3331b84554c6d6e2e633d77de02453b9a73f88.zip
Merge branch 'maint-0.3.3' into maint-0.3.4
-rw-r--r--changes/bug286196
-rw-r--r--src/or/hs_service.c6
2 files changed, 10 insertions, 2 deletions
diff --git a/changes/bug28619 b/changes/bug28619
new file mode 100644
index 0000000000..86be8cb2fb
--- /dev/null
+++ b/changes/bug28619
@@ -0,0 +1,6 @@
+ o Minor bugfixes (hidden service v3):
+ - When deleting an ephemeral onion service (DEL_ONION), do not close any
+ rendezvous circuits in order to let the existing client connections
+ finish by themselves or closed by the application. The HS v2 is doing
+ that already so now we have the same behavior for all versions. Fixes
+ bug 28619; bugfix on 0.3.3.1-alpha.
diff --git a/src/or/hs_service.c b/src/or/hs_service.c
index 1b6316dabe..54ceb9dcad 100644
--- a/src/or/hs_service.c
+++ b/src/or/hs_service.c
@@ -3140,8 +3140,10 @@ hs_service_del_ephemeral(const char *address)
goto err;
}
- /* Close circuits, remove from map and finally free. */
- close_service_circuits(service);
+ /* Close introduction circuits, remove from map and finally free. Notice
+ * that the rendezvous circuits aren't closed in order for any existing
+ * connections to finish. We let the application terminate them. */
+ close_service_intro_circuits(service);
remove_service(hs_service_map, service);
hs_service_free(service);