Merge remote-tracking branch 'origin/maint-0.2.2'

This commit is contained in:
Nick Mathewson 2011-05-28 01:55:34 -04:00
commit 42b15a0aaa
9 changed files with 15 additions and 13 deletions

6
changes/bug3208 Normal file
View File

@ -0,0 +1,6 @@
o Removed options:
- Remove undocumented option "-F" from tor-resolve: it hasn't done
anything since 0.2.1.16-rc.
o Minor bugfixes:
- Fix warnings from GCC 4.6's "-Wunused-but-set-variable" option.

View File

@ -3782,7 +3782,6 @@ void
entry_guards_compute_status(or_options_t *options, time_t now) entry_guards_compute_status(or_options_t *options, time_t now)
{ {
int changed = 0; int changed = 0;
int severity = LOG_DEBUG;
digestmap_t *reasons; digestmap_t *reasons;
if (! entry_guards) if (! entry_guards)
@ -3809,8 +3808,6 @@ entry_guards_compute_status(or_options_t *options, time_t now)
if (remove_dead_entry_guards(now)) if (remove_dead_entry_guards(now))
changed = 1; changed = 1;
severity = changed ? LOG_DEBUG : LOG_INFO;
if (changed) { if (changed) {
SMARTLIST_FOREACH_BEGIN(entry_guards, entry_guard_t *, entry) { SMARTLIST_FOREACH_BEGIN(entry_guards, entry_guard_t *, entry) {
const char *reason = digestmap_get(reasons, entry->identity); const char *reason = digestmap_get(reasons, entry->identity);

View File

@ -649,6 +649,7 @@ command_process_netinfo_cell(cell_t *cell, or_connection_t *conn)
/* XXX maybe act on my_apparent_addr, if the source is sufficiently /* XXX maybe act on my_apparent_addr, if the source is sufficiently
* trustworthy. */ * trustworthy. */
(void)my_apparent_addr;
if (connection_or_set_state_open(conn)<0) if (connection_or_set_state_open(conn)<0)
connection_mark_for_close(TO_CONN(conn)); connection_mark_for_close(TO_CONN(conn));

View File

@ -3228,7 +3228,6 @@ control_event_circuit_status(origin_circuit_t *circ, circuit_status_event_t tp,
{ {
const char *status; const char *status;
char extended_buf[96]; char extended_buf[96];
int providing_reason=0;
if (!EVENT_IS_INTERESTING(EVENT_CIRCUIT_STATUS)) if (!EVENT_IS_INTERESTING(EVENT_CIRCUIT_STATUS))
return 0; return 0;
tor_assert(circ); tor_assert(circ);
@ -3252,7 +3251,6 @@ control_event_circuit_status(origin_circuit_t *circ, circuit_status_event_t tp,
const char *reason_str = circuit_end_reason_to_control_string(reason_code); const char *reason_str = circuit_end_reason_to_control_string(reason_code);
char *reason = NULL; char *reason = NULL;
size_t n=strlen(extended_buf); size_t n=strlen(extended_buf);
providing_reason=1;
if (!reason_str) { if (!reason_str) {
reason = tor_malloc(16); reason = tor_malloc(16);
tor_snprintf(reason, 16, "UNKNOWN_%d", reason_code); tor_snprintf(reason, 16, "UNKNOWN_%d", reason_code);

View File

@ -2005,7 +2005,6 @@ connection_dir_client_reached_eof(dir_connection_t *conn)
router_get_trusteddirserver_by_digest(conn->identity_digest); router_get_trusteddirserver_by_digest(conn->identity_digest);
char *rejected_hdr = http_get_header(headers, char *rejected_hdr = http_get_header(headers,
"X-Descriptor-Not-New: "); "X-Descriptor-Not-New: ");
int rejected = 0;
if (rejected_hdr) { if (rejected_hdr) {
if (!strcmp(rejected_hdr, "Yes")) { if (!strcmp(rejected_hdr, "Yes")) {
log_info(LD_GENERAL, log_info(LD_GENERAL,
@ -2018,7 +2017,6 @@ connection_dir_client_reached_eof(dir_connection_t *conn)
* last descriptor, not on the published time of the last * last descriptor, not on the published time of the last
* descriptor. If those are different, that's a bad thing to * descriptor. If those are different, that's a bad thing to
* do. -NM */ * do. -NM */
rejected = 1;
} }
tor_free(rejected_hdr); tor_free(rejected_hdr);
} }

View File

@ -1585,7 +1585,7 @@ networkstatus_compute_consensus(smartlist_t *votes,
* is the same flag as votes[j]->known_flags[b]. */ * is the same flag as votes[j]->known_flags[b]. */
int *named_flag; /* Index of the flag "Named" for votes[j] */ int *named_flag; /* Index of the flag "Named" for votes[j] */
int *unnamed_flag; /* Index of the flag "Unnamed" for votes[j] */ int *unnamed_flag; /* Index of the flag "Unnamed" for votes[j] */
int chosen_named_idx, chosen_unnamed_idx; int chosen_named_idx;
strmap_t *name_to_id_map = strmap_new(); strmap_t *name_to_id_map = strmap_new();
char conflict[DIGEST_LEN]; char conflict[DIGEST_LEN];
@ -1603,7 +1603,6 @@ networkstatus_compute_consensus(smartlist_t *votes,
for (i = 0; i < smartlist_len(votes); ++i) for (i = 0; i < smartlist_len(votes); ++i)
unnamed_flag[i] = named_flag[i] = -1; unnamed_flag[i] = named_flag[i] = -1;
chosen_named_idx = smartlist_string_pos(flags, "Named"); chosen_named_idx = smartlist_string_pos(flags, "Named");
chosen_unnamed_idx = smartlist_string_pos(flags, "Unnamed");
/* Build the flag index. */ /* Build the flag index. */
SMARTLIST_FOREACH(votes, networkstatus_t *, v, SMARTLIST_FOREACH(votes, networkstatus_t *, v,

View File

@ -1028,6 +1028,9 @@ request_parse(u8 *packet, ssize_t length, struct evdns_server_port *port, struct
GET16(answers); GET16(answers);
GET16(authority); GET16(authority);
GET16(additional); GET16(additional);
(void)additional;
(void)authority;
(void)answers;
if (flags & 0x8000) return -1; /* Must not be an answer. */ if (flags & 0x8000) return -1; /* Must not be an answer. */
flags &= 0x0110; /* Only RD and CD get preserved. */ flags &= 0x0110; /* Only RD and CD get preserved. */

View File

@ -69,7 +69,7 @@ test_crypto_rng(void)
uint64_t big; uint64_t big;
char *host; char *host;
j = crypto_rand_int(100); j = crypto_rand_int(100);
if (i < 0 || i >= 100) if (j < 0 || j >= 100)
allok = 0; allok = 0;
big = crypto_rand_uint64(U64_LITERAL(1)<<40); big = crypto_rand_uint64(U64_LITERAL(1)<<40);
if (big >= (U64_LITERAL(1)<<40)) if (big >= (U64_LITERAL(1)<<40))
@ -240,11 +240,13 @@ test_crypto_sha(void)
/* Test SHA-1 with a test vector from the specification. */ /* Test SHA-1 with a test vector from the specification. */
i = crypto_digest(data, "abc", 3); i = crypto_digest(data, "abc", 3);
test_memeq_hex(data, "A9993E364706816ABA3E25717850C26C9CD0D89D"); test_memeq_hex(data, "A9993E364706816ABA3E25717850C26C9CD0D89D");
tt_int_op(i, ==, 0);
/* Test SHA-256 with a test vector from the specification. */ /* Test SHA-256 with a test vector from the specification. */
i = crypto_digest256(data, "abc", 3, DIGEST_SHA256); i = crypto_digest256(data, "abc", 3, DIGEST_SHA256);
test_memeq_hex(data, "BA7816BF8F01CFEA414140DE5DAE2223B00361A3" test_memeq_hex(data, "BA7816BF8F01CFEA414140DE5DAE2223B00361A3"
"96177A9CB410FF61F20015AD"); "96177A9CB410FF61F20015AD");
tt_int_op(i, ==, 0);
/* Test HMAC-SHA-1 with test cases from RFC2202. */ /* Test HMAC-SHA-1 with test cases from RFC2202. */

View File

@ -319,7 +319,7 @@ main(int argc, char **argv)
{ {
uint32_t sockshost; uint32_t sockshost;
uint16_t socksport = 0, port_option = 0; uint16_t socksport = 0, port_option = 0;
int isSocks4 = 0, isVerbose = 0, isReverse = 0, force = 0; int isSocks4 = 0, isVerbose = 0, isReverse = 0;
char **arg; char **arg;
int n_args; int n_args;
struct in_addr a; struct in_addr a;
@ -349,8 +349,6 @@ main(int argc, char **argv)
isSocks4 = 0; isSocks4 = 0;
else if (!strcmp("-x", arg[0])) else if (!strcmp("-x", arg[0]))
isReverse = 1; isReverse = 1;
else if (!strcmp("-F", arg[0]))
force = 1;
else if (!strcmp("-p", arg[0])) { else if (!strcmp("-p", arg[0])) {
int p; int p;
if (n_args < 2) { if (n_args < 2) {