[or-cvs] r13647: Even though the man page said that "TrackHostExits ." should (in tor/trunk: . src/or)

[arma at seul.org]

Author: arma
Date: 2008-02-21 03:08:13 -0500 (Thu, 21 Feb 2008)
New Revision: 13647

Modified:
   tor/trunk/ChangeLog
   tor/trunk/src/or/circuituse.c
Log:
Even though the man page said that "TrackHostExits ." should
work, nobody had ever implemented it. Bugfix on 0.1.0.x.


Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-02-21 06:45:04 UTC (rev 13646)
+++ tor/trunk/ChangeLog	2008-02-21 08:08:13 UTC (rev 13647)
@@ -67,6 +67,8 @@
       on the socket. Bugfix on 0.1.2.x.
     - Change the behavior of "getinfo status/good-server-descriptor"
       so it doesn't return failure when any authority disappears.
+    - Even though the man page said that "TrackHostExits ." should
+      work, nobody had ever implemented it. Bugfix on 0.1.0.x.
 
   o Code simplifications and refactoring:
     - Remove the tor_strpartition function: its logic was confused,

Modified: tor/trunk/src/or/circuituse.c
===================================================================
--- tor/trunk/src/or/circuituse.c	2008-02-21 06:45:04 UTC (rev 13646)
+++ tor/trunk/src/or/circuituse.c	2008-02-21 08:08:13 UTC (rev 13647)
@@ -1185,7 +1185,8 @@
 
   SMARTLIST_FOREACH(options->TrackHostExits, const char *, cp, {
     if (cp[0] == '.') { /* match end */
-      if (!strcasecmpend(conn->socks_request->address, cp) ||
+      if (cp[1] == '\0' ||
+          !strcasecmpend(conn->socks_request->address, cp) ||
           !strcasecmp(conn->socks_request->address, &cp[1]))
           found_needle = 1;
     } else if (strcasecmp(cp, conn->socks_request->address) == 0) {