[tor-commits] [tor/release-0.2.6] Revert "Remove obsolete workaround in dirserv_thinks_router_is_hs_dir()"

nickm at torproject.org nickm at torproject.org
Wed May 13 15:05:57 UTC 2015


commit 26c344a563610fcd293cd82bbb0b08dce737e5f3
Author: David Goulet <dgoulet at ev0ke.net>
Date:   Tue Apr 28 14:22:49 2015 -0400

    Revert "Remove obsolete workaround in dirserv_thinks_router_is_hs_dir()"
    
    Fixes #15850, part of #15801. Change file is added by this commit. The
    original comment in the reverted commit is removed because right now we
    *need* a DirPort until #15849 is implemented so no doubt nor confusion there
    anymore.
    
    This reverts commit 80bed1ac96a3035f8c55ddced5528f0d7d16d386.
    
    Signed-off-by: David Goulet <dgoulet at ev0ke.net>
---
 changes/bug15850 |    4 ++++
 src/or/dirserv.c |    2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/changes/bug15850 b/changes/bug15850
new file mode 100644
index 0000000..48a7c7b
--- /dev/null
+++ b/changes/bug15850
@@ -0,0 +1,4 @@
+  o Major bugfix
+    - Revert commit that made directory authority assign the HSDir flag to
+      relay without a DirPort which is bad because relay can't handle
+      BEGIN_DIR cells. Fixes #15850. Bugfix on tor-0.2.6.3-alpha;
diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index e5a5b54..a024be8 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -1305,7 +1305,7 @@ dirserv_thinks_router_is_hs_dir(const routerinfo_t *router,
   else
     uptime = real_uptime(router, now);
 
-  return (router->wants_to_be_hs_dir &&
+  return (router->wants_to_be_hs_dir && router->dir_port &&
           uptime >= get_options()->MinUptimeHidServDirectoryV2 &&
           router_is_active(router, node, now));
 }





More information about the tor-commits mailing list