aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorteor <teor@torproject.org>2019-11-25 12:36:41 +1000
committerteor <teor@torproject.org>2019-11-25 12:36:41 +1000
commitc17ab20ac0d3efb1b9c46fa12685db5b09015481 (patch)
treefdf1ad8a4712c95021baa48d46bf56964ad09912
parented6f2f61a260a08d9e8f315e4397335fa39383be (diff)
parenta7ccd9a997bc85c0e03926b7d24f7e05a4a51d39 (diff)
downloadtor-c17ab20ac0d3efb1b9c46fa12685db5b09015481.tar.gz
tor-c17ab20ac0d3efb1b9c46fa12685db5b09015481.zip
Merge remote-tracking branch 'tor-github/pr/1441' into maint-0.4.0
-rw-r--r--changes/bug321088
-rw-r--r--src/core/mainloop/mainloop.c2
2 files changed, 9 insertions, 1 deletions
diff --git a/changes/bug32108 b/changes/bug32108
new file mode 100644
index 0000000000..2806fa3e5d
--- /dev/null
+++ b/changes/bug32108
@@ -0,0 +1,8 @@
+ o Major bugfixes (relay):
+ - Relays now respect their AccountingMax bandwidth again. When relays
+ entered "soft" hibernation (which typically starts when we've hit
+ 90% of our AccountingMax), we had stopped checking whether we should
+ enter hard hibernation. Soft hibernation refuses new connections and
+ new circuits, but the existing circuits can continue, meaning that
+ relays could have exceeded their configured AccountingMax. Fixes
+ bug 32108; bugfix on 0.4.0.1-alpha.
diff --git a/src/core/mainloop/mainloop.c b/src/core/mainloop/mainloop.c
index 18e87fa87a..193df61d01 100644
--- a/src/core/mainloop/mainloop.c
+++ b/src/core/mainloop/mainloop.c
@@ -1397,7 +1397,7 @@ STATIC periodic_event_item_t periodic_events[] = {
/* This is a legacy catch-all callback that runs once per second if
* we are online and active. */
CALLBACK(second_elapsed, NET_PARTICIPANT,
- FL(NEED_NET)|FL(RUN_ON_DISABLE)),
+ FL(RUN_ON_DISABLE)),
/* XXXX Do we have a reason to do this on a callback? Does it do any good at
* all? For now, if we're dormant, we can let our listeners decay. */