Fix compile warnings on Snow Leopard

Big thanks to nickm and arma for helping me with this!

diff --git a/src/common/address.c b/src/common/address.c
index fac9d50e1579787fc896568a8786011cc23d7a76..88cbbb74703a6db773abbb7a421cf2472b083582 100644 (file)
--- a/src/common/address.c
+++ b/src/common/address.c
@@ -373,10 +373,10 @@ tor_addr_parse_reverse_lookup_name(tor_addr_t *result, const char *address,
       return -1; /* malformed. */
 
     /* reverse the bytes */
-    inaddr.s_addr = (((inaddr.s_addr & 0x000000fful) << 24)
-                     |((inaddr.s_addr & 0x0000ff00ul) << 8)
-                     |((inaddr.s_addr & 0x00ff0000ul) >> 8)
-                     |((inaddr.s_addr & 0xff000000ul) >> 24));
+    inaddr.s_addr = (((inaddr.s_addr & (uint32_t)0x000000fful) << 24)
+                     |((inaddr.s_addr & (uint32_t)0x0000ff00ul) << 8)
+                     |((inaddr.s_addr & (uint32_t)0x00ff0000ul) >> 8)
+                     |((inaddr.s_addr & (uint32_t)0xff000000ul) >> 24));
 
     if (result) {
       tor_addr_from_in(result, &inaddr);

diff --git a/src/common/log.c b/src/common/log.c
index ea09fca1672373cf4e069d6fb369e62375002d6e..423a687a516e9ed6ac18b23b5fcb5c4f8edaa6f2 100644 (file)
--- a/src/common/log.c
+++ b/src/common/log.c
@@ -149,8 +149,8 @@ _log_prefix(char *buf, size_t buf_len, int severity)
   t = (time_t)now.tv_sec;
 
   n = strftime(buf, buf_len, "%b %d %H:%M:%S", tor_localtime_r(&t, &tm));
-  r = tor_snprintf(buf+n, buf_len-n, ".%.3ld [%s] ",
-                   (long)now.tv_usec / 1000, sev_to_string(severity));
+  r = tor_snprintf(buf+n, buf_len-n, ".%.3i [%s] ",
+                   (int)now.tv_usec / 1000, sev_to_string(severity));
   if (r<0)
     return buf_len-1;
   else

diff --git a/src/common/tortls.c b/src/common/tortls.c
index f14eab18a51b22de9cf77840d6564b404f7b5c54..aeb0ca0800fd2538beb55f9932296db55c1a7eae 100644 (file)
--- a/src/common/tortls.c
+++ b/src/common/tortls.c
@@ -1443,8 +1443,8 @@ tor_tls_used_v1_handshake(tor_tls_t *tls)
  * buffer and *<b>wbuf_bytes</b> to the amount actually used. */
 void
 tor_tls_get_buffer_sizes(tor_tls_t *tls,
-                         int *rbuf_capacity, int *rbuf_bytes,
-                         int *wbuf_capacity, int *wbuf_bytes)
+                         size_t *rbuf_capacity, size_t *rbuf_bytes,
+                         size_t *wbuf_capacity, size_t *wbuf_bytes)
 {
   if (tls->ssl->s3->rbuf.buf)
     *rbuf_capacity = tls->ssl->s3->rbuf.len;

diff --git a/src/common/tortls.h b/src/common/tortls.h
index 44e3b499ef117c990eb39092434568812da7be73..d00690911ce2fabf169edfe106bcd964c78a0b5d 100644 (file)
--- a/src/common/tortls.h
+++ b/src/common/tortls.h
@@ -73,8 +73,8 @@ void tor_tls_get_n_raw_bytes(tor_tls_t *tls,
                              size_t *n_read, size_t *n_written);
 
 void tor_tls_get_buffer_sizes(tor_tls_t *tls,
-                              int *rbuf_capacity, int *rbuf_bytes,
-                              int *wbuf_capacity, int *wbuf_bytes);
+                              size_t *rbuf_capacity, size_t *rbuf_bytes,
+                              size_t *wbuf_capacity, size_t *wbuf_bytes);
 
 int tor_tls_used_v1_handshake(tor_tls_t *tls);
 

diff --git a/src/common/util.c b/src/common/util.c
index 7b9e5eb5624e84e5aa571b2f014b5a2c2960ca57..9dcf9fba64173be482936873e50961749f582f99 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -1816,7 +1816,8 @@ write_chunks_to_file_impl(const char *fname, const smartlist_t *chunks,
                           int open_flags)
 {
   open_file_t *file = NULL;
-  int fd, result;
+  int fd;
+  ssize_t result;
   fd = start_writing_to_file(fname, open_flags, 0600, &file);
   if (fd<0)
     return -1;
@@ -1901,7 +1902,7 @@ read_file_to_str(const char *filename, int flags, struct stat *stat_out)
   int fd; /* router file */
   struct stat statbuf;
   char *string;
-  int r;
+  ssize_t r;
   int bin = flags & RFTS_BIN;
 
   tor_assert(filename);
@@ -1960,7 +1961,7 @@ read_file_to_str(const char *filename, int flags, struct stat *stat_out)
        * match for size. */
       int save_errno = errno;
       log_warn(LD_FS,"Could read only %d of %ld bytes of file \"%s\".",
-               r, (long)statbuf.st_size,filename);
+               (int)r, (long)statbuf.st_size,filename);
       tor_free(string);
       close(fd);
       errno = save_errno;

diff --git a/src/or/command.c b/src/or/command.c
index 98f093a72bc2045558d43fc5a81a12e7489657e8..67e463723f41db75241971e4e913959ec1ccb671 100644 (file)
--- a/src/or/command.c
+++ b/src/or/command.c
@@ -575,7 +575,7 @@ command_process_netinfo_cell(cell_t *cell, or_connection_t *conn)
     /* Consider all the other addresses; if any matches, this connection is
      * "canonical." */
     tor_addr_t addr;
-    const char *next = decode_address_from_payload(&addr, cp, end-cp);
+    const char *next = decode_address_from_payload(&addr, cp, (int)(end-cp));
     if (next == NULL) {
       log_fn(LOG_PROTOCOL_WARN,  LD_OR,
              "Bad address in netinfo cell; closing connection.");

diff --git a/src/or/config.c b/src/or/config.c
index 84146c1063ed080c76e79e3534621082c2927e86..8fd70bec9f00c8aee8e423ab59bbdfca02612092 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -2504,7 +2504,8 @@ is_local_addr(const tor_addr_t *addr)
      * the same /24 as last_resolved_addr will be the same as checking whether
      * it was on net 0, which is already done by is_internal_IP.
      */
-    if ((last_resolved_addr & 0xffffff00ul) == (ip & 0xffffff00ul))
+    if ((last_resolved_addr & (uint32_t)0xffffff00ul)
+        == (ip & (uint32_t)0xffffff00ul))
       return 1;
   }
   return 0;
@@ -4187,7 +4188,7 @@ options_init_from_string(const char *cf,
  err:
   config_free(&options_format, newoptions);
   if (*msg) {
-    int len = strlen(*msg)+256;
+    int len = (int)strlen(*msg)+256;
     char *newmsg = tor_malloc(len);
 
     tor_snprintf(newmsg, len, "Failed to parse/validate config: %s", *msg);

diff --git a/src/or/control.c b/src/or/control.c
index 328da21862fa55e7133fa3827a1843fff4f8d625..5688b8e71f88eaed4e8c2d7a9b41e3c641a05ff8 100644 (file)
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -2601,7 +2601,7 @@ handle_control_resolve(control_connection_t *conn, uint32_t len,
   int is_reverse = 0;
   (void) len; /* body is nul-terminated; it's safe to ignore the length */
 
-  if (!(conn->event_mask & (1L<<EVENT_ADDRMAP))) {
+  if (!(conn->event_mask & ((uint32_t)1L<<EVENT_ADDRMAP))) {
     log_warn(LD_CONTROL, "Controller asked us to resolve an address, but "
              "isn't listening for ADDRMAP events.  It probably won't see "
              "the answer.");

diff --git a/src/or/directory.c b/src/or/directory.c
index 7fc2fb1bdcd22fb488b8b9cfedbcacfe8dd2c761..5b8637a39d872f5f74767fbd44a6cf25a437f580 100644 (file)
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -872,7 +872,7 @@ static char *
 directory_get_consensus_url(int supports_conditional_consensus)
 {
   char *url;
-  int len;
+  size_t len;
 
   if (supports_conditional_consensus) {
     char *authority_id_list;
@@ -2342,7 +2342,7 @@ client_likes_consensus(networkstatus_t *v, const char *want_url)
   need_at_least = smartlist_len(want_authorities)/2+1;
   SMARTLIST_FOREACH(want_authorities, const char *, d, {
     char want_digest[DIGEST_LEN];
-    int want_len = strlen(d)/2;
+    size_t want_len = strlen(d)/2;
     if (want_len > DIGEST_LEN)
       want_len = DIGEST_LEN;
 

diff --git a/src/or/eventdns.c b/src/or/eventdns.c
index 4ae17a40c32b16202efc30658fd69f5fa63a5745..a889e803edad05e5e2fc1e14cd5fcb4c25ffc60e 100644 (file)
--- a/src/or/eventdns.c
+++ b/src/or/eventdns.c
@@ -2385,7 +2385,7 @@ out1:
 
 /* exported function */
 int
-evdns_nameserver_add(unsigned long int address) {
+evdns_nameserver_add(uint32_t address) {
        struct sockaddr_in sin;
        memset(&sin, 0, sizeof(sin));
        sin.sin_family = AF_INET;
@@ -2416,13 +2416,13 @@ evdns_nameserver_ip_add(const char *ip_as_string) {
 
        cp = strchr(ip_as_string, ':');
        if (*ip_as_string == '[') {
-               int len;
+               size_t len;
                if (!(cp = strchr(ip_as_string, ']'))) {
                        log(EVDNS_LOG_DEBUG, "Nameserver missing closing ]");
                        return 4;
                }
                len = cp-(ip_as_string + 1);
-               if (len > (int)sizeof(buf)-1) {
+               if (len > sizeof(buf)-1) {
                        log(EVDNS_LOG_DEBUG, "[Nameserver] does not fit in buffer.");
                        return 4;
                }

diff --git a/src/or/eventdns.h b/src/or/eventdns.h
index 734bacf2d28b4e1b25ab6045ea58f8368cd6910c..bf3b64d08a873ab3ca027d7d8da5c36074c94931 100644 (file)
--- a/src/or/eventdns.h
+++ b/src/or/eventdns.h
@@ -112,7 +112,7 @@
  *
  * API reference:
  *
- * int evdns_nameserver_add(unsigned long int address)
+ * int evdns_nameserver_add(uint32_t address)
  *      Add a nameserver. The address should be an IP address in
  *      network byte order. The type of address is chosen so that
  *      it matches in_addr.s_addr.
@@ -258,7 +258,7 @@ typedef void (*evdns_callback_type) (int result, char type, int count, int ttl,
 int evdns_init(void);
 void evdns_shutdown(int fail_requests);
 const char *evdns_err_to_string(int err);
-int evdns_nameserver_add(unsigned long int address);
+int evdns_nameserver_add(uint32_t address);
 int evdns_count_nameservers(void);
 int evdns_clear_nameservers_and_suspend(void);
 int evdns_resume(void);

diff --git a/src/or/main.c b/src/or/main.c
index 62335d3421a60e45a63b383eeaa74279274110dc..ca09af05611c4b8c8a432ced3827ef5d39b170bc 100644 (file)
--- a/src/or/main.c
+++ b/src/or/main.c
@@ -1608,7 +1608,7 @@ dumpstats(int severity)
 {
   time_t now = time(NULL);
   time_t elapsed;
-  int rbuf_cap, wbuf_cap, rbuf_len, wbuf_len;
+  size_t rbuf_cap, wbuf_cap, rbuf_len, wbuf_len;
 
   log(severity, LD_GENERAL, "Dumping stats:");
 
@@ -1644,7 +1644,7 @@ dumpstats(int severity)
           log(severity, LD_GENERAL,
               "Conn %d: %d/%d bytes used on OpenSSL read buffer; "
               "%d/%d bytes used on write buffer.",
-              i, rbuf_len, rbuf_cap, wbuf_len, wbuf_cap);
+              i, (int)rbuf_len, (int)rbuf_cap, (int)wbuf_len, (int)wbuf_cap);
         }
       }
     }

diff --git a/src/or/rendclient.c b/src/or/rendclient.c
index a5d7c1016ea3852d898df7122772663787ced836..38383e8abbf8f8f87612a91a5ddd5573244e6cd1 100644 (file)
--- a/src/or/rendclient.c
+++ b/src/or/rendclient.c
@@ -146,7 +146,7 @@ rend_client_send_introduction(origin_circuit_t *introcirc,
              REND_DESC_COOKIE_LEN);
       v3_shift += 2+REND_DESC_COOKIE_LEN;
     }
-    set_uint32(tmp+v3_shift+1, htonl(time(NULL)));
+    set_uint32(tmp+v3_shift+1, htonl((uint32_t)time(NULL)));
     v3_shift += 4;
   } /* if version 2 only write version number */
   else if (entry->parsed->protocols & (1<<2)) {

diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index 2fd041d33e9114a7a478ec11540d3f66a7951d91..7ba00993c4a86c80b65b14e09d305e022b551621 100644 (file)
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -1013,7 +1013,7 @@ rend_service_introduce(origin_circuit_t *circuit, const char *request,
     /* Check timestamp. */
     memcpy((char*)&ts, buf+1+v3_shift, sizeof(uint32_t));
     v3_shift += 4;
-    ts = ntohl(ts);
+    ts = ntohl((uint32_t)ts);
     if ((now - ts) < -1 * REND_REPLAY_TIME_INTERVAL / 2 ||
         (now - ts) > REND_REPLAY_TIME_INTERVAL / 2) {
       log_warn(LD_REND, "INTRODUCE2 cell is too %s. Discarding.",

diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index 8021158e318df5e5c35187a39bd4c59625035686..97dd20f4e38ca54a85d6e12b29785428e6855df8 100644 (file)
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -1917,8 +1917,9 @@ routerstatus_parse_entry_from_string(memarea_t *area,
     for (i=0; i < tok->n_args; ++i) {
       if (!strcmpstart(tok->args[i], "Bandwidth=")) {
         int ok;
-        rs->bandwidth = tor_parse_ulong(strchr(tok->args[i], '=')+1, 10,
-                                        0, UINT32_MAX, &ok, NULL);
+        rs->bandwidth = (uint32_t)tor_parse_ulong(strchr(tok->args[i], '=')+1,
+                                                  10, 0, UINT32_MAX,
+                                                  &ok, NULL);
         if (!ok) {
           log_warn(LD_DIR, "Invalid Bandwidth %s", escaped(tok->args[i]));
           goto err;

diff --git a/src/or/test.c b/src/or/test.c
index 7b7411e2f85635003d92a49847b4dcc125e6269f..e06dd5951f56a1f896e63d6a5960e66b2d05a269 100644 (file)
--- a/src/or/test.c
+++ b/src/or/test.c
@@ -410,7 +410,7 @@ test_crypto_dh(void)
   char p2[DH_BYTES];
   char s1[DH_BYTES];
   char s2[DH_BYTES];
-  int s1len, s2len;
+  ssize_t s1len, s2len;
 
   test_eq(crypto_dh_get_bytes(dh1), DH_BYTES);
   test_eq(crypto_dh_get_bytes(dh2), DH_BYTES);

diff --git a/src/tools/tor-gencert.c b/src/tools/tor-gencert.c
index 4971668c9f513a3a4383aa93884d3cf11abbd3f5..c7d9282076d5aa96117e96574d524acbbcd9b551 100644 (file)
--- a/src/tools/tor-gencert.c
+++ b/src/tools/tor-gencert.c
@@ -70,7 +70,7 @@ show_help(void)
 static void
 crypto_log_errors(int severity, const char *doing)
 {
-  unsigned int err;
+  unsigned long err;
   const char *msg, *lib, *func;
   while ((err = ERR_get_error()) != 0) {
     msg = (const char*)ERR_reason_error_string(err);
@@ -94,7 +94,7 @@ load_passphrase(void)
 {
   char *cp;
   char buf[1024]; /* "Ought to be enough for anybody." */
-  int n = read_all(passphrase_fd, buf, sizeof(buf), 0);
+  ssize_t n = read_all(passphrase_fd, buf, sizeof(buf), 0);
   if (n < 0) {
     log_err(LD_GENERAL, "Couldn't read from passphrase fd: %s",
             strerror(errno));

diff --git a/src/tools/tor-resolve.c b/src/tools/tor-resolve.c
index fe4e882416617c8e3e2c33ae2927871398d09243..f12c3d8dd3fe926da90aae7114fb47cbce8d14ee 100644 (file)
--- a/src/tools/tor-resolve.c
+++ b/src/tools/tor-resolve.c
@@ -51,7 +51,7 @@ static void usage(void) ATTR_NORETURN;
 /** Set *<b>out</b> to a newly allocated SOCKS4a resolve request with
  * <b>username</b> and <b>hostname</b> as provided.  Return the number
  * of bytes in the request. */
-static int
+static ssize_t
 build_socks_resolve_request(char **out,
                             const char *username,
                             const char *hostname,
@@ -184,7 +184,7 @@ do_resolve(const char *hostname, uint32_t sockshost, uint16_t socksport,
   int s;
   struct sockaddr_in socksaddr;
   char *req = NULL;
-  int len = 0;
+  ssize_t len = 0;
 
   tor_assert(hostname);
   tor_assert(result_addr);