[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] r9625: Two XXX012 comments turned out to be bogus; remove them and (in tor/trunk: . src/or)



Author: nickm
Date: 2007-02-23 16:56:10 -0500 (Fri, 23 Feb 2007)
New Revision: 9625

Modified:
   tor/trunk/
   tor/trunk/src/or/dns.c
Log:
 r11898@catbus:  nickm | 2007-02-23 16:55:43 -0500
 Two XXX012 comments turned out to be bogus; remove them and document why.



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r11898] on 8246c3cf-6607-4228-993b-4d95d33730f1

Modified: tor/trunk/src/or/dns.c
===================================================================
--- tor/trunk/src/or/dns.c	2007-02-23 20:13:02 UTC (rev 9624)
+++ tor/trunk/src/or/dns.c	2007-02-23 21:56:10 UTC (rev 9625)
@@ -565,6 +565,11 @@
  * if resolve valid, put it into <b>exitconn</b>-\>addr and return 1.
  * If resolve failed, unlink exitconn if needed, free it, and return -1.
  *
+ * (For EXIT_PURPOSE_RESOLVE connections, send back a RESOLVED error cell
+ * on returning -1.  For EXIT_PURPOSE_CONNECT connections, there's no
+ * need to send back an END cell, since connection_exit_begin_conn will
+ * do that for us.)
+ *
  * If <b>oncirc</b> is provided, and this is a resolve request, we have
  * a cached answer, send the answer back along oncirc; otherwise, send
  * the answer back along <b>exitconn</b>'s attached circuit.
@@ -607,7 +612,6 @@
         escaped_safe_str(exitconn->_base.address));
     if (is_resolve)
       send_resolved_cell(exitconn, oncirc, RESOLVED_TYPE_ERROR);
-    /* XXXX012 send error in connect case? -NM */
     circ = circuit_get_by_edge_conn(exitconn);
     if (circ)
       circuit_detach_stream(circ, exitconn);
@@ -697,7 +701,6 @@
         log_debug(LD_EXIT,"Connection (fd %d) found cached error for %s",
                   exitconn->_base.s,
                   escaped_safe_str(exitconn->_base.address));
-        /*  XXXX012 send back indication of failure for connect case? -NM*/
         if (is_resolve)
           send_resolved_cell(exitconn, oncirc, RESOLVED_TYPE_ERROR);
         circ = circuit_get_by_edge_conn(exitconn);