Browse Source

r8861@totoro: nickm | 2006-10-03 14:49:54 -0400
Fix bug in r8579: TrackHostExits .foo.bar implies TrackHostExits foo.bar; also fix logic error.


svn:r8592

Nick Mathewson 17 years ago
parent
commit
4c10464e13
1 changed files with 2 additions and 6 deletions
  1. 2 6
      src/or/circuituse.c

+ 2 - 6
src/or/circuituse.c

@@ -1106,7 +1106,6 @@ static void
 consider_recording_trackhost(edge_connection_t *conn, origin_circuit_t *circ)
 {
   int found_needle = 0;
-  char *str;
   or_options_t *options = get_options();
   size_t len;
   char *new_address;
@@ -1120,12 +1119,9 @@ consider_recording_trackhost(edge_connection_t *conn, origin_circuit_t *circ)
 
   SMARTLIST_FOREACH(options->TrackHostExits, const char *, cp, {
     if (cp[0] == '.') { /* match end */
-      if (!strcasecmpend(conn->socks_request->address, cp)) {
-        if (str == conn->socks_request->address
-          || strcmp(str, &cp[1]) == 0) {
+      if (!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) {
       found_needle = 1;
     }