[tor-commits] [tor/master] In comments and logs, say "UTC" not "GMT"

nickm at torproject.org nickm at torproject.org
Fri Nov 23 15:06:48 UTC 2012


commit 864e15cd1c596b2a9dbc610d3aa78b49a4f33355
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Nov 23 10:05:16 2012 -0500

    In comments and logs, say "UTC" not "GMT"
    
    Fix for #6113.
    
    Note that the RFC1123 times we generate still all say 'GMT'.  I'm
    going to suggest this is not worth changing.
---
 changes/bug6113        |    3 +++
 src/common/tortls.c    |    2 +-
 src/common/util.c      |   11 ++++++-----
 src/or/confparse.h     |    2 +-
 src/or/hibernate.c     |    4 ++--
 src/or/networkstatus.c |    4 ++--
 src/or/statefile.c     |    2 +-
 7 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/changes/bug6113 b/changes/bug6113
new file mode 100644
index 0000000..436243d
--- /dev/null
+++ b/changes/bug6113
@@ -0,0 +1,3 @@
+  o Trivial bugfixes:
+    - Fix log messages and comments to avoid saying "GMT" when we mean
+      "UTC". Fixes bug 6113.
diff --git a/src/common/tortls.c b/src/common/tortls.c
index 76924b3..af3059a 100644
--- a/src/common/tortls.c
+++ b/src/common/tortls.c
@@ -2077,7 +2077,7 @@ log_cert_lifetime(int severity, const X509 *cert, const char *problem)
   BIO_get_mem_ptr(bio, &buf);
   s2 = tor_strndup(buf->data, buf->length);
 
-  strftime(mytime, 32, "%b %d %H:%M:%S %Y GMT", tor_gmtime_r(&now, &tm));
+  strftime(mytime, 32, "%b %d %H:%M:%S %Y UTC", tor_gmtime_r(&now, &tm));
 
   log(severity, LD_GENERAL,
       "(certificate lifetime runs from %s through %s. Your time is %s.)",
diff --git a/src/common/util.c b/src/common/util.c
index 1b0603a..44cdd64 100644
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -1342,7 +1342,7 @@ n_leapdays(int y1, int y2)
 static const int days_per_month[] =
   { 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
 
-/** Compute a time_t given a struct tm.  The result is given in GMT, and
+/** Compute a time_t given a struct tm.  The result is given in UTC, and
  * does not account for leap seconds.  Return 0 on success, -1 on failure.
  */
 int
@@ -1383,10 +1383,11 @@ static const char *MONTH_NAMES[] =
   { "Jan", "Feb", "Mar", "Apr", "May", "Jun",
     "Jul", "Aug", "Sep", "Oct", "Nov", "Dec" };
 
-/** Set <b>buf</b> to the RFC1123 encoding of the GMT value of <b>t</b>.
+/** Set <b>buf</b> to the RFC1123 encoding of the UTC value of <b>t</b>.
  * The buffer must be at least RFC1123_TIME_LEN+1 bytes long.
  *
- * (RFC1123 format is Fri, 29 Sep 2006 15:54:20 GMT)
+ * (RFC1123 format is "Fri, 29 Sep 2006 15:54:20 GMT". Note the "GMT"
+ * rather than "UTC".)
  */
 void
 format_rfc1123_time(char *buf, time_t t)
@@ -1404,8 +1405,8 @@ format_rfc1123_time(char *buf, time_t t)
   memcpy(buf+8, MONTH_NAMES[tm.tm_mon], 3);
 }
 
-/** Parse the RFC1123 encoding of some time (in GMT) from <b>buf</b>,
- * and store the result in *<b>t</b>.
+/** Parse the (a subset of) the RFC1123 encoding of some time (in UTC) from
+ * <b>buf</b>, and store the result in *<b>t</b>.
  *
  * Return 0 on success, -1 on failure.
 */
diff --git a/src/or/confparse.h b/src/or/confparse.h
index f33208e..20616be 100644
--- a/src/or/confparse.h
+++ b/src/or/confparse.h
@@ -23,7 +23,7 @@ typedef enum config_type_t {
   CONFIG_TYPE_BOOL,         /**< A boolean value, expressed as 0 or 1. */
   CONFIG_TYPE_AUTOBOOL,     /**< A boolean+auto value, expressed 0 for false,
                              * 1 for true, and -1 for auto  */
-  CONFIG_TYPE_ISOTIME,      /**< An ISO-formatted time relative to GMT. */
+  CONFIG_TYPE_ISOTIME,      /**< An ISO-formatted time relative to UTC. */
   CONFIG_TYPE_CSV,          /**< A list of strings, separated by commas and
                               * optional whitespace. */
   CONFIG_TYPE_LINELIST,     /**< Uninterpreted config lines */
diff --git a/src/or/hibernate.c b/src/or/hibernate.c
index 9e54eb4..7208996 100644
--- a/src/or/hibernate.c
+++ b/src/or/hibernate.c
@@ -890,12 +890,12 @@ hibernate_end_time_elapsed(time_t now)
       /* We weren't sleeping before; we should sleep now. */
       log_notice(LD_ACCT,
                  "Accounting period ended. Commencing hibernation until "
-                 "%s GMT", buf);
+                 "%s UTC", buf);
       hibernate_go_dormant(now);
     } else {
       log_notice(LD_ACCT,
              "Accounting period ended. This period, we will hibernate"
-             " until %s GMT",buf);
+             " until %s UTC",buf);
     }
   }
 }
diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c
index f33de4c..63426e8 100644
--- a/src/or/networkstatus.c
+++ b/src/or/networkstatus.c
@@ -775,7 +775,7 @@ router_set_networkstatus_v2(const char *s, time_t arrived_at,
     long delta = now - ns->published_on;
     format_time_interval(dbuf, sizeof(dbuf), delta);
     log_warn(LD_GENERAL, "Network status from %s was published %s in the "
-             "future (%s GMT). Check your time and date settings! "
+             "future (%s UTC). Check your time and date settings! "
              "Not caching.",
              source_desc, dbuf, published);
     control_event_general_status(LOG_WARN,
@@ -1874,7 +1874,7 @@ networkstatus_set_current_consensus(const char *consensus,
     format_iso_time(tbuf, c->valid_after);
     format_time_interval(dbuf, sizeof(dbuf), delta);
     log_warn(LD_GENERAL, "Our clock is %s behind the time published in the "
-             "consensus network status document (%s GMT).  Tor needs an "
+             "consensus network status document (%s UTC).  Tor needs an "
              "accurate clock to work correctly. Please check your time and "
              "date settings!", dbuf, tbuf);
     control_event_general_status(LOG_WARN,
diff --git a/src/or/statefile.c b/src/or/statefile.c
index beb9cf8..55ba0c1 100644
--- a/src/or/statefile.c
+++ b/src/or/statefile.c
@@ -416,7 +416,7 @@ or_state_save(time_t now)
   format_local_iso_time(tbuf, now);
   tor_asprintf(&contents,
                "# Tor state file last generated on %s local time\n"
-               "# Other times below are in GMT\n"
+               "# Other times below are in UTC\n"
                "# You *do not* need to edit this file.\n\n%s",
                tbuf, state);
   tor_free(state);





More information about the tor-commits mailing list