[tor-commits] [tor/master] Fix offending unittest and add changes file for #23862.

[nickm@xxxxxxxxxxxxxx]

commit 1a6814c096b0fa6fbfd2a1688b29dbc8143d6568
Author: George Kadianakis <desnacked@xxxxxxxxxx>
Date:   Wed Oct 25 19:54:48 2017 +0300

    Fix offending unittest and add changes file for #23862.
---
 changes/bug23682           | 5 +++++
 src/test/test_routerlist.c | 3 +--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/changes/bug23682 b/changes/bug23682
new file mode 100644
index 000000000..301ce7367
--- /dev/null
+++ b/changes/bug23682
@@ -0,0 +1,5 @@
+  o Minor bugfixes (entry guards):
+    - Tor now updates its guard state when it reads a consensus regardless of
+      whether it's missing descriptors. That makes tor use its primary guards
+      to fetch descriptors in some edge cases where it would have used fallback
+      directories in the past. Fixes bug 23862; bugfix on 0.3.0.1-alpha.
\ No newline at end of file
diff --git a/src/test/test_routerlist.c b/src/test/test_routerlist.c
index b131dd959..81451a4bc 100644
--- a/src/test/test_routerlist.c
+++ b/src/test/test_routerlist.c
@@ -508,8 +508,7 @@ test_directory_guard_fetch_with_no_dirinfo(void *arg)
   tt_int_op(retval, OP_EQ, 0);
 
   /* Make sure that our primary guard was chosen */
-  /* BUG #23862 falls back to dirserver list!! */
-  expect_log_msg_containing("falling back to dirserver list");
+  expect_log_msg_containing("Selected primary guard router3");
 
  done:
   tor_free(consensus_text_md);



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits