[tor-commits] [tor/release-0.2.2] Fix two potential bugs in the bug1863 code.

arma at torproject.org arma at torproject.org
Fri Feb 25 18:48:55 UTC 2011


commit 0642b927928df4ec556a76fed4b0b4523d515a7d
Author: Karsten Loesing <karsten.loesing at gmx.net>
Date:   Thu Feb 24 16:44:54 2011 +0100

    Fix two potential bugs in the bug1863 code.
---
 src/or/rephist.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/or/rephist.c b/src/or/rephist.c
index 265fff8..3f4a704 100644
--- a/src/or/rephist.c
+++ b/src/or/rephist.c
@@ -1513,7 +1513,7 @@ rep_hist_update_bwhist_state_section(or_state_t *state,
     smartlist_free(*s_values);
   }
   if (*s_maxima) {
-    SMARTLIST_FOREACH(*s_values, char *, val, tor_free(val));
+    SMARTLIST_FOREACH(*s_maxima, char *, val, tor_free(val));
     smartlist_free(*s_maxima);
   }
   if (! server_mode(get_options())) {
@@ -1541,7 +1541,7 @@ rep_hist_update_bwhist_state_section(or_state_t *state,
   i = (b->num_maxes_set <= b->next_max_idx) ? 0 : b->next_max_idx;
   for (j=0; j < b->num_maxes_set; ++j,++i) {
     uint64_t maxval;
-    if (i > NUM_TOTALS)
+    if (i >= NUM_TOTALS)
       i = 0;
     tor_asprintf(&cp, U64_FORMAT, U64_PRINTF_ARG(b->totals[i] & ~0x3ff));
     smartlist_add(*s_values, cp);





More information about the tor-commits mailing list