Fix some proposal-104 bugs.

svn:r9972

diff --git a/src/or/or.h b/src/or/or.h
index ff9dfe397f8f61a1695a23d5c514c08920ea0877..f7ee77e5050f58a9b368fdfda01fda38a9c36647 100644 (file)
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -2722,7 +2722,7 @@ void rep_hist_dump_stats(time_t now, int severity);
 void rep_hist_note_bytes_read(int num_bytes, time_t when);
 void rep_hist_note_bytes_written(int num_bytes, time_t when);
 int rep_hist_bandwidth_assess(void);
-char *rep_hist_get_bandwidth_lines(void);
+char *rep_hist_get_bandwidth_lines(int for_extrainfo);
 void rep_hist_update_state(or_state_t *state);
 int rep_hist_load_state(or_state_t *state, char **err);
 void rep_history_clean(time_t before);

diff --git a/src/or/rephist.c b/src/or/rephist.c
index 4f18a093e0a4dd4c4b14f994155f9ae487302816..c41a334687dd611802cfb0f70124eeaf914c2f59 100644 (file)
--- a/src/or/rephist.c
+++ b/src/or/rephist.c
@@ -613,7 +613,7 @@ rep_hist_fill_bandwidth_history(char *buf, size_t len, bw_array_t *b)
  * history in its descriptor.
  */
 char *
-rep_hist_get_bandwidth_lines(void)
+rep_hist_get_bandwidth_lines(int for_extrainfo)
 {
   char *buf, *cp;
   char t[ISO_TIME_LEN+1];
@@ -629,7 +629,8 @@ rep_hist_get_bandwidth_lines(void)
     b = r?read_array:write_array;
     tor_assert(b);
     format_iso_time(t, b->next_period-NUM_SECS_BW_SUM_INTERVAL);
-    tor_snprintf(cp, len-(cp-buf), "opt %s %s (%d s) ",
+    tor_snprintf(cp, len-(cp-buf), "%s%s %s (%d s) ",
+                 for_extrainfo ? "" : "opt ", 
                  r ? "read-history" : "write-history", t,
                  NUM_SECS_BW_SUM_INTERVAL);
     cp += strlen(cp);

diff --git a/src/or/router.c b/src/or/router.c
index c93f40bca6107aaeacef801330885e1c1eed5278..27c803159b6a7785231a7307530fe5bf1dc2ee2a 100644 (file)
--- a/src/or/router.c
+++ b/src/or/router.c
@@ -1244,7 +1244,7 @@ router_dump_router_to_string(char *s, size_t maxlen, routerinfo_t *router,
   format_iso_time(published, router->cache_info.published_on);
 
   /* How busy have we been? */
-  bandwidth_usage = rep_hist_get_bandwidth_lines();
+  bandwidth_usage = rep_hist_get_bandwidth_lines(0);
 
   if (router->declared_family && smartlist_len(router->declared_family)) {
     size_t n;
@@ -1397,7 +1397,7 @@ extrainfo_dump_to_string(char *s, size_t maxlen, extrainfo_t *extrainfo,
   base16_encode(identity, sizeof(identity),
                 extrainfo->cache_info.identity_digest, DIGEST_LEN);
   format_iso_time(published, extrainfo->cache_info.published_on);
-  bandwidth_usage = rep_hist_get_bandwidth_lines();
+  bandwidth_usage = rep_hist_get_bandwidth_lines(1);
 
   result = tor_snprintf(s, maxlen,
                         "extra-info %s %s\n"

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 9b77e3801f8db9e4b17dec74bc4194ef2fed5415..63f3eec297831a9189eb581c4ced3351510e8bf9 100644 (file)
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -1468,7 +1468,7 @@ signed_descriptor_get_body(signed_descriptor_t *desc)
     r = desc->signed_descriptor_body;
   }
   tor_assert(r);
-  tor_assert(!memcmp("router ", r, 7));
+  tor_assert(!memcmp("router ", r, 7) || !memcmp("extra-info ", r, 11));
 #if 0
   tor_assert(!memcmp("\n-----END SIGNATURE-----\n",
                      r + len - 25, 25));

diff --git a/src/or/test.c b/src/or/test.c
index 6d2736555f81df0dcf3e83f97bb36c64cbb7efde..73de80fe578ec28b03154903df725acc31a33bde 100644 (file)
--- a/src/or/test.c
+++ b/src/or/test.c
@@ -1716,7 +1716,7 @@ test_dir_format(void)
   r2.exit_policy = &ex1;
   r2.nickname = tor_strdup("Fred");
 
-  bw_lines = rep_hist_get_bandwidth_lines();
+  bw_lines = rep_hist_get_bandwidth_lines(0);
   test_assert(bw_lines);
   test_assert(!strcmpstart(bw_lines, "opt write-history "));