Browse Source

dtrt when only relaybandwidthburst is set

fixes bug 2470
Roger Dingledine 13 years ago
parent
commit
20b75989ac
2 changed files with 7 additions and 0 deletions
  1. 5 0
      changes/bug2470
  2. 2 0
      src/or/config.c

+ 5 - 0
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.

+ 2 - 0
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 "