Merge remote-tracking branch 'ahf-github/maint-0.2.9' into maint-0.2.9

This commit is contained in:
Nick Mathewson 2018-06-29 13:00:52 -04:00
commit b426e83f07
4 changed files with 9 additions and 2 deletions

View File

@ -2765,6 +2765,8 @@ frac_nodes_with_descriptors(const smartlist_t *sl,
if (node_has_descriptor(node)) if (node_has_descriptor(node))
n_with_descs++; n_with_descs++;
}); });
tor_free(bandwidths);
return ((double)n_with_descs) / (double)smartlist_len(sl); return ((double)n_with_descs) / (double)smartlist_len(sl);
} }

View File

@ -409,6 +409,7 @@ disk_state_parse_commits(sr_state_t *state,
if (commit == NULL) { if (commit == NULL) {
/* Ignore badly formed commit. It could also be a authority /* Ignore badly formed commit. It could also be a authority
* fingerprint that we don't know about so it shouldn't be used. */ * fingerprint that we don't know about so it shouldn't be used. */
smartlist_free(args);
continue; continue;
} }
/* We consider parseable commit from our disk state to be valid because /* We consider parseable commit from our disk state to be valid because

View File

@ -446,10 +446,10 @@ test_hs_auth_cookies(void *arg)
#define TEST_COOKIE_ENCODED_STEALTH "YWJjZGVmZ2hpamtsbW5vcB" #define TEST_COOKIE_ENCODED_STEALTH "YWJjZGVmZ2hpamtsbW5vcB"
#define TEST_COOKIE_ENCODED_INVALID "YWJjZGVmZ2hpamtsbW5vcD" #define TEST_COOKIE_ENCODED_INVALID "YWJjZGVmZ2hpamtsbW5vcD"
char *encoded_cookie; char *encoded_cookie = NULL;
uint8_t raw_cookie[REND_DESC_COOKIE_LEN]; uint8_t raw_cookie[REND_DESC_COOKIE_LEN];
rend_auth_type_t auth_type; rend_auth_type_t auth_type;
char *err_msg; char *err_msg = NULL;
int re; int re;
(void)arg; (void)arg;
@ -495,6 +495,9 @@ test_hs_auth_cookies(void *arg)
tor_free(err_msg); tor_free(err_msg);
done: done:
tor_free(encoded_cookie);
tor_free(err_msg);
return; return;
} }

View File

@ -788,6 +788,7 @@ test_sr_setup_commits(void)
tt_assert(!commit_has_reveal_value(commit_d)); tt_assert(!commit_has_reveal_value(commit_d));
done: done:
tor_free(place_holder);
authority_cert_free(auth_cert); authority_cert_free(auth_cert);
} }