Small tweaks to 2841 code

    - const-ify some transport_t pointers
    - Remove a vestigial argument to parse_bridge_line
    - Make it compile without warnings on my laptop with
      --enable-gcc-warnings

diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 7857eda0b9133d1b851c65c3e4fe527e39e6a61a..b00a35dcce3c936068f4590c2878a9906ecb8868 100644 (file)
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -123,7 +123,7 @@ static int onion_append_hop(crypt_path_t **head_ptr, extend_info_t *choice);
 
 static void entry_guards_changed(void);
 
-static transport_t *transport_get_by_name(const char *name);
+static const transport_t *transport_get_by_name(const char *name);
 static void transport_free(transport_t *transport);
 static void bridge_free(bridge_info_t *bridge);
 
@@ -4603,7 +4603,7 @@ transport_free(transport_t *transport)
 
 /** Returns the transport in our transport list that has the name <b>name</b>.
  *  Else returns NULL. */
-static transport_t *
+static const transport_t *
 transport_get_by_name(const char *name)
 {
   tor_assert(name);
@@ -4822,7 +4822,7 @@ find_bridge_by_digest(const char *digest)
  */
 int
 find_transport_by_bridge_addrport(const tor_addr_t *addr, uint16_t port,
-                                  transport_t **transport)
+                                  const transport_t **transport)
 {
   if (!bridge_list)
     return 1;

diff --git a/src/or/circuitbuild.h b/src/or/circuitbuild.h
index 71ea6080e37db85304ac65386560493cf556c84e..54f82500fc1c41820ac04097fb4878a5ebf91282 100644 (file)
--- a/src/or/circuitbuild.h
+++ b/src/or/circuitbuild.h
@@ -143,7 +143,7 @@ void clear_transport_list(void);
 int transport_add_from_config(const tor_addr_t *addr, uint16_t port,
                                const char *name, int socks_ver);
 int find_transport_by_bridge_addrport(const tor_addr_t *addr, uint16_t port,
-                                      transport_t **transport);
+                                      const transport_t **transport);
 void validate_pluggable_transports_config(void);
 
 #endif

diff --git a/src/or/config.c b/src/or/config.c
index 12320e07f900167231d0455ad4c7e0958ca81e0c..9096839c151622de1cdd53e11fce9fab62837528 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -570,8 +570,7 @@ static int options_transition_affects_descriptor(or_options_t *old_options,
 static int check_nickname_list(const char *lst, const char *name, char **msg);
 static void config_register_addressmaps(or_options_t *options);
 
-static int parse_bridge_line(const char *line, int validate_only,
-                             or_options_t *options);
+static int parse_bridge_line(const char *line, int validate_only);
 static int parse_client_transport_line(const char *line, int validate_only);
 static int parse_dir_server_line(const char *line,
                                  dirinfo_type_t required_type,
@@ -1223,7 +1222,7 @@ options_act(or_options_t *old_options)
   if (options->Bridges) {
     mark_bridge_list();
     for (cl = options->Bridges; cl; cl = cl->next) {
-      if (parse_bridge_line(cl->value, 0, options)<0) {
+      if (parse_bridge_line(cl->value, 0)<0) {
         log_warn(LD_BUG,
                  "Previously validated Bridge line could not be added!");
         return -1;
@@ -3696,7 +3695,7 @@ options_validate(or_options_t *old_options, or_options_t *options,
 
   if (options->Bridges) {
     for (cl = options->Bridges; cl; cl = cl->next) {
-      if (parse_bridge_line(cl->value, 1, options)<0)
+      if (parse_bridge_line(cl->value, 1)<0)
         REJECT("Bridge line did not parse. See logs for details.");
     }
   }
@@ -4579,8 +4578,7 @@ options_init_logs(or_options_t *options, int validate_only)
  * <b>validate_only</b> is 0, and the line is well-formed, then add
  * the bridge described in the line to our internal bridge list. */
 static int
-parse_bridge_line(const char *line, int validate_only,
-                  or_options_t *options)
+parse_bridge_line(const char *line, int validate_only)
 {
   smartlist_t *items = NULL;
   int r;

diff --git a/src/or/connection.c b/src/or/connection.c
index e0865885f855f6b3509954a61c7915f3eb256167..05937ac8642504682ced6d40a33f97095ae3af89 100644 (file)
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -4130,7 +4130,7 @@ get_proxy_addrport(tor_addr_t *addr, uint16_t *port,
     goto done;
   } else if (options->ClientTransportPlugin ||
              options->Bridges) {
-    transport_t *transport=NULL;
+    const transport_t *transport=NULL;
     int r;
     r = find_transport_by_bridge_addrport(&conn->addr, conn->port, &transport);
     if (r == 0) { /* transport found */
@@ -4197,5 +4197,6 @@ proxy_type_to_string(int proxy_type)
   case PROXY_NONE:      return "NULL";
   default:              tor_assert(0);
   }
+  return NULL; /*Unreached*/
 }
 

diff --git a/src/or/connection_or.c b/src/or/connection_or.c
index 4cbd440a0109b069350c98a2d4259a7d09de1b94..2de25f68075b635ed478cde5f3fe699cdf94dade 100644 (file)
--- a/src/or/connection_or.c
+++ b/src/or/connection_or.c
@@ -336,7 +336,7 @@ connection_or_finished_connecting(or_connection_t *or_conn)
   else if (get_options()->Socks5Proxy)
     proxy_type = PROXY_SOCKS5;
   else if (get_options()->ClientTransportPlugin) {
-    transport_t *transport=NULL;
+    const transport_t *transport=NULL;
     int r;
     r = find_transport_by_bridge_addrport(&conn->addr,conn->port,&transport);
     if (r == 0) {