aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorteor <teor@torproject.org>2019-04-19 11:44:38 +1000
committerteor <teor@torproject.org>2019-04-19 11:44:38 +1000
commitb1762f8d1254a89cd181ecf41db666d9bc75fa34 (patch)
treec082de2b95c80c6319570dcea22a1a21e52518a5
parent8154b33f9c01bc8da652849805e4badf531bc4b1 (diff)
parent85c598cbc2d4ca1a7c729864002b5b222980ce19 (diff)
downloadtor-b1762f8d1254a89cd181ecf41db666d9bc75fa34.tar.gz
tor-b1762f8d1254a89cd181ecf41db666d9bc75fa34.zip
Merge remote-tracking branch 'tor-github/pr/638' into maint-0.3.4
-rw-r--r--changes/bug290174
-rw-r--r--doc/tor.1.txt2
-rw-r--r--src/or/router.c11
3 files changed, 10 insertions, 7 deletions
diff --git a/changes/bug29017 b/changes/bug29017
new file mode 100644
index 0000000000..5c4a53c43f
--- /dev/null
+++ b/changes/bug29017
@@ -0,0 +1,4 @@
+ o Minor bugfixes (stats):
+ - When ExtraInfoStatistics is 0, stop including PaddingStatistics in
+ relay and bridge extra-info documents. Fixes bug 29017;
+ bugfix on 0.3.1.1-alpha.
diff --git a/doc/tor.1.txt b/doc/tor.1.txt
index 6dd1b60b23..3f7a8c39af 100644
--- a/doc/tor.1.txt
+++ b/doc/tor.1.txt
@@ -2278,7 +2278,7 @@ is non-zero):
extra-info document. (Default: 0)
[[PaddingStatistics]] **PaddingStatistics** **0**|**1**::
- Relays only.
+ Relays and bridges only.
When this option is enabled, Tor collects statistics for padding cells
sent and received by this relay, in addition to total cell counts.
These statistics are rounded, and omitted if traffic is low. This
diff --git a/src/or/router.c b/src/or/router.c
index 03959e907d..2f7d8349f9 100644
--- a/src/or/router.c
+++ b/src/or/router.c
@@ -3421,12 +3421,11 @@ extrainfo_dump_to_string(char **s_out, extrainfo_t *extrainfo,
"conn-bi-direct", now, &contents) > 0) {
smartlist_add(chunks, contents);
}
- }
-
- if (options->PaddingStatistics) {
- contents = rep_hist_get_padding_count_lines();
- if (contents)
- smartlist_add(chunks, contents);
+ if (options->PaddingStatistics) {
+ contents = rep_hist_get_padding_count_lines();
+ if (contents)
+ smartlist_add(chunks, contents);
+ }
}
/* Add information about the pluggable transports we support. */