February 2016 - tor-commits - lists.torproject.org

[tor/master] Fix memory leaks in routerlist/pick_directory_server_impl test
by nickm＠torproject.org 24 Feb '16

24 Feb '16

commit a3f764ea692cefa534d05e6e661b8b9c3a9e8429 Author: Nick Mathewson <nickm(a)torproject.org> Date: Wed Feb 24 15:13:29 2016 -0500 Fix memory leaks in routerlist/pick_directory_server_impl test --- src/test/test_routerlist.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/src/test/test_routerlist.c b/src/test/test_routerlist.c index 31533d0..8b37cc0 100644 --- a/src/test/test_routerlist.c +++ b/src/test/test_routerlist.c @@ -30,7 +30,7 @@ extern const char AUTHORITY_SIGNKEY_2[]; extern const char AUTHORITY_CERT_3[]; extern const char AUTHORITY_SIGNKEY_3[]; -void construct_consensus(const char **consensus_text_md); +void construct_consensus(char **consensus_text_md); /* 4 digests + 3 sep + pre + post + NULL */ static char output[4*BASE64_DIGEST256_LEN+3+2+2+1]; @@ -120,7 +120,7 @@ test_routerlist_launch_descriptor_downloads(void *arg) } void -construct_consensus(const char **consensus_text_md) +construct_consensus(char **consensus_text_md) { networkstatus_t *vote = NULL; networkstatus_t *v1 = NULL, *v2 = NULL, *v3 = NULL; @@ -180,6 +180,14 @@ construct_consensus(const char **consensus_text_md) networkstatus_vote_free(v1); networkstatus_vote_free(v2); networkstatus_vote_free(v3); + smartlist_free(votes); + authority_cert_free(cert1); + authority_cert_free(cert2); + authority_cert_free(cert3); + crypto_pk_free(sign_skey_1); + crypto_pk_free(sign_skey_2); + crypto_pk_free(sign_skey_3); + crypto_pk_free(sign_skey_leg); } static void @@ -188,7 +196,7 @@ test_router_pick_directory_server_impl(void *arg) (void)arg; networkstatus_t *con_md = NULL; - const char *consensus_text_md = NULL; + char *consensus_text_md = NULL; int flags = PDS_IGNORE_FASCISTFIREWALL|PDS_RETRY_IF_NO_SERVERS; or_options_t *options = get_options_mutable(); const routerstatus_t *rs = NULL; @@ -363,6 +371,8 @@ test_router_pick_directory_server_impl(void *arg) if (options->ReachableORAddresses || options->ReachableDirAddresses) policies_free_all(); + tor_free(consensus_text_md); + networkstatus_vote_free(con_md); } connection_t *mocked_connection = NULL;

1 0

[tor/master] Fix a leak in test_have_enough_mem_for_dircache
by nickm＠torproject.org 24 Feb '16

24 Feb '16

commit ba0ddd7467329bb7b6b201c0bd11bf99abcd0d7e Author: Nick Mathewson <nickm(a)torproject.org> Date: Wed Feb 24 15:06:12 2016 -0500 Fix a leak in test_have_enough_mem_for_dircache --- src/test/test_options.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/test_options.c b/src/test/test_options.c index 547464a..68e8f91 100644 --- a/src/test/test_options.c +++ b/src/test/test_options.c @@ -210,7 +210,7 @@ test_have_enough_mem_for_dircache(void *arg) { (void)arg; or_options_t *opt=NULL; - or_options_t *dflt; + or_options_t *dflt=NULL; config_line_t *cl=NULL; char *msg=NULL;; int r; @@ -292,7 +292,7 @@ test_have_enough_mem_for_dircache(void *arg) done: if (msg) tor_free(msg); - tor_free(dflt); + or_options_free(dflt); or_options_free(opt); config_free_lines(cl); return;

1 0

[tor-messenger-build/master] Account for async send when closing convo window w/ otr
by arlo＠torproject.org 24 Feb '16

24 Feb '16

commit ec7194b41700eee0c1f583c321538ace92f6de68 Author: Arlo Breault <arlolra(a)gmail.com> Date: Wed Feb 24 11:50:22 2016 -0800 Account for async send when closing convo window w/ otr --- projects/instantbird/trac-13312.patch | 29 +++++++++++++++++------------ 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/projects/instantbird/trac-13312.patch b/projects/instantbird/trac-13312.patch index 37a81a7..7eb92b2 100644 --- a/projects/instantbird/trac-13312.patch +++ b/projects/instantbird/trac-13312.patch @@ -629,10 +629,10 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. +var NSGetFactory = XPCOMUtils.generateNSGetFactory([TwitterProtocol]); # HG changeset patch # User Arlo Breault <arlolra(a)gmail.com> -# Date 1456282265 28800 -# Tue Feb 23 18:51:05 2016 -0800 +# Date 1456343323 28800 +# Wed Feb 24 11:48:43 2016 -0800 # Branch THUNDERBIRD420b2_2015101216_RELBRANCH -# Node ID 01edd1bf5e56e382611823a8f0ab604743c78b02 +# Node ID 1302396bc5605fff3ba7b2b7a762db98630378fa # Parent 9d547deb66802e3068767b5df2e687f6a56c5099 Bug 955642 - Handle Twitter direct messages (DMs) @@ -1273,7 +1273,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. if (this.incoming) { actions.push( new Action(_("action.retweet"), function() { -@@ -118,17 +127,104 @@ function Action(aLabel, aAction, aTweet) +@@ -118,17 +127,109 @@ function Action(aLabel, aAction, aTweet) this._action = aAction; this._tweet = aTweet; } @@ -1299,6 +1299,11 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. + this.writeMessage("twitter.com", aMessage, flags); + }, + onSentCallback: function(aMsg, aData) { ++ // The conversation may have been unitialized in the time it takes for ++ // the async callback to fire. Use `_observers` as a proxy for uninit'd. ++ if (!Array.isArray(this._observers)) ++ return; ++ + let tweet = JSON.parse(aData); + // The OTR extension requires that the protocol not modify the message + // (see the notes at `imIOutgoingMessage`). That's the contract we made. @@ -1379,7 +1384,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. let entries = []; for (let [name, userInfo] of aAccount._userInfo) { entries.push([userInfo.id_str, name]); -@@ -139,17 +235,17 @@ function Conversation(aAccount) +@@ -139,17 +240,17 @@ function Conversation(aAccount) // If the user's info has already been received, update the timeline topic. if (aAccount._userInfo.has(aAccount.name)) { @@ -1398,7 +1403,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. inReplyToStatusId: null, startReply: function(aTweet) { this.inReplyToStatusId = aTweet.id_str; -@@ -169,203 +265,66 @@ Conversation.prototype = { +@@ -169,203 +270,66 @@ Conversation.prototype = { .map(aNick => "@" + aNick) .join(" ") + " "; @@ -1627,7 +1632,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. let chatBuddy = new ChatBuddy(aNick, this._account); this._participants.set(aNick, chatBuddy); this.notifyObservers(new nsSimpleEnumerator([chatBuddy]), -@@ -377,23 +336,60 @@ Conversation.prototype = { +@@ -377,23 +341,60 @@ Conversation.prototype = { set nick(aNick) {}, get topicSettable() { return this.nick == this._account.name; }, get topic() { return this._topic; }, // can't add a setter without redefining the getter @@ -1688,7 +1693,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. // Unfortunately, for backwards compatibility we retain this normalization, // which can cause edge cases for usernames with underscores. normalize: aString => aString.replace(/[^a-z0-9]/gi, "").toLowerCase(), -@@ -554,16 +550,21 @@ Account.prototype = { +@@ -554,16 +555,21 @@ Account.prototype = { reTweet: function(aTweet, aOnSent, aOnError, aThis) { let url = "1.1/statuses/retweet/" + aTweet.id_str + ".json"; this.signAndSend(url, null, [], aOnSent, aOnError, aThis); @@ -1710,7 +1715,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. }, stopFollowing: function(aUserName) { // friendships/destroy will return the user in case of success. -@@ -615,39 +616,17 @@ Account.prototype = { +@@ -615,39 +621,17 @@ Account.prototype = { getParams = "?q=" + trackQuery + lastMsgParam + "&count=100"; let url = "1.1/search/tweets.json" + getParams; this._pendingRequests.push( @@ -1751,7 +1756,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. this._doneWithTimelineRequest(aRequest); }, -@@ -685,17 +664,17 @@ Account.prototype = { +@@ -685,17 +669,17 @@ Account.prototype = { this.reportConnected(); // If the conversation already exists, notify it we are back online. @@ -1770,7 +1775,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. delete this._timelineBuffer; delete this._pendingRequests; -@@ -747,18 +726,23 @@ Account.prototype = { +@@ -747,18 +731,23 @@ Account.prototype = { continue; let msg; try { @@ -1796,7 +1801,7 @@ diff --git a/chat/protocols/twitter/twitter.js b/chat/protocols/twitter/twitter. userInfoIds.add(userInfo.id_str); let ids = msg.friends.filter( aId => !userInfoIds.has(aId.toString())); -@@ -1099,16 +1083,29 @@ Account.prototype = { +@@ -1099,16 +1088,29 @@ Account.prototype = { this.config = JSON.parse(aData); },

1 0

[translation/https_everywhere_completed] Update translations for https_everywhere_completed
by translation＠torproject.org 24 Feb '16

24 Feb '16

commit 7e5a10b477635053c7b8b48636cc77435c629ab1 Author: Translation commit bot <translation(a)torproject.org> Date: Wed Feb 24 19:45:21 2016 +0000 Update translations for https_everywhere_completed --- hu/https-everywhere.dtd | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/hu/https-everywhere.dtd b/hu/https-everywhere.dtd index 3816a69..a8f0342 100644 --- a/hu/https-everywhere.dtd +++ b/hu/https-everywhere.dtd @@ -14,6 +14,7 @@ <!ENTITY https-everywhere.about.or "vagy"> <!ENTITY https-everywhere.about.donate_eff "Az EFF támogatása"> +<!ENTITY https-everywhere.menu.donate_eff_imperative "Támogatás az EFF-nek"> <!ENTITY https-everywhere.menu.about "A HTTPS Everywhere névjegye"> <!ENTITY https-everywhere.menu.observatory "SSL Observatory Preferenciák"> <!ENTITY https-everywhere.menu.globalEnable "HTTPS Everywhere bekapcsolása"> @@ -41,3 +42,15 @@ <!ENTITY https-everywhere.chrome.stable_rules_description "Titkosított kapcsolatok kényszerítése ezekhez a weboldalakhoz:"> <!ENTITY https-everywhere.chrome.experimental_rules "Kísérleti szabályok"> <!ENTITY https-everywhere.chrome.experimental_rules_description "Figyelmeztetéseket és megszakításokat okozhat. Alapértelmezésben letiltva."> +<!ENTITY https-everywhere.chrome.add_rule "Szabály felvétel az oldalhoz"> +<!ENTITY https-everywhere.chrome.add_new_rule "Új szabály az oldalhoz"> +<!ENTITY https-everywhere.chrome.always_https_for_host "Mindig https-t használjon ehhez a kiszolgálóhoz"> +<!ENTITY https-everywhere.chrome.host "Kiszolgáló:"> +<!ENTITY https-everywhere.chrome.show_advanced "Speciális beállítások megjelenítése"> +<!ENTITY https-everywhere.chrome.hide_advanced "Speciális beállítások elrejtése"> +<!ENTITY https-everywhere.chrome.rule_name "Szabály neve"> +<!ENTITY https-everywhere.chrome.regex "Illeszkedő regex"> +<!ENTITY https-everywhere.chrome.redirect_to "Átirányítás ide:"> +<!ENTITY https-everywhere.chrome.status_cancel_button "Mégsem"> + +

1 0

[translation/https_everywhere] Update translations for https_everywhere
by translation＠torproject.org 24 Feb '16

24 Feb '16

commit 243923410b2b265784d4d7b7c35ebb9a2eb0762a Author: Translation commit bot <translation(a)torproject.org> Date: Wed Feb 24 19:45:15 2016 +0000 Update translations for https_everywhere --- hu/https-everywhere.dtd | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/hu/https-everywhere.dtd b/hu/https-everywhere.dtd index 206e44a..a8f0342 100644 --- a/hu/https-everywhere.dtd +++ b/hu/https-everywhere.dtd @@ -14,7 +14,7 @@ <!ENTITY https-everywhere.about.or "vagy"> <!ENTITY https-everywhere.about.donate_eff "Az EFF támogatása"> -<!ENTITY https-everywhere.menu.donate_eff_imperative "Donate to EFF"> +<!ENTITY https-everywhere.menu.donate_eff_imperative "Támogatás az EFF-nek"> <!ENTITY https-everywhere.menu.about "A HTTPS Everywhere névjegye"> <!ENTITY https-everywhere.menu.observatory "SSL Observatory Preferenciák"> <!ENTITY https-everywhere.menu.globalEnable "HTTPS Everywhere bekapcsolása"> @@ -42,15 +42,15 @@ <!ENTITY https-everywhere.chrome.stable_rules_description "Titkosított kapcsolatok kényszerítése ezekhez a weboldalakhoz:"> <!ENTITY https-everywhere.chrome.experimental_rules "Kísérleti szabályok"> <!ENTITY https-everywhere.chrome.experimental_rules_description "Figyelmeztetéseket és megszakításokat okozhat. Alapértelmezésben letiltva."> -<!ENTITY https-everywhere.chrome.add_rule "Add a rule for this site"> -<!ENTITY https-everywhere.chrome.add_new_rule "Add a new rule for this site"> -<!ENTITY https-everywhere.chrome.always_https_for_host "Always use https for this host"> -<!ENTITY https-everywhere.chrome.host "Host:"> -<!ENTITY https-everywhere.chrome.show_advanced "Show advanced"> -<!ENTITY https-everywhere.chrome.hide_advanced "Hide advanced"> -<!ENTITY https-everywhere.chrome.rule_name "Rule name"> -<!ENTITY https-everywhere.chrome.regex "Matching regex"> -<!ENTITY https-everywhere.chrome.redirect_to "Redirect to"> -<!ENTITY https-everywhere.chrome.status_cancel_button "Cancel"> +<!ENTITY https-everywhere.chrome.add_rule "Szabály felvétel az oldalhoz"> +<!ENTITY https-everywhere.chrome.add_new_rule "Új szabály az oldalhoz"> +<!ENTITY https-everywhere.chrome.always_https_for_host "Mindig https-t használjon ehhez a kiszolgálóhoz"> +<!ENTITY https-everywhere.chrome.host "Kiszolgáló:"> +<!ENTITY https-everywhere.chrome.show_advanced "Speciális beállítások megjelenítése"> +<!ENTITY https-everywhere.chrome.hide_advanced "Speciális beállítások elrejtése"> +<!ENTITY https-everywhere.chrome.rule_name "Szabály neve"> +<!ENTITY https-everywhere.chrome.regex "Illeszkedő regex"> +<!ENTITY https-everywhere.chrome.redirect_to "Átirányítás ide:"> +<!ENTITY https-everywhere.chrome.status_cancel_button "Mégsem">

1 0

[tor/master] Remove the freelist from memarea.c
by nickm＠torproject.org 24 Feb '16

24 Feb '16

commit 73c433a48a15808c87fafbbfe43da60cc4ab7b0e Author: Nick Mathewson <nickm(a)torproject.org> Date: Wed Feb 24 14:32:09 2016 -0500 Remove the freelist from memarea.c This is in accordance with our usual policy against freelists, now that working allocators are everywhere. It should also make memarea.c's coverage higher. I also doubt that this code ever helped performance. --- changes/remove_memarea_freelist | 4 +++ src/common/memarea.c | 75 +++++++++++------------------------------ src/common/memarea.h | 1 - src/common/timing.c | 0 src/or/main.c | 1 - src/test/test_dir.c | 2 -- src/test/test_options.c | 24 ------------- 7 files changed, 23 insertions(+), 84 deletions(-) diff --git a/changes/remove_memarea_freelist b/changes/remove_memarea_freelist new file mode 100644 index 0000000..dd9520c --- /dev/null +++ b/changes/remove_memarea_freelist @@ -0,0 +1,4 @@ + o Removed code: + - We no longer maintain an internal freelist in memarea.c. Allocators + should be good enough to make this code unnecessary, and it's doubtful + that it ever had any performance benefit. diff --git a/src/common/memarea.c b/src/common/memarea.c index a8e6d45..b648c29 100644 --- a/src/common/memarea.c +++ b/src/common/memarea.c @@ -105,56 +105,32 @@ struct memarea_t { memarea_chunk_t *first; /**< Top of the chunk stack: never NULL. */ }; -/** How many chunks will we put into the freelist before freeing them? */ -#define MAX_FREELIST_LEN 4 -/** The number of memarea chunks currently in our freelist. */ -static int freelist_len=0; -/** A linked list of unused memory area chunks. Used to prevent us from - * spinning in malloc/free loops. */ -static memarea_chunk_t *freelist = NULL; - /** Helper: allocate a new memarea chunk of around chunk_size bytes. */ static memarea_chunk_t * -alloc_chunk(size_t sz, int freelist_ok) +alloc_chunk(size_t sz) { tor_assert(sz < SIZE_T_CEILING); - if (freelist && freelist_ok) { - memarea_chunk_t *res = freelist; - freelist = res->next_chunk; - res->next_chunk = NULL; - --freelist_len; - CHECK_SENTINEL(res); - return res; - } else { - size_t chunk_size = freelist_ok ? CHUNK_SIZE : sz; - memarea_chunk_t *res; - chunk_size += SENTINEL_LEN; - res = tor_malloc(chunk_size); - res->next_chunk = NULL; - res->mem_size = chunk_size - CHUNK_HEADER_SIZE - SENTINEL_LEN; - res->next_mem = res->U_MEM; - tor_assert(res->next_mem+res->mem_size+SENTINEL_LEN == - ((char*)res)+chunk_size); - tor_assert(realign_pointer(res->next_mem) == res->next_mem); - SET_SENTINEL(res); - return res; - } + + size_t chunk_size = sz < CHUNK_SIZE ? CHUNK_SIZE : sz; + memarea_chunk_t *res; + chunk_size += SENTINEL_LEN; + res = tor_malloc(chunk_size); + res->next_chunk = NULL; + res->mem_size = chunk_size - CHUNK_HEADER_SIZE - SENTINEL_LEN; + res->next_mem = res->U_MEM; + tor_assert(res->next_mem+res->mem_size+SENTINEL_LEN == + ((char*)res)+chunk_size); + tor_assert(realign_pointer(res->next_mem) == res->next_mem); + SET_SENTINEL(res); + return res; } -/** Release chunk from a memarea, either by adding it to the freelist - * or by freeing it if the freelist is already too big. */ +/** Release chunk from a memarea. */ static void chunk_free_unchecked(memarea_chunk_t *chunk) { CHECK_SENTINEL(chunk); - if (freelist_len < MAX_FREELIST_LEN) { - ++freelist_len; - chunk->next_chunk = freelist; - freelist = chunk; - chunk->next_mem = chunk->U_MEM; - } else { - tor_free(chunk); - } + tor_free(chunk); } /** Allocate and return new memarea. */ @@ -162,7 +138,7 @@ memarea_t * memarea_new(void) { memarea_t *head = tor_malloc(sizeof(memarea_t)); - head->first = alloc_chunk(CHUNK_SIZE, 1); + head->first = alloc_chunk(CHUNK_SIZE); return head; } @@ -197,19 +173,6 @@ memarea_clear(memarea_t *area) area->first->next_mem = area->first->U_MEM; } -/** Remove all unused memarea chunks from the internal freelist. */ -void -memarea_clear_freelist(void) -{ - memarea_chunk_t *chunk, *next; - freelist_len = 0; - for (chunk = freelist; chunk; chunk = next) { - next = chunk->next_chunk; - tor_free(chunk); - } - freelist = NULL; -} - /** Return true iff p is in a range that has been returned by an * allocation from area. */ int @@ -241,12 +204,12 @@ memarea_alloc(memarea_t *area, size_t sz) if (sz+CHUNK_HEADER_SIZE >= CHUNK_SIZE) { /* This allocation is too big. Stick it in a special chunk, and put * that chunk second in the list. */ - memarea_chunk_t *new_chunk = alloc_chunk(sz+CHUNK_HEADER_SIZE, 0); + memarea_chunk_t *new_chunk = alloc_chunk(sz+CHUNK_HEADER_SIZE); new_chunk->next_chunk = chunk->next_chunk; chunk->next_chunk = new_chunk; chunk = new_chunk; } else { - memarea_chunk_t *new_chunk = alloc_chunk(CHUNK_SIZE, 1); + memarea_chunk_t *new_chunk = alloc_chunk(CHUNK_SIZE); new_chunk->next_chunk = chunk; area->first = chunk = new_chunk; } diff --git a/src/common/memarea.h b/src/common/memarea.h index d14f3a2..4b2f3b4 100644 --- a/src/common/memarea.h +++ b/src/common/memarea.h @@ -18,7 +18,6 @@ char *memarea_strdup(memarea_t *area, const char *s); char *memarea_strndup(memarea_t *area, const char *s, size_t n); void memarea_get_stats(memarea_t *area, size_t *allocated_out, size_t *used_out); -void memarea_clear_freelist(void); void memarea_assert_ok(memarea_t *area); #endif diff --git a/src/common/timing.c b/src/common/timing.c new file mode 100644 index 0000000..e69de29 diff --git a/src/or/main.c b/src/or/main.c index 96a56fc..11caea5 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -3100,7 +3100,6 @@ tor_free_all(int postfork) connection_free_all(); connection_edge_free_all(); scheduler_free_all(); - memarea_clear_freelist(); nodelist_free_all(); microdesc_free_all(); ext_orport_free_all(); diff --git a/src/test/test_dir.c b/src/test/test_dir.c index 72e619f..49719b9 100644 --- a/src/test/test_dir.c +++ b/src/test/test_dir.c @@ -497,7 +497,6 @@ test_dir_routerinfo_parsing(void *arg) #undef CHECK_FAIL #undef CHECK_OK done: - memarea_clear_freelist(); routerinfo_free(ri); } @@ -601,7 +600,6 @@ test_dir_extrainfo_parsing(void *arg) done: escaped(NULL); - memarea_clear_freelist(); extrainfo_free(ei); routerinfo_free(ri); digestmap_free((digestmap_t*)map, routerinfo_free_wrapper_); diff --git a/src/test/test_options.c b/src/test/test_options.c index b2be8a9..547464a 100644 --- a/src/test/test_options.c +++ b/src/test/test_options.c @@ -158,7 +158,6 @@ test_options_validate_impl(const char *configuration, } done: - memarea_clear_freelist(); escaped(NULL); policies_free_all(); config_free_lines(cl); @@ -1949,7 +1948,6 @@ test_options_validate__use_bridges(void *ignored) done: NS_UNMOCK(geoip_get_country); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2187,7 +2185,6 @@ test_options_validate__publish_server_descriptor(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2263,7 +2260,6 @@ test_options_validate__testing(void *ignored) done: escaped(NULL); // This will free the leaking memory from the previous escaped - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2311,7 +2307,6 @@ test_options_validate__hidserv(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2336,7 +2331,6 @@ test_options_validate__predicted_ports(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2539,7 +2533,6 @@ test_options_validate__bandwidth(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2627,7 +2620,6 @@ test_options_validate__circuits(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -2661,7 +2653,6 @@ test_options_validate__port_forwarding(void *ignored) done: free_options_test_data(tdata); - memarea_clear_freelist(); policies_free_all(); tor_free(msg); } @@ -2691,7 +2682,6 @@ test_options_validate__tor2web(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -2758,7 +2748,6 @@ test_options_validate__rend(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -2878,7 +2867,6 @@ test_options_validate__accounting(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -3210,7 +3198,6 @@ test_options_validate__proxy(void *ignored) done: teardown_capture_of_logs(previous_log); free_options_test_data(tdata); - memarea_clear_freelist(); policies_free_all(); // sandbox_free_getaddrinfo_cache(); tor_free(msg); @@ -3438,7 +3425,6 @@ test_options_validate__control(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -3511,7 +3497,6 @@ test_options_validate__families(void *ignored) done: teardown_capture_of_logs(previous_log); - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -3536,7 +3521,6 @@ test_options_validate__addr_policies(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -3623,7 +3607,6 @@ test_options_validate__dir_auth(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -3749,7 +3732,6 @@ test_options_validate__transport(void *ignored) done: escaped(NULL); // This will free the leaking memory from the previous escaped - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -3833,7 +3815,6 @@ test_options_validate__constrained_sockets(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -4053,7 +4034,6 @@ test_options_validate__v3_auth(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -4088,7 +4068,6 @@ test_options_validate__virtual_addr(void *ignored) done: escaped(NULL); // This will free the leaking memory from the previous escaped - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg); @@ -4130,7 +4109,6 @@ test_options_validate__exits(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -4299,7 +4277,6 @@ test_options_validate__testing_options(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); teardown_capture_of_logs(previous_log); free_options_test_data(tdata); @@ -4353,7 +4330,6 @@ test_options_validate__accel(void *ignored) tor_free(msg); done: - memarea_clear_freelist(); policies_free_all(); free_options_test_data(tdata); tor_free(msg);

1 0

[tor-messenger-build/master] Bump ctypes-otr
by arlo＠torproject.org 24 Feb '16

24 Feb '16

commit 1f5b6dd26ac09b5947ab8bc83d0bec5088069d65 Author: Arlo Breault <arlolra(a)gmail.com> Date: Wed Feb 24 10:59:53 2016 -0800 Bump ctypes-otr --- projects/ctypes-otr/config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/projects/ctypes-otr/config b/projects/ctypes-otr/config index 299300c..611116d 100644 --- a/projects/ctypes-otr/config +++ b/projects/ctypes-otr/config @@ -1,7 +1,7 @@ # vim: filetype=yaml sw=2 version: '[% c("abbrev") %]' git_url: https://github.com/arlolra/ctypes-otr -git_hash: 61a6472c732dc4a03b5b315c645c9f265a67b9c6 +git_hash: d733f71db6b08f25c3e01fed071d3fe84bab6352 filename: 'ctypes-otr-[% c("version") %]-[% c("var/osname") %]-[% c("var/build_id") %].tar.gz' var: lib_glob: 'lib/*.so*'

1 0

[stem/master] Minor rewording for fallback directory addition
by atagar＠torproject.org 24 Feb '16

24 Feb '16

commit 7376fedaeb8b26397736e6f4bc02f913d86f49ca Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 24 10:01:00 2016 -0800 Minor rewording for fallback directory addition --- docs/change_log.rst | 3 ++- stem/descriptor/remote.py | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/docs/change_log.rst b/docs/change_log.rst index c8dc434..7af44da 100644 --- a/docs/change_log.rst +++ b/docs/change_log.rst @@ -58,7 +58,8 @@ The following are only available within Stem's `git repository * **Descriptors** - * Fallback directory information, lessing the load of `stem.descriptor.remote <api/descriptor/remote.html>`_ on the directory authorities + * Added `fallback directory information <api/descriptor/remote.html#stem.descriptor.remote.FallbackDirectory>`_. + * Lessened dirauth load of `stem.descriptor.remote <api/descriptor/remote.html>`_ by using fallback directories as well * Support for ed25519 descriptor fields (:spec:`5a79d67`) * Server descriptor validation fails with 'extra-info-digest line had an invalid value' from additions in proposal 228 (:trac:`16227`) * :class:`~stem.descriptor.server_descriptor.BridgeDescriptor` now has 'ntor_onion_key' like its unsanitized counterparts diff --git a/stem/descriptor/remote.py b/stem/descriptor/remote.py index 6739df5..13b49ab 100644 --- a/stem/descriptor/remote.py +++ b/stem/descriptor/remote.py @@ -816,10 +816,10 @@ def get_authorities(): class FallbackDirectory(Directory): """ - Tor directories tor uses as alternates for the authorities. These relays are - `hardcoded in tor <https://gitweb.torproject.org/tor.git/tree/src/or/fallback_dirs.inc>`_. + Particularly stable relays tor can instead of authorities when + bootstrapping. These relays are `hardcoded in tor <https://gitweb.torproject.org/tor.git/tree/src/or/fallback_dirs.inc>`_. - For example, following checks the performance of tor's fallback directories... + For example, the following checks the performance of tor's fallback directories... ::

1 0

[stem/master] Integ test failed when tor versions didn't include a space
by atagar＠torproject.org 24 Feb '16

24 Feb '16

commit bc0ba05b8c8d68c2085299e9e8f008ffbf622aa7 Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 24 09:44:39 2016 -0800 Integ test failed when tor versions didn't include a space Great catch from toralf. When using a tor release rather than the git version test_getinfo_version_parsing fails with... ValueError: '0.2.7.' isn't a properly formatted tor version --- test/integ/version.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/integ/version.py b/test/integ/version.py index 8347b35..7ba8e7a 100644 --- a/test/integ/version.py +++ b/test/integ/version.py @@ -62,5 +62,5 @@ class TestVersion(unittest.TestCase): # 250 OK tor_version = list(version_response)[0] - tor_version = tor_version[8:tor_version.find(' ', 8)] + tor_version = tor_version[8:].split(' ', 1)[0] stem.version.Version(tor_version)

1 0

[doctor/master] Support tracking multiple address or fingerprint entries
by atagar＠torproject.org 24 Feb '16

24 Feb '16

commit a1fdff709edd7af3ae3009b480687bbc67de7dc1 Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 24 09:33:40 2016 -0800 Support tracking multiple address or fingerprint entries dgoulet made the good point that we sometimes want to track multiple address ranges or fingerprints for a single bad actor. Supporting multiple 'name.address' and 'name.fingerprint' per entries. --- track_relays.py | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/track_relays.py b/track_relays.py index bc46216..deee467 100755 --- a/track_relays.py +++ b/track_relays.py @@ -32,8 +32,8 @@ class TrackedRelay(object): :var str identifier: brief identifier given to the entry :var str description: description of why we're tracking it :var datetime expires: when this entry expires - :var str address: address of the relay we're tracking - :var str fingerprint: fingerprint of the relay we're tracking + :var list addresses: address of the relay we're tracking + :var list fingerprints: fingerprint of the relay we're tracking """ def __init__(self, identifier, config): @@ -50,20 +50,20 @@ class TrackedRelay(object): except ValueError: raise ValueError("'%s.expires' is malformed. We expect it to be in the form 'Year-Month-Day'" % identifier) - self.address = config.get('%s.address' % identifier, None) - self.fingerprint = config.get('%s.fingerprint' % identifier, None) + self.addresses = config.get('%s.address' % identifier, []) + self.fingerprints = config.get('%s.fingerprint' % identifier, []) - if not self.address and not self.fingerprint: + if not self.addresses and not self.fingerprints: raise ValueError("We need either a '%s.address' or '%s.fingerprint' to track" % (identifier, identifier)) def __str__(self): attr = [] - if self.address: - attr.append('address: %s' % self.address) + for address in self.addresses: + attr.append('address: %s' % address) - if self.fingerprint: - attr.append('fingerprint: %s' % self.fingerprint) + for fingerprint in self.fingerprints: + attr.append('fingerprint: %s' % fingerprint) return '%s (%s)' % (self.identifier, ', '.join(attr)) @@ -110,17 +110,17 @@ def main(): tracked_fingerprints = {} for relay in get_tracked_relays(): - if relay.address: - if '/' in relay.address: + for address in relay.addresses: + if '/' in address: # It's a total hack, but taking advantage of exit policies where we # already support address ranges. - tracked_address_ranges[stem.exit_policy.ExitPolicyRule('accept %s:*' % relay.address)] = relay + tracked_address_ranges[stem.exit_policy.ExitPolicyRule('accept %s:*' % address)] = relay else: - tracked_addresses[relay.address] = relay + tracked_addresses[address] = relay - if relay.fingerprint: - tracked_fingerprints[relay.fingerprint] = relay + for fingerprint in relay.fingerprints: + tracked_fingerprints[fingerprint] = relay downloader = stem.descriptor.remote.DescriptorDownloader() found_relays = {} # mapping of TrackedRelay => RouterStatusEntry

1 0