March 2020 - tor-commits - lists.torproject.org

[tor/release-0.4.3] circpad_setup_machine_on_circ(): exit early on error.
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit 78bcfc1280b322ba57a10a116457616eeb742ab6 Author: Nick Mathewson <nickm(a)torproject.org> Date: Sat Mar 14 14:44:33 2020 -0400 circpad_setup_machine_on_circ(): exit early on error. This function does a nonfatal assertion to make sure that a machine is not registered twice, but Tobias Pulls found a case where it happens. Instead, make the function exit early so that it doesn't cause a remotely triggered memory leak. Fixes bug 33619; bugfix on 0.4.0.1-alpha. This is also tracked as TROVE-2020-004. --- changes/ticket33619 | 5 +++++ src/core/or/circuitpadding.c | 9 ++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/changes/ticket33619 b/changes/ticket33619 new file mode 100644 index 000000000..3c52858b3 --- /dev/null +++ b/changes/ticket33619 @@ -0,0 +1,5 @@ + o Major bugfixes (circuit padding, memory leaks): + - Avoid a remotely triggered memory leak in the case that a circuit + padding machine is somehow negotiated twice on the same circuit. Fixes + bug 33619; bugfix on 0.4.0.1-alpha. Found by Tobias Pulls. This is + also tracked as TROVE-2020-004. diff --git a/src/core/or/circuitpadding.c b/src/core/or/circuitpadding.c index a62cdcf9e..72b770113 100644 --- a/src/core/or/circuitpadding.c +++ b/src/core/or/circuitpadding.c @@ -2381,9 +2381,12 @@ circpad_setup_machine_on_circ(circuit_t *on_circ, return; } - tor_assert_nonfatal(on_circ->padding_machine[machine->machine_index] - == NULL); - tor_assert_nonfatal(on_circ->padding_info[machine->machine_index] == NULL); + IF_BUG_ONCE(on_circ->padding_machine[machine->machine_index] != NULL) { + return; + } + IF_BUG_ONCE(on_circ->padding_info[machine->machine_index] != NULL) { + return; + } /* Log message */ if (CIRCUIT_IS_ORIGIN(on_circ)) {

1 0

[tor/release-0.4.2] Merge branch 'trove_2020_004_041_v2' into maint-0.4.1
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit cec647ff3eab20c97a744a59b808eb49760acfd3 Merge: 5f4e14b8c 78bcfc128 Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 13:56:03 2020 -0400 Merge branch 'trove_2020_004_041_v2' into maint-0.4.1 changes/ticket33619 | 5 +++++ src/core/or/circuitpadding.c | 9 ++++++--- 2 files changed, 11 insertions(+), 3 deletions(-)

1 0

[tor/release-0.4.3] Merge branch 'trove_2020_004_041_v2' into maint-0.4.1
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit cec647ff3eab20c97a744a59b808eb49760acfd3 Merge: 5f4e14b8c 78bcfc128 Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 13:56:03 2020 -0400 Merge branch 'trove_2020_004_041_v2' into maint-0.4.1 changes/ticket33619 | 5 +++++ src/core/or/circuitpadding.c | 9 ++++++--- 2 files changed, 11 insertions(+), 3 deletions(-)

1 0

[tor/release-0.4.3] Merge branch 'maint-0.4.3' into release-0.4.3
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit 7e010cac39af14951a9f0aa17658eef49afd9db5 Merge: 6babe5157 6803373aa Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 13:56:10 2020 -0400 Merge branch 'maint-0.4.3' into release-0.4.3 changes/ticket33619 | 5 +++++ src/core/or/circuitpadding.c | 9 ++++++--- 2 files changed, 11 insertions(+), 3 deletions(-)

1 0

[tor-browser-build/master] Bug 33578: Bump Snowflake version to 58b52eb9f7
by boklm＠torproject.org 17 Mar '20

17 Mar '20

commit bbdb75f2417df66efecf66474bffc047ff557b48 Author: Cecylia Bocovich <cohosh(a)torproject.org> Date: Mon Mar 9 20:26:33 2020 +0000 Bug 33578: Bump Snowflake version to 58b52eb9f7 This commit bumps the version of Snowflake and includes a fix for removing the go.mod and go.sum files from the repository before building. --- projects/snowflake/build | 7 +++++++ projects/snowflake/config | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/projects/snowflake/build b/projects/snowflake/build index 76546d8..fe74e84 100644 --- a/projects/snowflake/build +++ b/projects/snowflake/build @@ -16,6 +16,13 @@ cd /var/tmp/build/[% project %]-[% c('version') %] mkdir -p "$GOPATH/src/git.torproject.org/pluggable-transports" ln -sf "$PWD" "$GOPATH/src/git.torproject.org/pluggable-transports/snowflake.git" +# Remove go.mod and go.sum files until we can build using Go module versioning +rm -f go.mod go.sum + +# Fix gopath location of versioned dependencies +ln -sf "$GOPATH/src/github.com/pion/webrtc" "$GOPATH/src/github.com/pion/webrtc/v2" +ln -sf "$GOPATH/src/github.com/pion/sdp" "$GOPATH/src/github.com/pion/sdp/v2" + cd client go build -ldflags '-s' cp -a client[% IF c("var/windows") %].exe[% END %] $PTDIR/snowflake-client[% IF c("var/windows") %].exe[% END %] diff --git a/projects/snowflake/config b/projects/snowflake/config index d800ee8..bf33032 100644 --- a/projects/snowflake/config +++ b/projects/snowflake/config @@ -1,7 +1,7 @@ # vim: filetype=yaml sw=2 version: '[% c("abbrev") %]' git_url: https://git.torproject.org/pluggable-transports/snowflake.git -git_hash: b4f4b29a037fd2444423835da5999f39525f3d24 +git_hash: 58b52eb9f7987789d8e90a982a6df16013e200dd filename: '[% project %]-[% c("version") %]-[% c("var/osname") %]-[% c("var/build_id") %].tar.gz' var:

1 0

[translation/https_everywhere_completed] https://gitweb.torproject.org/translation.git/commit/?h=https_everywhere_completed
by translation＠torproject.org 17 Mar '20

17 Mar '20

commit 5f7325de1a9d866434091be8a2805265c7fdd2ea Author: Translation commit bot <translation(a)torproject.org> Date: Tue Mar 17 16:16:18 2020 +0000 https://gitweb.torproject.org/translation.git/commit/?h=https_everywhere_co… --- fr/https-everywhere.dtd | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fr/https-everywhere.dtd b/fr/https-everywhere.dtd index ef8089f5a9..1c61620aa2 100644 --- a/fr/https-everywhere.dtd +++ b/fr/https-everywhere.dtd @@ -29,8 +29,8 @@ <!ENTITY https-everywhere.options.userRulesListed "Règles utilisateur de HTTPS partout"> <!ENTITY https-everywhere.options.disabledUrlsListed "Sites désactivés dans HTTPS partout"> <!ENTITY https-everywhere.options.updateChannelsWarning "Avertissement : L’ajout de canaux de mise à jour peut permettre à des assaillants de pirater votre navigateur. Ne modifiez cette section que si vous savez ce que vous faites !"> -<!ENTITY https-everywhere.options.addDisabledSite ""> -<!ENTITY https-everywhere.options.enterDisabledSite ""> +<!ENTITY https-everywhere.options.addDisabledSite "Ajouter un site à désactiver"> +<!ENTITY https-everywhere.options.enterDisabledSite "Saisir un site à Désactiver"> <!ENTITY https-everywhere.options.addUpdateChannel "Ajouter un canal de mise à jour"> <!ENTITY https-everywhere.options.enterUpdateChannelName "Saisir le nom du canal de mise à jour"> <!ENTITY https-everywhere.options.delete "Supprimer"> @@ -44,14 +44,14 @@ <!ENTITY https-everywhere.prefs.reset_defaults "Réinitialiser aux paramètres par défaut"> <!ENTITY https-everywhere.prefs.reset_defaults_message "Chaque ensemble de règles sera réinitialisé à son état par défaut. Poursuivre ?"> -<!ENTITY https-everywhere.cancel.he_blocking_explainer "HTTPS partout a remarqué que vous naviguiez vers une page non HTTPS et a tenté de plutôt vous envoyer vers la version HTTPS. La version HTTPS n’est pas proposée. Ce site ne prend probablement pas HTTPS en charge, mais il est aussi possible qu’un assaillant bloque la version HTTPS. Si vous souhaitez visualiser la version non chiffrée de cette page, vous pouvez quand même le faire en désactivant l’option « Chiffrer tous les sites admissibles (EASE) » dans votre extension HTTPS partout. Sachez que la désactivation de cette option peut rendre votre navigateur vulnérable aux attaques réseau par déclassement contre les sites Web que vous visitez."> +<!ENTITY https-everywhere.cancel.he_blocking_explainer "HTTPS partout a remarqué que vous naviguiez vers une page non HTTPS et a tenté de plutôt vous envoyer vers la version HTTPS. La version HTTPS n’est pas proposée. Ce site ne prend probablement pas HTTPS en charge, mais il est aussi possible qu’un assaillant bloque la version HTTPS. Si vous souhaitez visualiser la version non chiffrée de cette page, vous pouvez quand même le faire en désactivant l’option « Chiffrer tous les sites admissibles (CTSA) » dans votre extension HTTPS partout. Sachez que la désactivation de cette option peut rendre votre navigateur vulnérable aux attaques réseau par déclassement contre les sites Web que vous visitez."> <!ENTITY https-everywhere.cancel.he_blocking_network "attaques de rétrograde réseau"> <!ENTITY https-everywhere.cancel.open_page "Ouvrir la page non sécurisée"> <!ENTITY https-everywhere.cancel.http_once "N’ouvrir la page non sécurisée que pour cette session"> <!ENTITY https-everywhere.onboarding.intro "Chiffrez la Toile ! Utilisez automatiquement la sécurité HTTPS avec de nombreux sites."> -<!ENTITY https-everywhere.onboarding.header "Changement de la version 3.9.2020"> -<!ENTITY https-everywhere.onboarding.details_one ""> +<!ENTITY https-everywhere.onboarding.header "HTTPS partout a été mise à jour"> +<!ENTITY https-everywhere.onboarding.details_one "Optimisation du mode CTSA"> <!ENTITY https-everywhere.onboarding.details_two "Vous pouvez désormais mettre en liste blanche les sites à désactiver dans HTTPS partout"> <!ENTITY https-everywhere.onboarding.footer "HTTPS partout est un projet de la FFÉ">

1 0

[translation/https_everywhere] https://gitweb.torproject.org/translation.git/commit/?h=https_everywhere
by translation＠torproject.org 17 Mar '20

17 Mar '20

commit 86018951e8ca84303b2819dd03013d394da6fb18 Author: Translation commit bot <translation(a)torproject.org> Date: Tue Mar 17 16:16:11 2020 +0000 https://gitweb.torproject.org/translation.git/commit/?h=https_everywhere --- fr/https-everywhere.dtd | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fr/https-everywhere.dtd b/fr/https-everywhere.dtd index ac9c906b9d..1c61620aa2 100644 --- a/fr/https-everywhere.dtd +++ b/fr/https-everywhere.dtd @@ -44,14 +44,14 @@ <!ENTITY https-everywhere.prefs.reset_defaults "Réinitialiser aux paramètres par défaut"> <!ENTITY https-everywhere.prefs.reset_defaults_message "Chaque ensemble de règles sera réinitialisé à son état par défaut. Poursuivre ?"> -<!ENTITY https-everywhere.cancel.he_blocking_explainer "HTTPS partout a remarqué que vous naviguiez vers une page non HTTPS et a tenté de plutôt vous envoyer vers la version HTTPS. La version HTTPS n’est pas proposée. Ce site ne prend probablement pas HTTPS en charge, mais il est aussi possible qu’un assaillant bloque la version HTTPS. Si vous souhaitez visualiser la version non chiffrée de cette page, vous pouvez quand même le faire en désactivant l’option « Chiffrer tous les sites admissibles (CTDA) » dans votre extension HTTPS partout. Sachez que la désactivation de cette option peut rendre votre navigateur vulnérable aux attaques réseau par déclassement contre les sites Web que vous visitez."> +<!ENTITY https-everywhere.cancel.he_blocking_explainer "HTTPS partout a remarqué que vous naviguiez vers une page non HTTPS et a tenté de plutôt vous envoyer vers la version HTTPS. La version HTTPS n’est pas proposée. Ce site ne prend probablement pas HTTPS en charge, mais il est aussi possible qu’un assaillant bloque la version HTTPS. Si vous souhaitez visualiser la version non chiffrée de cette page, vous pouvez quand même le faire en désactivant l’option « Chiffrer tous les sites admissibles (CTSA) » dans votre extension HTTPS partout. Sachez que la désactivation de cette option peut rendre votre navigateur vulnérable aux attaques réseau par déclassement contre les sites Web que vous visitez."> <!ENTITY https-everywhere.cancel.he_blocking_network "attaques de rétrograde réseau"> <!ENTITY https-everywhere.cancel.open_page "Ouvrir la page non sécurisée"> <!ENTITY https-everywhere.cancel.http_once "N’ouvrir la page non sécurisée que pour cette session"> <!ENTITY https-everywhere.onboarding.intro "Chiffrez la Toile ! Utilisez automatiquement la sécurité HTTPS avec de nombreux sites."> -<!ENTITY https-everywhere.onboarding.header "HTTPS Everywhere has been updated!"> -<!ENTITY https-everywhere.onboarding.details_one "Optimisation du mode CTDA"> +<!ENTITY https-everywhere.onboarding.header "HTTPS partout a été mise à jour"> +<!ENTITY https-everywhere.onboarding.details_one "Optimisation du mode CTSA"> <!ENTITY https-everywhere.onboarding.details_two "Vous pouvez désormais mettre en liste blanche les sites à désactiver dans HTTPS partout"> <!ENTITY https-everywhere.onboarding.footer "HTTPS partout est un projet de la FFÉ">

1 0

[tor/master] Split a wide line.
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit bc4c89eb2f47ba0774a05aea3798fbec31f9b920 Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 11:53:01 2020 -0400 Split a wide line. --- src/test/test_hs_control.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/test/test_hs_control.c b/src/test/test_hs_control.c index 566cc21fe..9277711d2 100644 --- a/src/test/test_hs_control.c +++ b/src/test/test_hs_control.c @@ -661,8 +661,10 @@ test_hs_control_add_onion_with_bad_pubkey(void *arg) conn.current_cmd = tor_strdup("ADD_ONION"); } - args = tor_strdup("ED25519-V3:AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA" - "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA Port=9735,127.0.0.1 Flags=DiscardPK"); + args = tor_strdup("ED25519-V3:AAAAAAAAAAAAAAAAAAAAAAAAAAAAA" + "AAAAAAAAAAAAAAAAAAAAAAA" + "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA " + "Port=9735,127.0.0.1 Flags=DiscardPK"); retval = handle_control_command(&conn, (uint32_t) strlen(args), args); tt_int_op(retval, OP_EQ, 0);

1 0

[tor/master] Merge branch 'maint-0.4.3'
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit 7cd4dcf5a6e043cf1d46d9673d164303d69df3de Merge: 270fe0155 5ff8757aa Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 11:48:45 2020 -0400 Merge branch 'maint-0.4.3' src/test/test_hs_control.c | 42 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 42 insertions(+)

1 0

[tor/master] Merge branch 'maint-0.4.3'
by nickm＠torproject.org 17 Mar '20

17 Mar '20

commit 270fe01557cd6f0ca3a795e4801de82a46d87e89 Merge: 44f92e8e4 bbc80ea04 Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Mar 17 11:47:35 2020 -0400 Merge branch 'maint-0.4.3' changes/trove_2020_003 | 4 ++++ src/feature/hs/hs_client.c | 6 +++--- src/feature/hs/hs_service.c | 6 ++++++ src/lib/crypt_ops/crypto_ed25519.c | 2 +- 4 files changed, 14 insertions(+), 4 deletions(-) diff --cc src/feature/hs/hs_client.c index d5c1c5ca9,af8cb0b41..cc1b01d2e --- a/src/feature/hs/hs_client.c +++ b/src/feature/hs/hs_client.c @@@ -1845,10 -1845,10 +1845,10 @@@ hs_client_decode_descriptor(const char hs_descriptor_t **desc) { hs_desc_decode_status_t ret; - uint8_t subcredential[DIGEST256_LEN]; + hs_subcredential_t subcredential; ed25519_public_key_t blinded_pubkey; hs_client_service_authorization_t *client_auth = NULL; - curve25519_secret_key_t *client_auht_sk = NULL; + curve25519_secret_key_t *client_auth_sk = NULL; tor_assert(desc_str); tor_assert(service_identity_pk); @@@ -1869,9 -1869,9 +1869,9 @@@ } /* Parse descriptor */ - ret = hs_desc_decode_descriptor(desc_str, subcredential, + ret = hs_desc_decode_descriptor(desc_str, &subcredential, - client_auht_sk, desc); + client_auth_sk, desc); - memwipe(subcredential, 0, sizeof(subcredential)); + memwipe(&subcredential, 0, sizeof(subcredential)); if (ret != HS_DESC_DECODE_OK) { goto err; }

1 0