fix the other half of bug 1074

diff --git a/changes/bug1074-part2 b/changes/bug1074-part2
new file mode 100644 (file)
index 0000000..6644f49
--- /dev/null
+++ b/changes/bug1074-part2
@@ -0,0 +1,6 @@
+  o Major bugfixes:
+    - Stop sending a CLOCK_SKEW controller status event whenever
+      we fetch directory information from a relay that has a wrong clock.
+      Instead, only inform the controller when it's a trusted authority
+      that claims our clock is wrong. Bugfix on tor-0.1.2.6-alpha;
+      fixes the other half of bug 1074.

diff --git a/src/or/directory.c b/src/or/directory.c
index 09ebccdc4fe2f15a06e00413431823d5b44123a1..7150fce407fd2a5e95e4e7b08db14db6fd224e9d 100644 (file)
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -1468,9 +1468,10 @@ connection_dir_client_reached_eof(dir_connection_t *conn)
              delta>0 ? "ahead" : "behind", dbuf,
              delta>0 ? "behind" : "ahead");
       skewed = 1; /* don't check the recommended-versions line */
-      control_event_general_status(trusted ? LOG_WARN : LOG_NOTICE,
-                               "CLOCK_SKEW SKEW=%ld SOURCE=DIRSERV:%s:%d",
-                               delta, conn->_base.address, conn->_base.port);
+      if (trusted)
+        control_event_general_status(LOG_WARN,
+                                 "CLOCK_SKEW SKEW=%ld SOURCE=DIRSERV:%s:%d",
+                                 delta, conn->_base.address, conn->_base.port);
     } else {
       log_debug(LD_HTTP, "Time on received directory is within tolerance; "
                 "we are %ld seconds skewed.  (That's okay.)", delta);