February 2018 - tor-commits - lists.torproject.org

[tor/release-0.3.2] sched: When releasing a channel, do not BUG() if absent from the pending list
by nickm＠torproject.org 02 Feb '18

02 Feb '18

commit 005e228f80f0bd241d01c106f87e6bfe6dda6127 Author: David Goulet <dgoulet(a)torproject.org> Date: Fri Feb 2 08:48:34 2018 -0500 sched: When releasing a channel, do not BUG() if absent from the pending list The current code flow makes it that we can release a channel in a PENDING state but not in the pending list. This happens while the channel is being processed in the scheduler loop. Fixes #25125 Signed-off-by: David Goulet <dgoulet(a)torproject.org> --- src/or/scheduler.c | 26 +++++++++++++++----------- src/test/test_scheduler.c | 12 ++++++++++++ 2 files changed, 27 insertions(+), 11 deletions(-) diff --git a/src/or/scheduler.c b/src/or/scheduler.c index 47141c538..1984084fe 100644 --- a/src/or/scheduler.c +++ b/src/or/scheduler.c @@ -628,17 +628,21 @@ scheduler_release_channel,(channel_t *chan)) return; } - if (chan->scheduler_state == SCHED_CHAN_PENDING) { - if (SCHED_BUG(smartlist_pos(channels_pending, chan) == -1, chan)) { - log_warn(LD_SCHED, "Scheduler asked to release channel %" PRIu64 " " - "but it wasn't in channels_pending", - chan->global_identifier); - } else { - smartlist_pqueue_remove(channels_pending, - scheduler_compare_channels, - offsetof(channel_t, sched_heap_idx), - chan); - } + /* Try to remove the channel from the pending list regardless of its + * scheduler state. We can release a channel in many places in the tor code + * so we can't rely on the channel state (PENDING) to remove it from the + * list. + * + * For instance, the channel can change state from OPEN to CLOSING while + * being handled in the scheduler loop leading to the channel being in + * PENDING state but not in the pending list. Furthermore, we release the + * channel when it changes state to close and a second time when we free it. + * Not ideal at all but for now that is the way it is. */ + if (chan->sched_heap_idx != -1) { + smartlist_pqueue_remove(channels_pending, + scheduler_compare_channels, + offsetof(channel_t, sched_heap_idx), + chan); } if (the_scheduler->on_channel_free) { diff --git a/src/test/test_scheduler.c b/src/test/test_scheduler.c index 1f014c4f6..724a6b56b 100644 --- a/src/test/test_scheduler.c +++ b/src/test/test_scheduler.c @@ -502,6 +502,18 @@ perform_channel_state_tests(int KISTSchedRunInterval, int sched_type) scheduler_touch_channel(ch1); tt_assert(scheduler_compare_channels_mock_ctr > old_count); + /* Release the ch2 and then do it another time to make sure it doesn't blow + * up and we are still in a quiescent state. */ + scheduler_release_channel(ch2); + tt_int_op(ch2->scheduler_state, OP_EQ, SCHED_CHAN_IDLE); + tt_int_op(smartlist_len(channels_pending), OP_EQ, 1); + /* Cheat a bit so make the release more confused but also will tells us if + * the release did put the channel in the right state. */ + ch2->scheduler_state = SCHED_CHAN_PENDING; + scheduler_release_channel(ch2); + tt_int_op(ch2->scheduler_state, OP_EQ, SCHED_CHAN_IDLE); + tt_int_op(smartlist_len(channels_pending), OP_EQ, 1); + /* Close */ channel_mark_for_close(ch1); tt_int_op(ch1->state, OP_EQ, CHANNEL_STATE_CLOSING);

1 0

[tor/master] Merge branch 'maint-0.3.2'
by nickm＠torproject.org 02 Feb '18

02 Feb '18

commit 9e48338a12fd1fef840dc49c702b4a628af67cc0 Merge: 0fd91772b 4ceae7c6b Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Feb 2 12:03:54 2018 -0500 Merge branch 'maint-0.3.2' src/or/scheduler.c | 26 +++++++++++++++----------- src/test/test_scheduler.c | 12 ++++++++++++ 2 files changed, 27 insertions(+), 11 deletions(-)

1 0

[tor/release-0.3.2] Merge branch 'bug25125_032_01_squashed' into maint-0.3.2
by nickm＠torproject.org 02 Feb '18

02 Feb '18

commit 4ceae7c6b9742ce6b14f8bed34725e44522e52f2 Merge: 77634795b 005e228f8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Feb 2 12:03:48 2018 -0500 Merge branch 'bug25125_032_01_squashed' into maint-0.3.2 src/or/scheduler.c | 26 +++++++++++++++----------- src/test/test_scheduler.c | 12 ++++++++++++ 2 files changed, 27 insertions(+), 11 deletions(-)

1 0

[tor/release-0.3.2] Merge branch 'maint-0.3.2' into release-0.3.2
by nickm＠torproject.org 02 Feb '18

02 Feb '18

commit f5e37b2eb6b33185a30a2aadd513c7e4dfbc1379 Merge: 90716e3f9 4ceae7c6b Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Feb 2 12:03:54 2018 -0500 Merge branch 'maint-0.3.2' into release-0.3.2 src/or/scheduler.c | 26 +++++++++++++++----------- src/test/test_scheduler.c | 12 ++++++++++++ 2 files changed, 27 insertions(+), 11 deletions(-)

1 0

[user-manual/alpha] Pushing es-ES screenshots
by colin＠torproject.org 02 Feb '18

02 Feb '18

commit 8106cb034d5f349d202f51e201a37589fe98a336 Author: Colin Childs <colin(a)torproject.org> Date: Fri Feb 2 10:04:18 2018 -0600 Pushing es-ES screenshots --- es-ES/media/circumvention/bridges.png | Bin 117487 -> 223850 bytes es-ES/media/circumvention/configure.png | Bin 74740 -> 290268 bytes 2 files changed, 0 insertions(+), 0 deletions(-) diff --git a/es-ES/media/circumvention/bridges.png b/es-ES/media/circumvention/bridges.png index ce03eca..0b51401 100644 Binary files a/es-ES/media/circumvention/bridges.png and b/es-ES/media/circumvention/bridges.png differ diff --git a/es-ES/media/circumvention/configure.png b/es-ES/media/circumvention/configure.png index 33773e8..fc56ec6 100644 Binary files a/es-ES/media/circumvention/configure.png and b/es-ES/media/circumvention/configure.png differ

1 0

[user-manual/alpha] Pushing de images
by colin＠torproject.org 02 Feb '18

02 Feb '18

commit fa2f468d53f83387001e8ce97ad32c88c9cc083b Author: Colin Childs <colin(a)torproject.org> Date: Fri Feb 2 08:19:22 2018 -0600 Pushing de images --- de/media/circumvention/bridges.png | Bin 116636 -> 207233 bytes de/media/circumvention/configure.png | Bin 82220 -> 181646 bytes 2 files changed, 0 insertions(+), 0 deletions(-) diff --git a/de/media/circumvention/bridges.png b/de/media/circumvention/bridges.png index be471d9..34cf698 100644 Binary files a/de/media/circumvention/bridges.png and b/de/media/circumvention/bridges.png differ diff --git a/de/media/circumvention/configure.png b/de/media/circumvention/configure.png index 354490d..6a90c68 100644 Binary files a/de/media/circumvention/configure.png and b/de/media/circumvention/configure.png differ

1 0

[tor-browser/tor-browser-52.6.0esr-8.0-2] Bug 13575: Disable randomised Firefox HTTP cache decay user tests.
by gk＠torproject.org 02 Feb '18

02 Feb '18

commit f6d6512bf85d964fa5a470f2528042c527523a34 Author: Fernando Fernandez Mancera <ffmancera(a)riseup.net> Date: Sat Jan 20 21:54:00 2018 +0100 Bug 13575: Disable randomised Firefox HTTP cache decay user tests. Mozilla's experiment shouldn't be collecting any data on our users. If "browser.cache.frecency_experiment" is not set in firefox.js to "-1" it will default to "0" and then it is randomised between "1" and "4" inclusive, setting different HTTP cache decay times for the four groups. --- browser/app/profile/000-tor-browser.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/browser/app/profile/000-tor-browser.js b/browser/app/profile/000-tor-browser.js index 1302d2e2d6be..ca27504e4b7d 100644 --- a/browser/app/profile/000-tor-browser.js +++ b/browser/app/profile/000-tor-browser.js @@ -179,6 +179,9 @@ pref("media.webspeech.synth.enabled", false); // Bug 10283: Disable SpeechSynthe pref("dom.webaudio.enabled", false); // Bug 13017: Disable Web Audio API pref("dom.maxHardwareConcurrency", 1); // Bug 21675: Spoof single-core cpu pref("dom.w3c_touch_events.enabled", 0); // Bug 10286: Always disable Touch API +// Disable randomised Firefox HTTP cache decay user test groups (Bug: 13575) +pref("browser.cache.frecency_experiment", -1); + // Third party stuff pref("privacy.firstparty.isolate", true); // Always enforce first party isolation

1 0

[tor-browser-build/master] Fold in stable Changelog
by gk＠torproject.org 02 Feb '18

02 Feb '18

commit e15653dc249011b6448e585d96e6f0aef3f1f262 Author: Georg Koppen <gk(a)torproject.org> Date: Fri Feb 2 08:17:35 2018 +0000 Fold in stable Changelog --- .../tor-browser/Bundle-Data/Docs/ChangeLog.txt | 51 ++++++++++++++++++++++ 1 file changed, 51 insertions(+) diff --git a/projects/tor-browser/Bundle-Data/Docs/ChangeLog.txt b/projects/tor-browser/Bundle-Data/Docs/ChangeLog.txt index 5ee6e37..d591af5 100644 --- a/projects/tor-browser/Bundle-Data/Docs/ChangeLog.txt +++ b/projects/tor-browser/Bundle-Data/Docs/ChangeLog.txt @@ -21,6 +21,57 @@ Tor Browser 8.0a1 -- January 23 2018 * Bug 23892: Include Firefox and Tor debug files in final build directory * Bug 24842: include libasan.so.2 and libubsan.so.0 in debug builds +Tor Browser 7.5 -- January 23 2018 + * All Platforms + * Update Firefox to 52.6.0esr + * Update Tor to 0.3.2.9 + * Update OpenSSL to 1.0.2n + * Update Torbutton to 1.9.8.5 + * Bug 21847: Update copy for security slider + * Bug 21245: Add da translation to Torbutton and keep track of it + * Bug 24702: Remove Mozilla text from banner + * Bug 10573: Replace deprecated nsILocalFile with nsIFile (code clean-up) + * Translations update + * Update Tor Launcher to 0.2.14.3 + * Bug 23262: Implement integrated progress bar + * Bug 23261: implement configuration portion of new Tor Launcher UI + * Bug 24623: Revise "country that censors Tor" text + * Bug 24624: tbb-logo.svg may cause network access + * Bug 23240: Retrieve current bootstrap progress before showing progress bar + * Bug 24428: Bootstrap error message sometimes lost + * Bug 22232: Add README on use of bootstrap status messages + * Bug 10573: Replace deprecated nsILocalFile with nsIFile (code clean-up) + * Translations update + * Update HTTPS Everywhere to 2018.1.11 + * Update NoScript to 5.1.8.3 + * Bug 23104: CSS line-height reveals the platform Tor Browser is running on + * Bug 24398: Plugin-container process exhausts memory + * Bug 22501: Requests via javascript: violate FPI + * Bug 24756: Add noisebridge01 obfs4 bridge configuration + * Windows + * Bug 16010: Enable content sandboxing on Windows + * Bug 23230: Fix build error on Windows 64 + * OS X + * Bug 24566: Avoid white flashes when opening dialogs in Tor Browser + * Bug 23025: Add some hardening flags to macOS build + * Linux + * Bug 23970: Make "Print to File" work with sandboxing enabled + * Bug 23016: "Print to File" is broken on some non-english Linux systems + * Bug 10089: Set middlemouse.contentLoadURL to false by default + * Bug 18101: Suppress upload file dialog proxy bypass (linux part) + * Android + * Bug 22084: Spoof network information API + * Build System + * All Platforms + * Switch from gitian/tor-browser-bundle to rbm/tor-browser-build + * Windows + * Bug 22563: Update mingw-w64 to fix W^X violations + * Bug 20929: Bump GCC version to 5.4.0 + * Linux + * Bug 20929: Bump GCC version to 5.4.0 + * Bug 23892: Include Firefox and Tor debug files in final build directory + * Bug 24842: include libasan.so.2 and libubsan.so.0 in debug builds + Tor Browser 7.5a10 -- December 19 2017 * All Platforms * Update Tor to 0.3.2.7-rc

1 0

[tor-browser-build/maint-7.5] Bump rbm to latest master
by gk＠torproject.org 02 Feb '18

02 Feb '18

commit e0641719c6f52109a25304dbe438e3139d6e14f1 Author: Georg Koppen <gk(a)torproject.org> Date: Fri Feb 2 07:22:04 2018 +0000 Bump rbm to latest master --- rbm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rbm b/rbm index 5c89374..7494edc 160000 --- a/rbm +++ b/rbm @@ -1 +1 @@ -Subproject commit 5c89374df7957238a9565c0b938ee3f30f880438 +Subproject commit 7494edc6d2556c511c213823a6549410ba75f73b

1 0

[torbutton/master] Merge remote-tracking branch 'arthur/25016'
by gk＠torproject.org 02 Feb '18

02 Feb '18

commit f96a293d146d3dc54f8ecfaa1d3dfc669bf4198d Merge: f860499 1cf0f11 Author: Georg Koppen <gk(a)torproject.org> Date: Fri Feb 2 08:11:02 2018 +0000 Merge remote-tracking branch 'arthur/25016' src/chrome/content/aboutTor/aboutTor-content.js | 5 -- src/chrome/content/aboutTor/aboutTor.xhtml | 19 +--- src/chrome/content/aboutTor/donation_banner.js | 105 ---------------------- src/chrome/content/aboutTor/onion-hand.png | Bin 69055 -> 0 bytes src/chrome/content/torbutton.js | 4 +- src/chrome/skin/donation_banner.css | 113 ----------------------- src/modules/donation-banner.js | 115 ------------------------ 7 files changed, 2 insertions(+), 359 deletions(-)

1 0