Merge branch 'bug29959_040_squashed' into maint-0.4.0

torproject · Apr 5, 2019 · d016bba · d016bba
2 parents 42aae0e + 8e961b2
commit d016bba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 11 deletions.
diff --git a/changes/bug29959-040 b/changes/bug29959-040
@@ -0,0 +1,3 @@
+  o Minor bugfixes (directory authorities):
+    - Actually include the bandwidth-file-digest line in directory authority
+      votes. Fixes bug 29959; bugfix on 0.4.0.2-alpha.
diff --git a/src/feature/dirauth/dirvote.c b/src/feature/dirauth/dirvote.c
@@ -320,18 +320,17 @@ format_networkstatus_vote(crypto_pk_t *private_signing_key,
     if (!tor_digest256_is_zero((const char *)v3_ns->bw_file_digest256)) {
       /* Encode the digest. */
       char b64_digest_bw_file[BASE64_DIGEST256_LEN+1] = {0};
-      if (digest256_to_base64(b64_digest_bw_file,
-                              (const char *)v3_ns->bw_file_digest256)>0) {
-        /* "bandwidth-file-digest" 1*(SP algorithm "=" digest) NL */
-        char *digest_algo_b64_digest_bw_file = NULL;
-        tor_asprintf(&digest_algo_b64_digest_bw_file, "%s=%s",
-                     crypto_digest_algorithm_get_name(DIGEST_ALG_BW_FILE),
-                     b64_digest_bw_file);
-        /* No need for tor_strdup(""), format_line_if_present does it. */
-        bw_file_digest = format_line_if_present(
+      digest256_to_base64(b64_digest_bw_file,
+                          (const char *)v3_ns->bw_file_digest256);
+      /* "bandwidth-file-digest" 1*(SP algorithm "=" digest) NL */
+      char *digest_algo_b64_digest_bw_file = NULL;
+      tor_asprintf(&digest_algo_b64_digest_bw_file, "%s=%s",
+                   crypto_digest_algorithm_get_name(DIGEST_ALG_BW_FILE),
+                   b64_digest_bw_file);
+      /* No need for tor_strdup(""), format_line_if_present does it. */
+      bw_file_digest = format_line_if_present(
           "bandwidth-file-digest", digest_algo_b64_digest_bw_file);
-        tor_free(digest_algo_b64_digest_bw_file);
-      }
+      tor_free(digest_algo_b64_digest_bw_file);
     }
 
     smartlist_add_asprintf(chunks,