Consider IPv6 OR ports when deciding whether a routerinfo change is cosmetic.

Closes #6423.

diff --git a/changes/bug6423 b/changes/bug6423
new file mode 100644 (file)
index 0000000..2ea4f14
--- /dev/null
+++ b/changes/bug6423
@@ -0,0 +1,3 @@
+ o Minor features:
+   - Consider new, removed or changed IPv6 OR ports a non cosmetic
+     change.

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 4979b933ade100dd55a2acc6e0ace6a125cd4c2c..3c39e362df851a42a509f4405b0279d9abedda04 100644 (file)
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -5160,6 +5160,8 @@ router_differences_are_cosmetic(const routerinfo_t *r1, const routerinfo_t *r2)
   if (strcasecmp(r1->address, r2->address) ||
       strcasecmp(r1->nickname, r2->nickname) ||
       r1->or_port != r2->or_port ||
+      !tor_addr_eq(&r1->ipv6_addr, &r2->ipv6_addr) ||
+      r1->ipv6_orport != r2->ipv6_orport ||
       r1->dir_port != r2->dir_port ||
       r1->purpose != r2->purpose ||
       crypto_pk_cmp_keys(r1->onion_pkey, r2->onion_pkey) ||