tor/changes/ticket40289
David Goulet dfcb050bbf config: Do not compare for duplicate ORPorts with different addresses
We were just looking at the family which is not correct because it is possible
to have two explicit ORPort for the same family but different addresses. One
example is:

  ORPort 127.0.0.1:9001 NoAdvertise
  ORPort 1.2.3.4:9001 NoListen

Thus, this patch now ignores ports that have different addresses iff they are
both explicits. That is, if we have this example, also two different
addresses:

  ORPort 9001
  ORPort 127.0.0.1:9001 NoAdvertise

The first one is implicit and second one is explicit and thus we have to
consider them for removal which in this case would remove the "ORPort 9001" in
favor of the second port.

Fixes #40289

Signe-off-by: David Goulet <dgoulet@torproject.org>
2021-02-12 13:13:43 -05:00

7 lines
316 B
Plaintext

o Minor bugfixes (relay, config):
- Fix a problem in the removal of duplicate ORPort from the internal port
list when loading config file. We were removing wrong ports breaking valid
torrc uses cases for multiple ORPorts of the same address family. Fixes
bug 40289; bugfix on 0.4.5.1-alpha.