[tor-commits] [tor/master] Remove some XXXX commens in dns.c

nickm at torproject.org nickm at torproject.org
Thu Nov 15 19:47:53 UTC 2012


commit a4fce0fee855288d600636462f1ca9acf30e036f
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Nov 15 12:17:36 2012 -0500

    Remove some XXXX commens in dns.c
    
    Previously, I was freaking out about passing an unspec address to
    dns_found_answer() on an error, since I was using the address type to
    determine whether the error was an error on an ipv4 address lookup or
    on an ipv6 address lookup.  But now dns_found_answer() has a separate
    orig_query_type argument to tell what kind of query it is, so there's
    no need to freak out.
---
 src/or/dns.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/or/dns.c b/src/or/dns.c
index 9834bd7..870494a 100644
--- a/src/or/dns.c
+++ b/src/or/dns.c
@@ -1580,7 +1580,7 @@ evdns_callback(int result, char type, int count, int ttl, void *addresses,
   const char *hostname = NULL;
   int was_wildcarded = 0;
 
-  tor_addr_make_unspec(&addr); /*WRONG WRONG WRONG XXXX XXXXX IPV6 prop208*/
+  tor_addr_make_unspec(&addr);
 
   /* Keep track of whether IPv6 is working */
   if (type == DNS_IPv6_AAAA) {
@@ -1615,7 +1615,7 @@ evdns_callback(int result, char type, int count, int ttl, void *addresses,
                   safe_str(escaped_address),
                   escaped_safe_str(answer_buf));
         was_wildcarded = 1;
-        tor_addr_make_null(&addr, AF_INET); /* ???? */
+        tor_addr_make_unspec(&addr);
         result = DNS_ERR_NOTEXIST;
       } else {
         log_debug(LD_EXIT, "eventdns said that %s resolves to %s",
@@ -1637,7 +1637,7 @@ evdns_callback(int result, char type, int count, int ttl, void *addresses,
                   safe_str(escaped_address),
                   escaped_safe_str(answer_buf));
         was_wildcarded = 1;
-        tor_addr_make_unspec(&addr); /* WRONG WRONG ETC XXXXXXXX */
+        tor_addr_make_unspec(&addr);
         result = DNS_ERR_NOTEXIST;
       } else {
         log_debug(LD_EXIT, "eventdns said that %s resolves to %s",





More information about the tor-commits mailing list