summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Ransom <rransom.8774@gmail.com>2011-09-10 21:40:09 -0400
committerRobert Ransom <rransom.8774@gmail.com>2011-09-10 21:56:05 -0400
commit8ea6d29fe468dc91d1c7dfc45ad5b4196e89ebdd (patch)
treeb13b33ce3cd9f22e76fc759d15f1e62fcf963b5a
parentb10735903b8135c9295e356ad7ea2222a339ab14 (diff)
downloadtor-8ea6d29fe468dc91d1c7dfc45ad5b4196e89ebdd.tar.gz
tor-8ea6d29fe468dc91d1c7dfc45ad5b4196e89ebdd.zip
Demote 'INTRODUCE2 cell is too {old,new}' message to info level
-rw-r--r--changes/bug2442b8
-rw-r--r--src/or/rendservice.c4
2 files changed, 11 insertions, 1 deletions
diff --git a/changes/bug2442b b/changes/bug2442b
new file mode 100644
index 0000000000..02e1636e91
--- /dev/null
+++ b/changes/bug2442b
@@ -0,0 +1,8 @@
+ * Minor bugfixes:
+
+ - Demote the 'INTRODUCE2 cell is too {old,new}' log message to
+ info level. There is nothing that a hidden service's operator
+ can do to fix its clients' clocks. Bugfix on 0.2.1.6-alpha;
+ fixes part of bug 2442.
+
+
diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index 5f55537bc0..6ed9650fc1 100644
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -1019,7 +1019,9 @@ rend_service_introduce(origin_circuit_t *circuit, const uint8_t *request,
v3_shift += 4;
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.",
+ /* This is far more likely to mean that a client's clock is
+ * skewed than that a replay attack is in progress. */
+ log_info(LD_REND, "INTRODUCE2 cell is too %s. Discarding.",
(now - ts) < 0 ? "old" : "new");
return -1;
}