[tor-commits] [tor/master] Fix an issue in my mapaddress domains code spotted by arma

nickm at torproject.org nickm at torproject.org
Wed Nov 30 19:10:01 UTC 2011


commit 66859e2d4a7823a98a6842e5cf3b09c9fa47c704
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Nov 25 20:13:55 2011 -0500

    Fix an issue in my mapaddress domains code spotted by arma
    
    MapAddress *.torproject.org torproject.org would have been interpreted
    as a map from a domain to itself, and would have cleared the mapping.
    Now we require not only a match of domains, but of wildcards.
---
 src/or/connection_edge.c |   12 ++++++++----
 1 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c
index f7b2411..4f26a6b 100644
--- a/src/or/connection_edge.c
+++ b/src/or/connection_edge.c
@@ -1178,15 +1178,18 @@ addressmap_have_mapping(const char *address, int update_expiry)
  * <b>new_address</b> should be a newly dup'ed string, which we'll use or
  * free as appropriate. We will leave address alone.
  *
- * If <b>new_address</b> is NULL, or equal to <b>address</b>, remove
- * any mappings that exist from <b>address</b>.
- *
  * If <b>wildcard_addr</b> is true, then the mapping will match any address
  * equal to <b>address</b>, or any address ending with a period followed by
  * <b>address</b>.  If <b>wildcard_addr</b> and <b>wildcard_new_addr</b> are
  * both true, the mapping will rewrite addresses that end with
  * ".<b>address</b>" into ones that end with ".<b>new_address</b>."
  *
+ * If <b>new_address</b> is NULL, or <b>new_address</b> is equal to
+ * <b>address</b> and <b>wildcard_addr</b> is equal to
+ * <b>wildcard_new_addr</b>, remove any mappings that exist from
+ * <b>address</b>.
+ *
+ *
  * It is an error to set <b>wildcard_new_addr</b> if <b>wildcard_addr</b> is
  * not set. */
 void
@@ -1201,7 +1204,8 @@ addressmap_register(const char *address, char *new_address, time_t expires,
     tor_assert(wildcard_addr);
 
   ent = strmap_get(addressmap, address);
-  if (!new_address || !strcasecmp(address,new_address)) {
+  if (!new_address || (!strcasecmp(address,new_address) &&
+                       wildcard_addr == wildcard_new_addr)) {
     /* Remove the mapping, if any. */
     tor_free(new_address);
     if (ent) {





More information about the tor-commits mailing list