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

[tor/master] Merge remote-tracking branch 'dgoulet/bug24895_029_02' into maint-0.2.9
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit a2aaf9509ba578f4e7705b506ee9a0f764d24ff2 Merge: 36567c5ca 490ae26b2 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:29:15 2018 -0500 Merge remote-tracking branch 'dgoulet/bug24895_029_02' into maint-0.2.9 changes/bug24895 | 8 ++++++++ src/or/rendservice.c | 29 ++++++++++++++++++++++++----- 2 files changed, 32 insertions(+), 5 deletions(-)

1 0

[tor/maint-0.3.2] Merge branch 'maint-0.3.1' into maint-0.3.2
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit 129456e22257d24c380bed0f9ebc9d68955a01df Merge: de3cf40fa 7f2e23e7a Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:50 2018 -0500 Merge branch 'maint-0.3.1' into maint-0.3.2

1 0

[tor/maint-0.3.1] Merge remote-tracking branch 'dgoulet/bug24895_031_02' into maint-0.3.1
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit 00a42277eaa5838e5b7d0b07a53db01780232511 Merge: 483a59ec2 f98f7ca89 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:02 2018 -0500 Merge remote-tracking branch 'dgoulet/bug24895_031_02' into maint-0.3.1 changes/bug24895 | 8 ++++++++ src/or/hs_common.c | 17 +++++++++++++++++ src/or/hs_common.h | 1 + src/or/hs_service.c | 1 + src/or/rendservice.c | 10 ++++++++-- 5 files changed, 35 insertions(+), 2 deletions(-)

1 0

[tor/maint-0.3.2] Merge remote-tracking branch 'dgoulet/bug24895_029_02' into maint-0.2.9
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit a2aaf9509ba578f4e7705b506ee9a0f764d24ff2 Merge: 36567c5ca 490ae26b2 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:29:15 2018 -0500 Merge remote-tracking branch 'dgoulet/bug24895_029_02' into maint-0.2.9 changes/bug24895 | 8 ++++++++ src/or/rendservice.c | 29 ++++++++++++++++++++++++----- 2 files changed, 32 insertions(+), 5 deletions(-)

1 0

[tor/master] Merge remote-tracking branch 'dgoulet/bug24895_031_02' into maint-0.3.1
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit 00a42277eaa5838e5b7d0b07a53db01780232511 Merge: 483a59ec2 f98f7ca89 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:02 2018 -0500 Merge remote-tracking branch 'dgoulet/bug24895_031_02' into maint-0.3.1 changes/bug24895 | 8 ++++++++ src/or/hs_common.c | 17 +++++++++++++++++ src/or/hs_common.h | 1 + src/or/hs_service.c | 1 + src/or/rendservice.c | 10 ++++++++-- 5 files changed, 35 insertions(+), 2 deletions(-)

1 0

[tor/master] Merge branch 'maint-0.3.1' into maint-0.3.2
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit 129456e22257d24c380bed0f9ebc9d68955a01df Merge: de3cf40fa 7f2e23e7a Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:50 2018 -0500 Merge branch 'maint-0.3.1' into maint-0.3.2

1 0

[tor/master] Merge branch 'maint-0.3.0' into maint-0.3.1
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit 7f2e23e7acc0735e5ae933535a1b617a0860e528 Merge: 00a42277e b39c50cde Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:12 2018 -0500 Merge branch 'maint-0.3.0' into maint-0.3.1 "ours" merge to avoid maint-0.3.0's version of 24895.

1 0

[tor/master] Merge remote-tracking branch 'dgoulet/bug24895_032_02' into maint-0.3.2
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit de3cf40fa658d62a61bb620c9dba8e90105da339 Merge: e2aacf100 f870f9c8b Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:30:44 2018 -0500 Merge remote-tracking branch 'dgoulet/bug24895_032_02' into maint-0.3.2 changes/bug24895 | 8 ++++++++ src/or/hs_circuit.c | 7 ++++++- src/or/hs_common.c | 17 +++++++++++++++++ src/or/hs_common.h | 2 ++ src/or/rendservice.c | 4 ++-- 5 files changed, 35 insertions(+), 3 deletions(-)

1 0

[tor/master] Merge branch 'maint-0.2.9' into maint-0.3.0
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit b39c50cde8b1d7e3e27d5a6fc2e58ff208982637 Merge: 1cf11b694 a2aaf9509 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Jan 19 16:29:43 2018 -0500 Merge branch 'maint-0.2.9' into maint-0.3.0 Conflicts: src/or/rendservice.c changes/bug24895 | 8 ++++++++ src/or/rendservice.c | 26 ++++++++++++++++++++++++-- 2 files changed, 32 insertions(+), 2 deletions(-)

1 0

[tor/release-0.2.9] MAX_REND_FAILURES is 1, but we would try three times
by nickm＠torproject.org 19 Jan '18

19 Jan '18

commit a15eb9ff439623f800de813c1a78eeb5d61f7f5a Author: Roger Dingledine <arma(a)torproject.org> Date: Mon Jan 15 16:30:55 2018 -0500 MAX_REND_FAILURES is 1, but we would try three times Fix an "off by 2" error in counting rendezvous failures on the onion service side. While we thought we would stop the rendezvous attempt after one failed circuit, we were actually making three circuit attempts before giving up. Fixes bug 24895; bugfix on 0.0.6. --- changes/bug24895 | 6 ++++++ src/or/rendservice.c | 5 ++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/changes/bug24895 b/changes/bug24895 new file mode 100644 index 000000000..7b90f6d26 --- /dev/null +++ b/changes/bug24895 @@ -0,0 +1,6 @@ + o Major bugfixes (onion services): + - Fix an "off by 2" error in counting rendezvous failures on the + onion service side. While we thought we would stop the rendezvous + attempt after one failed circuit, we were actually making three + circuit attempts before giving up. Fixes bug 24895; bugfix on 0.0.6. + diff --git a/src/or/rendservice.c b/src/or/rendservice.c index a8c383444..acc431a57 100644 --- a/src/or/rendservice.c +++ b/src/or/rendservice.c @@ -2930,8 +2930,11 @@ rend_service_relaunch_rendezvous(origin_circuit_t *oldcirc) } oldcirc->hs_service_side_rend_circ_has_been_relaunched = 1; + /* We check failure_count >= MAX_REND_FAILURES-1 below rather than + * failure_count >= MAX_REND_FAILURES, because we increment the failure + * count for our current failure *after* this clause. */ if (!oldcirc->build_state || - oldcirc->build_state->failure_count > MAX_REND_FAILURES || + oldcirc->build_state->failure_count >= MAX_REND_FAILURES-1 || oldcirc->build_state->expiry_time < time(NULL)) { log_info(LD_REND, "Attempt to build circuit to %s for rendezvous has failed "

1 0