tor-commits September 2014

tor-commits@lists.torproject.org

22 participants
980 discussions

[tor/master] Merge remote-tracking branch 'origin/maint-0.2.5'
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit 2914d56ea4c767bbfa956afba64032c90a27f4eb Merge: 121f4a9 6215ebb Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Sep 11 08:59:24 2014 -0400 Merge remote-tracking branch 'origin/maint-0.2.5' changes/bug13124 | 8 ++++++++ src/or/config.c | 4 ++-- 2 files changed, 10 insertions(+), 2 deletions(-)

1 0

[tor/master] Reduce log severity for unused ClientTransportPlugin lines
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit 6215ebb26676e3fa5d18ddce114f273d9a5dee2a Author: Roger Dingledine <arma(a)torproject.org> Date: Thu Sep 11 08:02:37 2014 -0400 Reduce log severity for unused ClientTransportPlugin lines Tor Browser includes several ClientTransportPlugin lines in its torrc-defaults file, leading every Tor Browser user who looks at her logs to see these notices and wonder if they're dangerous. Resolves bug 13124; bugfix on 0.2.5.3-alpha. --- changes/bug13124 | 8 ++++++++ src/or/config.c | 4 ++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/changes/bug13124 b/changes/bug13124 new file mode 100644 index 0000000..be7df70 --- /dev/null +++ b/changes/bug13124 @@ -0,0 +1,8 @@ + o Minor bugfixes: + - Reduce the log severity of the "Pluggable transport proxy does + not provide any needed transports and will not be launched." + message, since Tor Browser includes several ClientTransportPlugin + lines in its torrc-defaults file, leading every Tor Browser user + who looks at her logs to see these notices and wonder if they're + dangerous. Resolves bug 13124; bugfix on 0.2.5.3-alpha. + diff --git a/src/or/config.c b/src/or/config.c index c4fc8dd..31a3586 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -4827,8 +4827,8 @@ parse_client_transport_line(const or_options_t *options, if (is_managed) { /* managed */ if (!validate_only && is_useless_proxy) { - log_notice(LD_GENERAL, "Pluggable transport proxy (%s) does not provide " - "any needed transports and will not be launched.", line); + log_info(LD_GENERAL, "Pluggable transport proxy (%s) does not provide " + "any needed transports and will not be launched.", line); } /* If we are not just validating, use the rest of the line as the

1 0

[tor/maint-0.2.5] Reduce log severity for unused ClientTransportPlugin lines
by nickm＠torproject.org 11 Sep '14

11 Sep '14

1 0

[tor/master] Merge remote-tracking branch 'public/bug13104_025'
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit 121f4a9ca43fa88282778adac4597fac8d9deafc Merge: 48558ed d02937a Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Sep 11 08:30:41 2014 -0400 Merge remote-tracking branch 'public/bug13104_025' src/test/test_util.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-)

1 0

[tor/master] Fix "comparison is always false" warnings in new test_util_di_ops
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit d02937a20399a3b1c49a3503cabc5b07dc6b5e99 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Sep 11 08:28:46 2014 -0400 Fix "comparison is always false" warnings in new test_util_di_ops Having a constant zero means that unsigned < 0 is always false. --- src/test/test_util.c | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/test/test_util.c b/src/test/test_util.c index e105435..09a27b7 100644 --- a/src/test/test_util.c +++ b/src/test/test_util.c @@ -3278,20 +3278,24 @@ test_util_di_ops(void) * against each possible (single-byte) bit difference * some arithmetic bugs only appear with certain bit patterns */ { - const uint8_t z = 0; + uint8_t zz = 0; uint8_t ii = 0; + int z; for (i = 0; i < 256; i++) { ii = (uint8_t)i; - test_eq(tor_memeq(&z, &ii, 1), z == ii); + test_eq(tor_memeq(&zz, &ii, 1), zz == ii); } /* exhaustively white-box test tor_memcmp * against each possible single-byte numeric difference * some arithmetic bugs only appear with certain bit patterns */ - for (i = 0; i < 256; i++) { - ii = (uint8_t)i; - test_eq(tor_memcmp(&z, &ii, 1) > 0 ? GT : EQ, z > ii ? GT : EQ); - test_eq(tor_memcmp(&ii, &z, 1) < 0 ? LT : EQ, ii < z ? LT : EQ); + for (z = 0; z < 256; z++) { + for (i = 0; i < 256; i++) { + ii = (uint8_t)i; + zz = (uint8_t)z; + test_eq(tor_memcmp(&zz, &ii, 1) > 0 ? GT : EQ, zz > ii ? GT : EQ); + test_eq(tor_memcmp(&ii, &zz, 1) < 0 ? LT : EQ, ii < zz ? LT : EQ); + } } }

1 0

[tor-browser-bundle/master] No need to apply the patch for bug 8402 anymore.
by gk＠torproject.org 11 Sep '14

11 Sep '14

commit 416e13ed4e32164d9fa56173cd517c4b2be45b6e Author: Georg Koppen <gk(a)torproject.org> Date: Thu Sep 11 13:10:53 2014 +0000 No need to apply the patch for bug 8402 anymore. Bug 8402 landed on tor master recently which breaks compilation if we'd still apply the patch for it when building nightlies. --- gitian/descriptors/linux/gitian-tor.yml | 2 -- gitian/descriptors/mac/gitian-tor.yml | 2 -- gitian/descriptors/windows/gitian-tor.yml | 2 -- 3 files changed, 6 deletions(-) diff --git a/gitian/descriptors/linux/gitian-tor.yml b/gitian/descriptors/linux/gitian-tor.yml index bb99d02..3678913 100644 --- a/gitian/descriptors/linux/gitian-tor.yml +++ b/gitian/descriptors/linux/gitian-tor.yml @@ -87,8 +87,6 @@ script: | then git am ~/build/bug8402-master.patch git am ~/build/bug11200-hang-0.2.5.patch - else - git am ~/build/bug8402-master.patch fi fi mkdir -p $OUTDIR/src diff --git a/gitian/descriptors/mac/gitian-tor.yml b/gitian/descriptors/mac/gitian-tor.yml index 7a75287..88fb40f 100644 --- a/gitian/descriptors/mac/gitian-tor.yml +++ b/gitian/descriptors/mac/gitian-tor.yml @@ -82,8 +82,6 @@ script: | then git am ~/build/bug8402-master.patch git am ~/build/bug11200-hang-0.2.5.patch - else - git am ~/build/bug8402-master.patch fi fi mkdir -p $OUTDIR/src diff --git a/gitian/descriptors/windows/gitian-tor.yml b/gitian/descriptors/windows/gitian-tor.yml index 277f2d0..ad218f5 100644 --- a/gitian/descriptors/windows/gitian-tor.yml +++ b/gitian/descriptors/windows/gitian-tor.yml @@ -82,8 +82,6 @@ script: | then git am ~/build/bug8402-master.patch git am ~/build/bug11200-hang-0.2.5.patch - else - git am ~/build/bug8402-master.patch fi fi mkdir -p $OUTDIR/src

1 0

[tor-browser-bundle/master] Merge remote-tracking branch 'pc/bug13091'
by gk＠torproject.org 11 Sep '14

11 Sep '14

commit e21bb4837a7affd469e413a2664f4a2bb5c2de4e Merge: b5f07db 41b90ea Author: Georg Koppen <gk(a)torproject.org> Date: Thu Sep 11 10:38:02 2014 +0000 Merge remote-tracking branch 'pc/bug13091' gitian/build-helpers/fix-info-plist.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

1 0

[tor-browser-bundle/master] Bug 13091: Use "Tor Browser" everywhere (space included)
by gk＠torproject.org 11 Sep '14

11 Sep '14

commit 41b90eae7d3d1a3664bc0d482862961a8ed3fcad Author: Kathy Brade <brade(a)pearlcrescent.com> Date: Wed Sep 10 09:52:33 2014 -0400 Bug 13091: Use "Tor Browser" everywhere (space included) Use "TorBrowser" instead "Tor Browser" in Info.plist on Mac OS. --- gitian/build-helpers/fix-info-plist.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gitian/build-helpers/fix-info-plist.py b/gitian/build-helpers/fix-info-plist.py index ccb6c68..ac531ec 100755 --- a/gitian/build-helpers/fix-info-plist.py +++ b/gitian/build-helpers/fix-info-plist.py @@ -25,7 +25,7 @@ COPYRIGHT = "Tor Browser %s Copyright %s The Tor Project" % (TORBROWSER_VERSION, plist = plistlib.readPlist(sys.stdin) -plist["CFBundleGetInfoString"] = "TorBrowser %s" % TORBROWSER_VERSION +plist["CFBundleGetInfoString"] = "Tor Browser %s" % TORBROWSER_VERSION plist["CFBundleShortVersionString"] = TORBROWSER_VERSION plist["NSHumanReadableCopyright"] = COPYRIGHT

1 0

[tor/master] Merge remote-tracking branch 'public/bug13104_025'
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit 48558ed1aa070ef121b339eef88e7822f8e45978 Merge: 73ee161 2491ead Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Sep 11 00:11:26 2014 -0400 Merge remote-tracking branch 'public/bug13104_025' changes/bug13104 | 4 + src/common/util.c | 31 ++++++-- src/or/routerlist.c | 5 +- src/test/test_dir.c | 27 +++++++ src/test/test_util.c | 211 ++++++++++++++++++++++++++++++++++++++++++++++---- 5 files changed, 256 insertions(+), 22 deletions(-) diff --cc src/or/routerlist.c index a018181,0df411b..1faa05f --- a/src/or/routerlist.c +++ b/src/or/routerlist.c @@@ -1804,10 -1802,10 +1804,10 @@@ scale_array_elements_to_u64(u64_dbl_t * uint64_t *total_out) { double total = 0.0; - double scale_factor; + double scale_factor = 0.0; int i; /* big, but far away from overflowing an int64_t */ -#define SCALE_TO_U64_MAX (INT64_MAX / 4) +#define SCALE_TO_U64_MAX ((int64_t) (INT64_MAX / 4)) for (i = 0; i < n_entries; ++i) total += entries[i].dbl;

1 0

[tor/master] C90 compliance for #13104 fixes
by nickm＠torproject.org 11 Sep '14

11 Sep '14

commit 2491eadf002e993bee11aa76597ec7f361d8f6e5 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Sep 11 00:10:53 2014 -0400 C90 compliance for #13104 fixes --- src/test/test_util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/test/test_util.c b/src/test/test_util.c index eed37ca..e105435 100644 --- a/src/test/test_util.c +++ b/src/test/test_util.c @@ -3277,6 +3277,7 @@ test_util_di_ops(void) /* exhaustively white-box test tor_memeq * against each possible (single-byte) bit difference * some arithmetic bugs only appear with certain bit patterns */ + { const uint8_t z = 0; uint8_t ii = 0; for (i = 0; i < 256; i++) { @@ -3292,6 +3293,7 @@ test_util_di_ops(void) test_eq(tor_memcmp(&z, &ii, 1) > 0 ? GT : EQ, z > ii ? GT : EQ); test_eq(tor_memcmp(&ii, &z, 1) < 0 ? LT : EQ, ii < z ? LT : EQ); } + } tt_int_op(1, ==, safe_mem_is_zero("", 0)); tt_int_op(1, ==, safe_mem_is_zero("", 1));

1 0

← Newer
1
...
58
59
60
61
62
63
64
...
98
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

tor-commits September 2014