commit a8b8a1ee99fa843a5937b4d2a5973c444a4311e0 Author: David Goulet <dgou...@torproject.org> Date: Tue Jun 30 12:05:56 2020 -0400
addr: Only attempt Address resolution on non parsable lines In get_address_from_config(), we would attempt to resolve an Address line that is not from the requested family but that line could be a valid address from another family (v4 vs v6). This makes it that we don't attempt to resolve a valid address from another family. Found with unit test config/find_my_address_mixed. Fixed in #33235 Related to #33233 Signed-off-by: David Goulet <dgou...@torproject.org> --- src/app/config/resolve_addr.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/app/config/resolve_addr.c b/src/app/config/resolve_addr.c index c8b44de84..61662c340 100644 --- a/src/app/config/resolve_addr.c +++ b/src/app/config/resolve_addr.c @@ -193,6 +193,10 @@ get_address_from_config(const or_options_t *options, int warn_severity, explicit_ip = true; num_valid_addr++; continue; + } else if (af != -1) { + /* Parsable address but just not the one from the family we want. Skip + * it so we don't attempt a resolve. */ + continue; } /* Not an IP address. Considering this value a hostname and attempting to _______________________________________________ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits