Avoid casting smartlist index implicitly

diff --git a/changes/bug26282 b/changes/bug26282
new file mode 100644 (file)
index 0000000..c278f0b
--- /dev/null
+++ b/changes/bug26282
@@ -0,0 +1,4 @@
+  o Minor bugfixes (C correctness):
+    - Avoid casting smartlist index to int implicitly, as it may trigger
+      a warning (-Wshorten-64-to-32). Fixes bug 26282; bugfix on
+      0.2.3.13-alpha, 0.2.7.1-alpha and 0.2.1.1-alpha.

diff --git a/src/common/util.c b/src/common/util.c
index 53e4507f1f39459860e491a23e90e066255aab30..b0eb96306f3cba0500d9e95128fa931e71371fbe 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -4813,7 +4813,7 @@ process_environment_make(struct smartlist_t *env_vars)
 
   total_env_length = 1; /* terminating NUL of terminating empty string */
   for (i = 0; i < n_env_vars; ++i) {
-    const char *s = smartlist_get(env_vars, i);
+    const char *s = smartlist_get(env_vars, (int)i);
     size_t slen = strlen(s);
 
     tor_assert(slen + 1 != 0);
@@ -4843,7 +4843,7 @@ process_environment_make(struct smartlist_t *env_vars)
     const char *prev_env_var = NULL;
 
     for (i = 0; i < n_env_vars; ++i) {
-      const char *s = smartlist_get(env_vars_sorted, i);
+      const char *s = smartlist_get(env_vars_sorted, (int)i);
       size_t slen = strlen(s);
       size_t s_name_len = str_num_before(s, '=');
 

diff --git a/src/or/control.c b/src/or/control.c
index 0d637dce7ad840cdc13d30b7599d1df76f6629d9..95b16cf5a81e6da8a89b68dd14c5f68b368a654c 100644 (file)
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -4624,7 +4624,7 @@ handle_control_add_onion(control_connection_t *conn,
     static const char *max_s_prefix = "MaxStreams=";
     static const char *auth_prefix = "ClientAuth=";
 
-    const char *arg = smartlist_get(args, i);
+    const char *arg = smartlist_get(args, (int)i);
     if (!strcasecmpstart(arg, port_prefix)) {
       /* "Port=VIRTPORT[,TARGET]". */
       const char *port_str = arg + strlen(port_prefix);

diff --git a/src/or/geoip.c b/src/or/geoip.c
index 2c917c564d6e589b74065ae880b2c902da6665f9..d59043a7f6a336a12d8f71347f32f52af1dbfa62 100644 (file)
--- a/src/or/geoip.c
+++ b/src/or/geoip.c
@@ -150,7 +150,7 @@ geoip_add_entry(const tor_addr_t *low, const tor_addr_t *high,
     idx = ((uintptr_t)idxplus1_)-1;
   }
   {
-    geoip_country_t *c = smartlist_get(geoip_countries, idx);
+    geoip_country_t *c = smartlist_get(geoip_countries, (int)idx);
     tor_assert(!strcasecmp(c->countrycode, country));
   }