Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

router: Stop advertising incorrect auto IPv6 ORPorts #1625

Closed
wants to merge 4 commits into from

Conversation

Labels
None yet
Projects
None yet
2 participants
@teor2345
Copy link
Contributor

@teor2345 teor2345 commented Dec 20, 2019

When IPv6 ORPorts are set to "auto", tor relays and bridges would
advertise an incorrect port in their descriptor.

This may be a low-severity memory safety issue, because the published
port number may be derived from uninitialised or out-of-bounds memory
reads.

Fixes bug 32588; bugfix on 0.2.3.9-alpha.

@teor2345 teor2345 mentioned this pull request Dec 20, 2019
@coveralls
Copy link

@coveralls coveralls commented Dec 20, 2019

Pull Request Test Coverage Report for Build 7741

  • 0 of 18 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.004%) to 59.187%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/feature/relay/router.c 0 18 0.0%
Totals Coverage Status
Change from base Build 7615: -0.004%
Covered Lines: 43303
Relevant Lines: 73163

💛 - Coveralls

teor2345 added 2 commits Jan 8, 2020
When IPv6 ORPorts are set to "auto", tor relays and bridges would
advertise an incorrect port in their descriptor.

This may be a low-severity memory safety issue, because the published
port number may be derived from uninitialised or out-of-bounds memory
reads.

Fixes bug 32588; bugfix on 0.2.3.9-alpha.
Return early when there is no suitable IPv6 ORPort.

Code simplification and refactoring.

Cleanup after 32588.
@teor2345
Copy link
Contributor Author

@teor2345 teor2345 commented Jan 13, 2020

Obsoleted by #1654, leaving open for CI.

@teor2345
Copy link
Contributor Author

@teor2345 teor2345 commented Mar 20, 2020

Obsoleted by #1654.

@teor2345 teor2345 closed this Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment