summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2018-04-24 10:36:17 -0400
committerNick Mathewson <nickm@torproject.org>2018-04-24 10:36:17 -0400
commit9187cdb1cd17dd988c6e444705c6153a65948f1e (patch)
treec2532f37769f19d1c54640c7c96dfe8b9ad91550
parentd2951b381bcbd45798b6cf8f01c5ce7a4d9ad4b9 (diff)
parentb259008c56c44c32dfeea3ed8d4b06cd61751a88 (diff)
downloadtor-9187cdb1cd17dd988c6e444705c6153a65948f1e.tar.gz
tor-9187cdb1cd17dd988c6e444705c6153a65948f1e.zip
Merge remote-tracking branch 'dgoulet/bug25901_032_01' into maint-0.3.2
-rw-r--r--changes/bug259013
-rw-r--r--src/or/hs_service.c4
2 files changed, 7 insertions, 0 deletions
diff --git a/changes/bug25901 b/changes/bug25901
new file mode 100644
index 0000000000..3ad30c7676
--- /dev/null
+++ b/changes/bug25901
@@ -0,0 +1,3 @@
+ o Minor bugfixes (hidden service v3):
+ - Fix a memory leak when an hidden service v3 is configured and gets a
+ SIGHUP signal. Fixes bug 25901; bugfix on 0.3.2.1-alpha.
diff --git a/src/or/hs_service.c b/src/or/hs_service.c
index 45810c5c5f..8ddd68c69e 100644
--- a/src/or/hs_service.c
+++ b/src/or/hs_service.c
@@ -840,6 +840,10 @@ move_hs_state(hs_service_t *src_service, hs_service_t *dst_service)
/* Let's do a shallow copy */
dst->intro_circ_retry_started_time = src->intro_circ_retry_started_time;
dst->num_intro_circ_launched = src->num_intro_circ_launched;
+ /* Freeing a NULL replaycache triggers an info LD_BUG. */
+ if (dst->replay_cache_rend_cookie != NULL) {
+ replaycache_free(dst->replay_cache_rend_cookie);
+ }
dst->replay_cache_rend_cookie = src->replay_cache_rend_cookie;
src->replay_cache_rend_cookie = NULL; /* steal pointer reference */