tor icon indicating copy to clipboard operation
tor copied to clipboard

fix for circpad_add_matching_machines to be able to negotiate more machines

Open pylls opened this issue 6 years ago • 2 comments

As part of circuitpadding.c, in circpad_add_matching_machines(), the macros FOR_EACH_CIRCUIT_MACHINE_BEGIN and SMARTLIST_FOREACH_REVERSE_BEGIN currently expand to a for loop each. The outer loop goes over each machine index (currently 2, set by CIRCPAD_MAX_MACHINES), while the inner loop looks for a suitable machine for that index to negotiate. As soon as one is found and negotiated, currently, the function returns without looking for a machine for later indices in the outer loop. The fix replaces the return with a break of the inner loop, enabling the function to negotiate machines for later indices by continuing the outer loop.

pylls avatar Jul 08 '19 14:07 pylls

Pull Request Test Coverage Report for Build 5542

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 62.627%

Totals Coverage Status
Change from base Build 5510: 0.0%
Covered Lines: 46923
Relevant Lines: 74925

💛 - Coveralls

coveralls avatar Jul 08 '19 15:07 coveralls

https://trac.torproject.org/projects/tor/ticket/31111

pylls avatar Jul 09 '19 09:07 pylls