Fix a crash bug when testing reachability

Fixes bug 25415; bugfix on 0.3.3.2-alpha.

diff --git a/changes/bug25415 b/changes/bug25415
new file mode 100644 (file)
index 0000000..ec851ae
--- /dev/null
+++ b/changes/bug25415
@@ -0,0 +1,4 @@
+  o Major bugfixes (directory authority):
+    - Avoid a crash when testing router reachability on a router that could
+      have an ed25519 ID, but which does not. Fixes bug 25415; bugfix on
+      0.3.3.2-alpha.

diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 981efc67f71a2c60fbf400efb60a8c549f55c5ed..1c1610ff93acaf19a560bc85fea890a134e16cd3 100644 (file)
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -3402,7 +3402,8 @@ dirserv_single_reachability_test(time_t now, routerinfo_t *router)
   tor_assert(node);
 
   if (options->AuthDirTestEd25519LinkKeys &&
-      node_supports_ed25519_link_authentication(node, 1)) {
+      node_supports_ed25519_link_authentication(node, 1) &&
+      router->cache_info.signing_key_cert) {
     ed_id_key = &router->cache_info.signing_key_cert->signing_key;
   } else {
     ed_id_key = NULL;