[or-cvs] r16985: {tor} Commit fix for bug 807 from mwenge: send CLOSED controller e (in tor/trunk: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2008-09-26 16:25:35 -0400 (Fri, 26 Sep 2008)
New Revision: 16985

Modified:
   tor/trunk/ChangeLog
   tor/trunk/src/or/connection_edge.c
Log:
Commit fix for bug 807 from mwenge: send CLOSED controller event for reverse dns resolve, not FAILED.

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-09-26 19:09:12 UTC (rev 16984)
+++ tor/trunk/ChangeLog	2008-09-26 20:25:35 UTC (rev 16985)
@@ -67,6 +67,10 @@
       had met our limits precisely. Fixes bug 824. Patch from by rovv.
       Bugfix on 0.2.0.x (??).
 
+  o Minor bugfixes (controller):
+    - Make DNS resolved events into "CLOSED", not "FAILED".  Bugfix on
+      0.1.2.5-alpha.  Fix by Robert Hogan.  Resolves bug 807.
+
   o Minor features:
     - Update to the "September 1 2008" ip-to-country file.
     - Allow ports 465 and 587 in the default exit policy again. We had

Modified: tor/trunk/src/or/connection_edge.c
===================================================================
--- tor/trunk/src/or/connection_edge.c	2008-09-26 19:09:12 UTC (rev 16984)
+++ tor/trunk/src/or/connection_edge.c	2008-09-26 20:25:35 UTC (rev 16985)
@@ -1389,8 +1389,7 @@
                                              map_expires);
       connection_mark_unattached_ap(conn,
                                 END_STREAM_REASON_DONE |
-                                END_STREAM_REASON_FLAG_ALREADY_SOCKS_REPLIED |
-                                END_STREAM_REASON_FLAG_ALREADY_SENT_CLOSED);
+                                END_STREAM_REASON_FLAG_ALREADY_SOCKS_REPLIED);
       return 0;
     }
     if (options->ClientDNSRejectInternalAddresses) {