dtrt when only relaybandwidthburst is set

fixes bug 2470

diff --git a/changes/bug2470 b/changes/bug2470
new file mode 100644 (file)
index 0000000..8ff97b7
--- /dev/null
+++ b/changes/bug2470
@@ -0,0 +1,5 @@
+  o Major bugfixes:
+    - If relays set RelayBandwidthBurst but not RelayBandwidthRate,
+      Tor would ignore their RelayBandwidthBurst setting,
+      potentially using more bandwidth than expected. Bugfix on
+      0.2.0.1-alpha. Reported by Paul Wouters. Fixes bug 2470.

diff --git a/src/or/config.c b/src/or/config.c
index f8cfd29f84e000e68ae2ad447272f7ea807fff21..8066a23a4a57ab26754e4fd06b4f9eb33934c76f 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -3401,6 +3401,8 @@ options_validate(or_options_t *old_options, or_options_t *options,
 
   if (options->RelayBandwidthRate && !options->RelayBandwidthBurst)
     options->RelayBandwidthBurst = options->RelayBandwidthRate;
+  if (options->RelayBandwidthBurst && !options->RelayBandwidthRate)
+    options->RelayBandwidthRate = options->RelayBandwidthBurst;
 
   if (options->RelayBandwidthRate > options->RelayBandwidthBurst)
     REJECT("RelayBandwidthBurst must be at least equal "