mirror of
https://gitlab.torproject.org/tpo/core/tor.git
synced 2024-11-24 04:13:28 +01:00
Merge remote-tracking branch 'teor/bug18351'
This commit is contained in:
commit
54559e5845
6
changes/bug18351
Normal file
6
changes/bug18351
Normal file
@ -0,0 +1,6 @@
|
||||
o Minor bugfixes (logging):
|
||||
- Downgrade logs and backtraces about IP versions to
|
||||
info-level. Only log backtraces once each time tor runs.
|
||||
Assists in diagnosing bug 18351; bugfix on c3cc8e16e in
|
||||
tor-0.2.8.1-alpha.
|
||||
Reported by "sysrqb" and "Christian", patch by "teor".
|
@ -1802,11 +1802,15 @@ connection_connect_log_client_use_ip_version(const connection_t *conn)
|
||||
/* Check if we broke a mandatory address family restriction */
|
||||
if ((must_ipv4 && tor_addr_family(&real_addr) == AF_INET6)
|
||||
|| (must_ipv6 && tor_addr_family(&real_addr) == AF_INET)) {
|
||||
log_warn(LD_BUG, "%s connection to %s violated ClientUseIPv%s 0.",
|
||||
static int logged_backtrace = 0;
|
||||
log_info(LD_BUG, "Outgoing %s connection to %s violated ClientUseIPv%s 0.",
|
||||
conn->type == CONN_TYPE_OR ? "OR" : "Dir",
|
||||
fmt_addr(&real_addr),
|
||||
options->ClientUseIPv4 == 0 ? "4" : "6");
|
||||
log_backtrace(LOG_WARN, LD_BUG, "Address came from");
|
||||
if (!logged_backtrace) {
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Address came from");
|
||||
logged_backtrace = 1;
|
||||
}
|
||||
}
|
||||
|
||||
/* Bridges are allowed to break IPv4/IPv6 ORPort preferences to connect to
|
||||
@ -1819,9 +1823,10 @@ connection_connect_log_client_use_ip_version(const connection_t *conn)
|
||||
/* Check if we couldn't satisfy an address family preference */
|
||||
if ((!pref_ipv6 && tor_addr_family(&real_addr) == AF_INET6)
|
||||
|| (pref_ipv6 && tor_addr_family(&real_addr) == AF_INET)) {
|
||||
log_info(LD_NET, "Connection to %s doesn't satisfy ClientPreferIPv6%sPort "
|
||||
"%d, with ClientUseIPv4 %d, and fascist_firewall_use_ipv6 %d "
|
||||
"(ClientUseIPv6 %d and UseBridges %d).",
|
||||
log_info(LD_NET, "Outgoing connection to %s doesn't satisfy "
|
||||
"ClientPreferIPv6%sPort %d, with ClientUseIPv4 %d, and "
|
||||
"fascist_firewall_use_ipv6 %d (ClientUseIPv6 %d and UseBridges "
|
||||
"%d).",
|
||||
fmt_addr(&real_addr),
|
||||
conn->type == CONN_TYPE_OR ? "OR" : "Dir",
|
||||
conn->type == CONN_TYPE_OR ? options->ClientPreferIPv6ORPort
|
||||
|
@ -672,15 +672,20 @@ directory_choose_address_routerstatus(const routerstatus_t *status,
|
||||
FIREWALL_DIR_CONNECTION, 0,
|
||||
use_dir_ap);
|
||||
|
||||
/* We rejected both addresses. This isn't great. */
|
||||
/* We rejected all addresses in the relay's status. This means we can't
|
||||
* connect to it. */
|
||||
if (!have_or && !have_dir) {
|
||||
log_warn(LD_BUG, "Rejected all OR and Dir addresses from %s when "
|
||||
"launching a directory connection to: IPv4 %s OR %d Dir %d "
|
||||
"IPv6 %s OR %d Dir %d", routerstatus_describe(status),
|
||||
static int logged_backtrace = 0;
|
||||
log_info(LD_BUG, "Rejected all OR and Dir addresses from %s when "
|
||||
"launching an outgoing directory connection to: IPv4 %s OR %d "
|
||||
"Dir %d IPv6 %s OR %d Dir %d", routerstatus_describe(status),
|
||||
fmt_addr32(status->addr), status->or_port,
|
||||
status->dir_port, fmt_addr(&status->ipv6_addr),
|
||||
status->ipv6_orport, status->dir_port);
|
||||
log_backtrace(LOG_WARN, LD_BUG, "Addresses came from");
|
||||
if (!logged_backtrace) {
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Addresses came from");
|
||||
logged_backtrace = 1;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
|
||||
@ -1100,14 +1105,23 @@ directory_initiate_command_rend(const tor_addr_port_t *or_addr_port,
|
||||
|
||||
if (or_connection && (!or_addr_port->port
|
||||
|| tor_addr_is_null(&or_addr_port->addr))) {
|
||||
log_warn(LD_DIR, "Cannot make an OR connection without an OR port.");
|
||||
log_backtrace(LOG_WARN, LD_BUG, "Address came from");
|
||||
static int logged_backtrace = 0;
|
||||
log_warn(LD_DIR, "Cannot make an outgoing OR connection without an OR "
|
||||
"port.");
|
||||
if (!logged_backtrace) {
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Address came from");
|
||||
logged_backtrace = 1;
|
||||
}
|
||||
return;
|
||||
} else if (!or_connection && (!dir_addr_port->port
|
||||
|| tor_addr_is_null(&dir_addr_port->addr))) {
|
||||
log_warn(LD_DIR, "Cannot make a Dir connection without a Dir port.");
|
||||
log_backtrace(LOG_WARN, LD_BUG, "Address came from");
|
||||
|
||||
static int logged_backtrace = 0;
|
||||
log_warn(LD_DIR, "Cannot make an outgoing Dir connection without a Dir "
|
||||
"port.");
|
||||
if (!logged_backtrace) {
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Address came from");
|
||||
logged_backtrace = 1;
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -1560,20 +1560,24 @@ router_picked_poor_directory_log(const routerstatus_t *rs)
|
||||
/* We couldn't find a node, or the one we have doesn't fit our preferences.
|
||||
* This might be a bug. */
|
||||
if (!rs) {
|
||||
log_warn(LD_BUG, "Firewall denied all OR and Dir addresses for all relays "
|
||||
"when searching for a directory.");
|
||||
log_backtrace(LOG_WARN, LD_BUG, "Node search initiated by");
|
||||
static int logged_backtrace = 0;
|
||||
log_info(LD_BUG, "Wanted to make an outgoing directory connection, but "
|
||||
"all OR and Dir addresses for all relays were not reachable. "
|
||||
"Check ReachableAddresses, ClientUseIPv4, and similar options.");
|
||||
if (!logged_backtrace) {
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Node search initiated by");
|
||||
logged_backtrace = 1;
|
||||
}
|
||||
} else if (!fascist_firewall_allows_rs(rs, FIREWALL_OR_CONNECTION, 1)
|
||||
&& !fascist_firewall_allows_rs(rs, FIREWALL_DIR_CONNECTION, 1)
|
||||
) {
|
||||
log_info(LD_BUG, "Selected a directory %s with non-preferred OR and Dir "
|
||||
"addresses for launching a connection: "
|
||||
"addresses for launching an outgoing connection: "
|
||||
"IPv4 %s OR %d Dir %d IPv6 %s OR %d Dir %d",
|
||||
routerstatus_describe(rs),
|
||||
fmt_addr32(rs->addr), rs->or_port,
|
||||
rs->dir_port, fmt_addr(&rs->ipv6_addr),
|
||||
rs->ipv6_orport, rs->dir_port);
|
||||
log_backtrace(LOG_INFO, LD_BUG, "Node search initiated by");
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user