[tor-commits] [tor/master] Don't prepend reject *:* to Reachable(OR/Dir)Addresses in options_validate()

nickm at torproject.org nickm at torproject.org
Tue Aug 28 19:44:19 UTC 2018


commit fe18776349f22d84bf953905e82b6f5dc95eabcb
Author: Neel Chauhan <neel at neelc.org>
Date:   Fri Jul 20 21:36:23 2018 -0400

    Don't prepend reject *:* to Reachable(OR/Dir)Addresses in options_validate()
---
 src/app/config/config.c | 25 -------------------------
 1 file changed, 25 deletions(-)

diff --git a/src/app/config/config.c b/src/app/config/config.c
index c7310f775..adbba0659 100644
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -3359,7 +3359,6 @@ STATIC int
 options_validate(or_options_t *old_options, or_options_t *options,
                  or_options_t *default_options, int from_setconf, char **msg)
 {
-  int i;
   config_line_t *cl;
   const char *uname = get_uname();
   int n_ports=0;
@@ -3680,30 +3679,6 @@ options_validate(or_options_t *old_options, or_options_t *options,
     }
   }
 
-  /* Terminate Reachable*Addresses with reject *
-   */
-  for (i=0; i<3; i++) {
-    config_line_t **linep =
-      (i==0) ? &options->ReachableAddresses :
-        (i==1) ? &options->ReachableORAddresses :
-                 &options->ReachableDirAddresses;
-    if (!*linep)
-      continue;
-    /* We need to end with a reject *:*, not an implicit accept *:* */
-    for (;;) {
-      linep = &((*linep)->next);
-      if (!*linep) {
-        *linep = tor_malloc_zero(sizeof(config_line_t));
-        (*linep)->key = tor_strdup(
-          (i==0) ?  "ReachableAddresses" :
-            (i==1) ? "ReachableORAddresses" :
-                     "ReachableDirAddresses");
-        (*linep)->value = tor_strdup("reject *:*");
-        break;
-      }
-    }
-  }
-
   if ((options->ReachableAddresses ||
        options->ReachableORAddresses ||
        options->ReachableDirAddresses ||





More information about the tor-commits mailing list