[tor-commits] [tor/master] Fix a clang compilation warning

nickm at torproject.org nickm at torproject.org
Mon Mar 31 14:08:36 UTC 2014


commit 5e0cfba969e886b2f1769b70c425de658b229e9f
Author: Nick Mathewson <nickm at torproject.org>
Date:   Mon Mar 31 10:07:42 2014 -0400

    Fix a clang compilation warning
    
    Subtracting two time_t values was yielding something that maybe
    can't be fit in an int.
    
    Bugfix on 0389d4aa; bug not in any released Tor.
---
 src/or/microdesc.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/or/microdesc.c b/src/or/microdesc.c
index 454ec94..4977332 100644
--- a/src/or/microdesc.c
+++ b/src/or/microdesc.c
@@ -378,19 +378,19 @@ microdesc_cache_clean(microdesc_cache_t *cache, time_t cutoff, int force)
         /* Let's try to diagnose and fix #7164 . */
         smartlist_t *nodes = nodelist_find_nodes_with_microdesc(*mdp);
         const networkstatus_t *ns = networkstatus_get_latest_consensus();
-        int networkstatus_age = -1;
+        long networkstatus_age = -1;
         if (ns) {
           networkstatus_age = now - ns->valid_after;
         }
         log_warn(LD_BUG, "Microdescriptor seemed very old "
                  "(last listed %d hours ago vs %d hour cutoff), but is still "
                  "marked as being held by %d node(s). I found %d node(s) "
-                 "holding it. Current networkstatus is %d hours old.",
+                 "holding it. Current networkstatus is %ld hours old.",
                  (int)((now - (*mdp)->last_listed) / 3600),
                  (int)((now - cutoff) / 3600),
                  held_by_nodes,
                  smartlist_len(nodes),
-                 (int)(networkstatus_age / 3600));
+                 networkstatus_age / 3600);
 
         SMARTLIST_FOREACH_BEGIN(nodes, const node_t *, node) {
           const char *rs_match = "No RS";



More information about the tor-commits mailing list