April 2018 - tor-commits - lists.torproject.org

[tor/master] Add a unit test for post-loop events
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 62f4d5a2659d2a786c92299fb29a6a6be44f8caf Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon Apr 2 16:47:28 2018 -0400 Add a unit test for post-loop events This test works by having two post-loop events activate one another in a tight loop. If the "post-loop" mechanism didn't work, this would be enough to starve all other events. --- src/test/test_compat_libevent.c | 60 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 60 insertions(+) diff --git a/src/test/test_compat_libevent.c b/src/test/test_compat_libevent.c index 376524c2d..85f69bd62 100644 --- a/src/test/test_compat_libevent.c +++ b/src/test/test_compat_libevent.c @@ -121,10 +121,70 @@ test_compat_libevent_header_version(void *ignored) (void)0; } +/* Test for postloop events */ + +/* Event callback to increment a counter. */ +static void +increment_int_counter_cb(periodic_timer_t *timer, void *arg) +{ + (void)timer; + int *ctr = arg; + ++*ctr; +} + +static int activated_counter = 0; + +/* Mainloop event callback to activate another mainloop event */ +static void +activate_event_cb(mainloop_event_t *ev, void *arg) +{ + (void)ev; + mainloop_event_t **other_event = arg; + mainloop_event_activate(*other_event); + ++activated_counter; +} + +static void +test_compat_libevent_postloop_events(void *arg) +{ + (void)arg; + mainloop_event_t *a = NULL, *b = NULL; + periodic_timer_t *timed = NULL; + + tor_libevent_postfork(); + + /* If postloop events don't work, then these events will activate one + * another ad infinitum and, and the periodic event will never occur. */ + b = mainloop_event_postloop_new(activate_event_cb, &a); + a = mainloop_event_postloop_new(activate_event_cb, &b); + + int counter = 0; + struct timeval fifty_ms = { 0, 10 * 1000 }; + timed = periodic_timer_new(tor_libevent_get_base(), &fifty_ms, + increment_int_counter_cb, &counter); + + mainloop_event_activate(a); + int r; + do { + r = tor_libevent_run_event_loop(tor_libevent_get_base(), 0); + if (r == -1) + break; + } while (counter < 5); + + tt_int_op(activated_counter, OP_GE, 2); + + done: + mainloop_event_free(a); + mainloop_event_free(b); + periodic_timer_free(timed); +} + struct testcase_t compat_libevent_tests[] = { { "logging_callback", test_compat_libevent_logging_callback, TT_FORK, NULL, NULL }, { "header_version", test_compat_libevent_header_version, 0, NULL, NULL }, + { "postloop_events", test_compat_libevent_postloop_events, + TT_FORK, NULL, NULL }, END_OF_TESTCASES };

1 0

[tor/master] Move connection_ap_attach_pending(0) into a postloop event
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 320bd2b3a50511643e5d1ca6cd16708f20fa7d68 Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon Apr 2 15:20:21 2018 -0400 Move connection_ap_attach_pending(0) into a postloop event This is a second motivating case for our postloop event logic. --- src/or/connection_edge.c | 32 ++++++++++++++++++++++---------- src/or/main.c | 13 ------------- 2 files changed, 22 insertions(+), 23 deletions(-) diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c index 948c8722b..955f942c5 100644 --- a/src/or/connection_edge.c +++ b/src/or/connection_edge.c @@ -611,6 +611,12 @@ static smartlist_t *pending_entry_connections = NULL; static int untried_pending_connections = 0; +/** + * Mainloop event to tell us to scan for pending connections that can + * be attached. + */ +static mainloop_event_t *attach_pending_entry_connections_ev = NULL; + /** Common code to connection_(ap|exit)_about_to_close. */ static void connection_edge_about_to_close(edge_connection_t *edge_conn) @@ -956,6 +962,14 @@ connection_ap_attach_pending(int retry) untried_pending_connections = 0; } +static void +attach_pending_entry_connections_cb(mainloop_event_t *ev, void *arg) +{ + (void)ev; + (void)arg; + connection_ap_attach_pending(0); +} + /** Mark entry_conn as needing to get attached to a circuit. * * And entry_conn must be in AP_CONN_STATE_CIRCUIT_WAIT, @@ -973,9 +987,13 @@ connection_ap_mark_as_pending_circuit_(entry_connection_t *entry_conn, if (conn->marked_for_close) return; - if (PREDICT_UNLIKELY(NULL == pending_entry_connections)) + if (PREDICT_UNLIKELY(NULL == pending_entry_connections)) { pending_entry_connections = smartlist_new(); - + } + if (PREDICT_UNLIKELY(NULL == attach_pending_entry_connections_ev)) { + attach_pending_entry_connections_ev = mainloop_event_postloop_new( + attach_pending_entry_connections_cb, NULL); + } if (PREDICT_UNLIKELY(smartlist_contains(pending_entry_connections, entry_conn))) { log_warn(LD_BUG, "What?? pending_entry_connections already contains %p! " @@ -999,14 +1017,7 @@ connection_ap_mark_as_pending_circuit_(entry_connection_t *entry_conn, untried_pending_connections = 1; smartlist_add(pending_entry_connections, entry_conn); - /* Work-around for bug 19969: we handle pending_entry_connections at - * the end of run_main_loop_once(), but in many cases that function will - * take a very long time, if ever, to finish its call to event_base_loop(). - * - * So the fix is to tell it right now that it ought to finish its loop at - * its next available opportunity. - */ - tell_event_loop_to_run_external_code(); + mainloop_event_activate(attach_pending_entry_connections_ev); } /** Mark entry_conn as no longer waiting for a circuit. */ @@ -4165,5 +4176,6 @@ connection_edge_free_all(void) untried_pending_connections = 0; smartlist_free(pending_entry_connections); pending_entry_connections = NULL; + mainloop_event_free(attach_pending_entry_connections_ev); } diff --git a/src/or/main.c b/src/or/main.c index b499c1691..257af8b18 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -2867,19 +2867,6 @@ run_main_loop_once(void) if (main_loop_should_exit) return 0; - /* And here is where we put callbacks that happen "every time the event loop - * runs." They must be very fast, or else the whole Tor process will get - * slowed down. - * - * Note that this gets called once per libevent loop, which will make it - * happen once per group of events that fire, or once per second. */ - - /* If there are any pending client connections, try attaching them to - * circuits (if we can.) This will be pretty fast if nothing new is - * pending. - */ - connection_ap_attach_pending(0); - return 1; }

1 0

[tor/master] Changes files for post-loop events (25374)
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 2fe499eb3f4c4a15249f9f913a1cb4e59e94d03f Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon Apr 2 16:48:54 2018 -0400 Changes files for post-loop events (25374) --- changes/ticket25374 | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/changes/ticket25374 b/changes/ticket25374 new file mode 100644 index 000000000..5fc59d158 --- /dev/null +++ b/changes/ticket25374 @@ -0,0 +1,6 @@ + o Code simplification and refactoring: + - Our main loop has been simplified so that all important operations + happen inside events. Previously, some operations had to happen + outside the event loop, to prevent infinite sequences of event + activations. Closes ticket 25374. +

1 0

[tor/master] Move the "activate linked connections" logic to a postloop event.
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 5719dfb48f87a54aeb5982ff03345303bc058ebb Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon Apr 2 15:08:41 2018 -0400 Move the "activate linked connections" logic to a postloop event. A linked connection_t is one that gets its I/O, not from the network, but from another connection_t. When such a connection has something to write, we want the corresponding connection to run its read callback ... but not immediately, to avoid infinite recursion and/or event loop starvation. Previously we handled this case by activating the read events outside the event loop. Now we use the "postloop event" logic. This lets us simplify do_main_loop_once() a little. --- src/or/main.c | 39 +++++++++++++++++++++++++++++---------- 1 file changed, 29 insertions(+), 10 deletions(-) diff --git a/src/or/main.c b/src/or/main.c index 4cc51de07..b499c1691 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -410,6 +410,27 @@ connection_unlink(connection_t *conn) connection_free(conn); } +/** + * Callback: used to activate read events for all linked connections, so + * libevent knows to call their read callbacks. This callback run as a + * postloop event, so that the events _it_ activates don't happen until + * Libevent has a chance to check for other events. + */ +static void +schedule_active_linked_connections_cb(mainloop_event_t *event, void *arg) +{ + (void)event; + (void)arg; + + /* All active linked conns should get their read events activated, + * so that libevent knows to run their callbacks. */ + SMARTLIST_FOREACH(active_linked_connection_lst, connection_t *, conn, + event_active(conn->read_event, EV_READ, 1)); +} + +/** Event that invokes schedule_active_linked_connections_cb. */ +static mainloop_event_t *schedule_active_linked_connections_event = NULL; + /** Initialize the global connection list, closeable connection list, * and active connection list. */ STATIC void @@ -803,10 +824,7 @@ connection_start_reading_from_linked_conn(connection_t *conn) if (!conn->active_on_link) { conn->active_on_link = 1; smartlist_add(active_linked_connection_lst, conn); - /* make sure that the event_base_loop() function exits at - * the end of its run through the current connections, so we can - * activate read events for linked connections. */ - tell_event_loop_to_run_external_code(); + mainloop_event_activate(schedule_active_linked_connections_event); } else { tor_assert(smartlist_contains(active_linked_connection_lst, conn)); } @@ -2579,6 +2597,11 @@ do_main_loop(void) initialize_periodic_events(); } + if (!schedule_active_linked_connections_event) { + schedule_active_linked_connections_event = + mainloop_event_postloop_new(schedule_active_linked_connections_cb, NULL); + } + /* initialize dns resolve map, spawn workers if needed */ if (dns_init() < 0) { if (get_options()->ServerDNSAllowBrokenConfig) @@ -2783,17 +2806,12 @@ run_main_loop_once(void) errno = 0; #endif - /* All active linked conns should get their read events activated, - * so that libevent knows to run their callbacks. */ - SMARTLIST_FOREACH(active_linked_connection_lst, connection_t *, conn, - event_active(conn->read_event, EV_READ, 1)); - if (get_options()->MainloopStats) { /* We always enforce that EVLOOP_ONCE is passed to event_base_loop() if we * are collecting main loop statistics. */ called_loop_once = 1; } else { - called_loop_once = smartlist_len(active_linked_connection_lst) ? 1 : 0; + called_loop_once = 0; } /* Make sure we know (about) what time it is. */ @@ -3496,6 +3514,7 @@ tor_free_all(int postfork) tor_event_free(shutdown_did_not_work_event); tor_event_free(initialize_periodic_events_event); mainloop_event_free(directory_all_unreachable_cb_event); + mainloop_event_free(schedule_active_linked_connections_event); #ifdef HAVE_SYSTEMD_209 periodic_timer_free(systemd_watchdog_timer);

1 0

[tor/master] Generic mechaism for "post-loop" callbacks
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit c5a3e2ca44cfd3302a65ae247120fd4efb81a379 Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon Apr 2 15:02:53 2018 -0400 Generic mechaism for "post-loop" callbacks We've been labeling some events as happening "outside the event loop", to avoid Libevent starvation. This patch provides a cleaner mechanism to avoid that starvation. For background, the problem here is that Libevent only scans for new events once it has run all its active callbacks. So if the callbacks keep activating new callbacks, they could potentially starve Libevent indefinitely and keep it from ever checking for timed, socket, or signal events. To solve this, we add the ability to label some events as "post-loop". The rule for a "post-loop" event is that any events _it_ activates can only be run after libevent has re-scanned for new events at least once. --- src/common/compat_libevent.c | 115 ++++++++++++++++++++++++++++++++++++++++--- src/common/compat_libevent.h | 3 ++ 2 files changed, 110 insertions(+), 8 deletions(-) diff --git a/src/common/compat_libevent.c b/src/common/compat_libevent.c index 0d0e4337b..9936c0aac 100644 --- a/src/common/compat_libevent.c +++ b/src/common/compat_libevent.c @@ -79,6 +79,43 @@ tor_event_free_(struct event *ev) /** Global event base for use by the main thread. */ static struct event_base *the_event_base = NULL; +/** + * @defgroup postloop post-loop event helpers + * + * If we're not careful, Libevent can susceptible to infinite event chains: + * one event can activate another, whose callback activates another, whose + * callback activates another, ad infinitum. While this is happening, + * Libevent won't be checking timeouts, socket-based events, signals, and so + * on. + * + * We solve this problem by marking some events as "post-loop". A post-loop + * event behaves like any ordinary event, but any events that _it_ activates + * cannot run until Libevent has checked for other events at least once. + * + * @{ */ + +/** + * An event that stops Libevent from running any more events on the current + * iteration of its loop, until it has re-checked for socket events, signal + * events, timeouts, etc. + */ +static struct event *rescan_mainloop_ev = NULL; + +/** + * Callback to implement rescan_mainloop_ev: it simply exits the mainloop, + * and relies on Tor to re-enter the mainloop since no error has occurred. + */ +static void +rescan_mainloop_cb(evutil_socket_t fd, short events, void *arg) +{ + (void)fd; + (void)events; + struct event_base *the_base = arg; + event_base_loopbreak(the_base); +} + +/** @} */ + /* This is what passes for version detection on OSX. We set * MACOSX_KQUEUE_IS_BROKEN to true iff we're on a version of OSX before * 10.4.0 (aka 1040). */ @@ -130,6 +167,15 @@ tor_libevent_initialize(tor_libevent_cfg *torcfg) /* LCOV_EXCL_STOP */ } + rescan_mainloop_ev = event_new(the_event_base, -1, 0, + rescan_mainloop_cb, the_event_base); + if (!rescan_mainloop_ev) { + /* LCOV_EXCL_START */ + log_err(LD_GENERAL, "Unable to create rescan event: cannot continue."); + exit(1); // exit ok: libevent is broken. + /* LCOV_EXCL_STOP */ + } + log_info(LD_GENERAL, "Initialized libevent version %s using method %s. Good.", event_get_version(), tor_libevent_get_method()); @@ -251,6 +297,52 @@ mainloop_event_cb(evutil_socket_t fd, short what, void *arg) } /** + * As mainloop_event_cb, but implements a post-loop event. + */ +static void +mainloop_event_postloop_cb(evutil_socket_t fd, short what, void *arg) +{ + (void)fd; + (void)what; + + /* Note that if rescan_mainloop_ev is already activated, + * event_active() will do nothing: only the first post-loop event that + * happens each time through the event loop will cause it to be + * activated. + * + * Because event_active() puts events on a FIFO queue, every event + * that is made active _after_ rescan_mainloop_ev will get its + * callback run after rescan_mainloop_cb is called -- that is, on the + * next iteration of the loop. + */ + event_active(rescan_mainloop_ev, EV_READ, 1); + + mainloop_event_t *mev = arg; + mev->cb(mev, mev->userdata); +} + +/** + * Helper for mainloop_event_new() and mainloop_event_postloop_new(). + */ +static mainloop_event_t * +mainloop_event_new_impl(int postloop, + void (*cb)(mainloop_event_t *, void *), + void *userdata) +{ + tor_assert(cb); + + struct event_base *base = tor_libevent_get_base(); + mainloop_event_t *mev = tor_malloc_zero(sizeof(mainloop_event_t)); + mev->ev = tor_event_new(base, -1, 0, + postloop ? mainloop_event_postloop_cb : mainloop_event_cb, + mev); + tor_assert(mev->ev); + mev->cb = cb; + mev->userdata = userdata; + return mev; +} + +/** * Create and return a new mainloop_event_t to run the function cb. * * When run, the callback function will be passed the mainloop_event_t @@ -265,15 +357,21 @@ mainloop_event_t * mainloop_event_new(void (*cb)(mainloop_event_t *, void *), void *userdata) { - tor_assert(cb); + return mainloop_event_new_impl(0, cb, userdata); +} - struct event_base *base = tor_libevent_get_base(); - mainloop_event_t *mev = tor_malloc_zero(sizeof(mainloop_event_t)); - mev->ev = tor_event_new(base, -1, 0, mainloop_event_cb, mev); - tor_assert(mev->ev); - mev->cb = cb; - mev->userdata = userdata; - return mev; +/** + * As mainloop_event_new(), but create a post-loop event. + * + * A post-loop event behaves like any ordinary event, but any events + * that _it_ activates cannot run until Libevent has checked for other + * events at least once. + */ +mainloop_event_t * +mainloop_event_postloop_new(void (*cb)(mainloop_event_t *, void *), + void *userdata) +{ + return mainloop_event_new_impl(1, cb, userdata); } /** @@ -358,6 +456,7 @@ tor_init_libevent_rng(void) void tor_libevent_free_all(void) { + tor_event_free(rescan_mainloop_ev); if (the_event_base) event_base_free(the_event_base); the_event_base = NULL; diff --git a/src/common/compat_libevent.h b/src/common/compat_libevent.h index c8b037156..29c6ad375 100644 --- a/src/common/compat_libevent.h +++ b/src/common/compat_libevent.h @@ -37,6 +37,9 @@ void periodic_timer_free_(periodic_timer_t *); typedef struct mainloop_event_t mainloop_event_t; mainloop_event_t *mainloop_event_new(void (*cb)(mainloop_event_t *, void *), void *userdata); +mainloop_event_t * mainloop_event_postloop_new( + void (*cb)(mainloop_event_t *, void *), + void *userdata); void mainloop_event_activate(mainloop_event_t *event); int mainloop_event_schedule(mainloop_event_t *event, const struct timeval *delay);

1 0

[translation/support-miscellaneous_completed] Update translations for support-miscellaneous_completed
by translation＠torproject.org 13 Apr '18

13 Apr '18

commit d8ed6c3ecaf50e8d2dde4ca17d2e975789ea2dc4 Author: Translation commit bot <translation(a)torproject.org> Date: Fri Apr 13 15:50:36 2018 +0000 Update translations for support-miscellaneous_completed --- es.json | 114 ++++++++++++++++++++++++++++++++-------------------------------- 1 file changed, 57 insertions(+), 57 deletions(-) diff --git a/es.json b/es.json index 75836d7aa..8f48ce9f0 100644 --- a/es.json +++ b/es.json @@ -1,92 +1,92 @@ { - "miscellaneus-1": { - "id": "#miscellaneus-1", - "control": "miscellaneus-1", + "misc-1": { + "id": "#misc-1", + "control": "misc-1", "title": "Tengo una razón imperiosa para trazar a un usuario de Tor. ¿Me puede ayudar?", "description": "No hay nada que los desarrolladores de Tor puedan hacer para trazar a los usuarios de Tor. Las mismas protecciones que evitan que indeseables rompan el anonimato de Tor nos impiden a nosotros rastrear usuarios." }, - "miscellaneus-2": { - "id": "#miscellaneus-2", - "control": "miscellaneus-2", - "title": "¿Por qué no evitan que gente indeseable haga maldades cuando usa Tor?", - "description": "Tor está diseñado para defender los derechos humanos y la privacidad evitando que cualquiera censure cosas, incluso nosotros. Aborrecemos que haya algunas personas que usen Tor para hacer cosas terribles, pero no podemos hacer nada para librarnos de ellos sin socavar también los activistas pro derechos humanos, periodistas, supervivientes de abuso, y otras personas que usan Tor para cosas buenas. Si queremos evitar que ciertas personas usen Tor, básicamente estaríamos añadiendo una puerta trasera al software, lo que dejaría vulnerables a nuestros usuarios a ataques de regímenes opresivos y otros adversarios." + "misc-2": { + "id": "#misc-2", + "control": "misc-2", + "title": "¿Por qué no evitáis que indeseables haga maldades cuando usan Tor?", + "description": "Tor está diseñado para defender los derechos humanos y la privacidad evitando que cualquiera censure cosas, incluso nosotros. Odiamos que haya personas que usen Tor para hacer cosas terribles, pero no podemos hacer nada para librarnos de ellos sin socavar también los activistas pro derechos humanos, periodistas, supervivientes de abusos, y otras personas que usan Tor para cosas buenas. Si queremos evitar que ciertas personas usen Tor, básicamente estaríamos añadiendo una puerta trasera al software, lo que dejaría vulnerables a nuestros usuarios a ataques de regímenes opresivos y otros adversarios." }, - "miscellaneus-3": { - "id": "#miscellaneus-3", - "control": "miscellaneus-3", + "misc-3": { + "id": "#misc-3", + "control": "misc-3", "title": "¿Quién financia Tor?", - "description": "Tor está financiado por varios patrocinadores distintos, incluyendo agencias federales de EEUU, fundaciones privadas, y donantes individuales. Revise la lista completa de <a href=\"https://www.torproject.org/about/sponsors.html.en\">nuestros patrocinadores</a> y una serie de <a href=\"https://blog.torproject.org/category/tags/form-990\">artículos del blog</a> sobre nuestros informes financieros.Sentimos que hablar abiertamente sobre quienes nos financian y nuestro modelo de financiación es la mejor forma de mantener la confianza con nuestra comunidad. Siempre estamos buscando más diversidad para nuestras fuentes de financiación, especialmente desde fundaciones y personas a título individual." + "description": "Tor está financiado por varios patrocinadores distintos, incluyendo agencias federales de EEUU, fundaciones privadas, y donantes individuales. Revise la lista completa de <a href=\"https://www.torproject.org/about/sponsors.html.en\">nuestros patrocinadores</a> y una serie de <a href=\"https://blog.torproject.org/category/tags/form-990\">artículos del blog</a> sobre nuestros informes financieros.Creemos que hablar claramente sobre quienes nos financian y nuestro modelo de financiación es la mejor forma de mantener la confianza con nuestra comunidad. Siempre estamos buscando más diversidad para nuestras fuentes de financiación, especialmente desde fundaciones y personas a título individual." }, - "miscellaneus-4": { - "id": "#miscellaneus-4", - "control": "miscellaneus-4", + "misc-4": { + "id": "#misc-4", + "control": "misc-4", "title": "¿Puedo usar Tor con bittorrent?", "description": "No recomendamos usar Tor con bittorrent. Para más detalles lee nuestro <a href=\"https://blog.torproject.org/bittorrent-over-tor-isnt-good-idea\">artículo del blog sobre este tema</a>." }, - "miscellaneus-5": { - "id": "#miscellaneus-5", - "control": "miscellaneus-5", + "misc-5": { + "id": "#misc-5", + "control": "misc-5", "title": "¡Los archivos de mi equipo han sido bloqueados, y alguien me está exigiendo que descargue el Navegador Tor para pagar un rescate por mis archivos!", - "description": "Lo sentimos mucho, pero ha sido infectado por malware. The Tor Project no ha creado este malware. Los autores de este malware le están urgiendo a que descargue el Navegador Tor presumiblemente para que contacte con ellos de forma anónima con el rescate que le están exigiendo. Si este es su primer contacto con el Navegador Tor, entendemos que pueda pensar que somos unos indeseables que facultamos a gente todavía más indeseable. Pero, por favor, considere que nuestro software es usado cada día por una amplia variedad de propósitos por activistas pro derechos humanos, periodistas, supervivientes de violencia doméstica, informantes, agentes de la ley, y muchos otros. Desafortunadamente, la protección que puede proporcionar nuestro software a estos grupos de personas también puede ser abusada por criminales y autores de malware. The Tor Project no apoya ni excusa el uso de nuestro software para propósitos maliciosos." + "description": "Lo sentimos mucho, pero has sido infectado por malware. El Proyecto Tor no ha creado este malware. Los autores de este malware te están urgiendo a que descargues el Navegador Tor presumiblemente para que contactes con ellos de forma anónima con el rescate que te están exigiendo. Si este es tu primer contacto con el Navegador Tor, entendemos que puedas pensar que somos unos indeseables que facultamos a gente todavía más indeseable. Pero, por favor, considera que nuestro software es usado cada día por una amplia variedad de propósitos por activistas pro derechos humanos, periodistas, supervivientes de violencia doméstica, informantes, agentes de la ley, y muchos otros. Desafortunadamente, la protección que puede proporcionar nuestro software a estos grupos de personas también puede ser usada por criminales y autores de malware. El proyecto Tor no apoya ni excusa el uso de nuestro software para propósitos maliciosos." }, - "miscellaneus-6": { - "id": "#miscellaneus-6", - "control": "miscellaneus-6", - "title": "¿Tor conserva registros?", - "description": "Tor no conserva ningún registro que pueda identificar a un usuario concreto. Tomamos algunas mediciones seguras acerca de cómo funciona la red, que puede revisar en <a href=\"https://metrics.torproject.org/\">Tor Metrics</a>." + "misc-6": { + "id": "#misc-6", + "control": "misc-6", + "title": "¿Conserva Tor registros?", + "description": "Tor no conserva ningún registro que pueda identificar a un usuario concreto. Tomamos algunas mediciones seguras de cómo funciona la red, que puedes revisar en <a href=\"https://metrics.torproject.org/\">Tor Metrics</a>." }, - "miscellaneus-7": { - "id": "#miscellaneus-7", - "control": "miscellaneus-7", - "title": "¿Tor Project ofrece servicio de correo electrónico u otros servicios web protectores de la privacidad?", + "misc-7": { + "id": "#misc-7", + "control": "misc-7", + "title": "¿Ofrece el Proyecto Tor servicio de correo electrónico u otros servicios web protectores de la privacidad?", "description": "No, no proporcionamos ningún servicio en línea. Puedes leer una lista de todos nuestros proyectos de software en la <a href=\"https://www.torproject.org/projects/projects.html.en\">página de nuestro proyecto</a>." }, - "miscellaneus-8": { - "id": "#miscellaneus-8", - "control": "miscellaneus-8", + "misc-8": { + "id": "#misc-8", + "control": "misc-8", "title": "¿Puedo usar el logotipo de Tor en mi producto?", "description": "Puedes leer todo lo relacionado con eso en nuestra <a href=\"https://www.torproject.org/docs/trademark-faq.html\">página de preguntas frecuentes sobre marca registrada</a>." }, - "miscellaneus-9": { - "id": "#miscellaneus-9", - "control": "miscellaneus-9", + "misc-9": { + "id": "#misc-9", + "control": "misc-9", "title": "Estoy teniendo un problema actualizando o usando Vidalia.", "description": "Vidalia ya no es mantenido ni está soportado. Una gran porción de las características que ofrecía Vidalia ahora están integradas en el propio Navegador Tor." }, - "miscellaneus-10": { - "id": "#miscellaneus-10", - "control": "miscellaneus-10", - "title": "¿Tor Project ofrece alojamiento?", - "description": "No, Tor Project no ofrece servicios de alojamiento." + "misc-10": { + "id": "#misc-10", + "control": "misc-10", + "title": "¿Ofrece alojamiento el Proyecto Tor?", + "description": "No, el Proyecto Tor no ofrece servicios de alojamiento." }, - "miscellaneus-11": { - "id": "#miscellaneus-11", - "control": "miscellaneus-11", + "misc-11": { + "id": "#misc-11", + "control": "misc-11", "title": "¿Puedo cambiar el número de saltos que usa Tor?", "description": "Ahora mismo la longitud de la ruta está internamente codificada en 3 más el número de nodos en tu ruta que sean sensibles. Esto en casos normales es 3, pero por ejemplo, si estás accediendo a un servicio onion o a una dirección \".exit\" podrían ser más.<p class=\"mb-3\"No queremos animar a que la gente use rutas más largas que esta, ya que incrementaría la carga de la red sin (hasta donde podemos decir) proporcionar más seguridad. Además, usar rutas más largas que 3 podría dañar el anonimato, en primer lugar, porque facilita los ataques de <a href=\"https://www.freehaven.net/anonbib/#ccs07-doa\">denegación de seguridad</a>, y en segundo, porque podría actuar como un identificador si sólo un pequeño número de usuarios tiene la misma longitud de ruta que tú." }, - "miscellaneus-12": { - "id": "#miscellaneus-12", - "control": "miscellaneus-12", + "misc-12": { + "id": "#misc-12", + "control": "misc-12", "title": "¿Cómo puedo compartir ficheros anónimamente a través de Tor?", "description": "Muchos nodos de salida están configurados para bloquear ciertos tipos de tráfico de compartición de ficheros, como bittorrent. Específicamente bittorrent <a href=\"https://blog.torproject.org/bittorrent-over-tor-isnt-good-idea\">no es anónimo sobre Tor</a>.Para compartición de ficheros a través de Tor, <a href=\"https://onionshare.org/\">OnionShare</a> es una buena opción." }, - "miscellaneus-13": { - "id": "#miscellaneus-13", - "control": "miscellaneus-13", + "misc-13": { + "id": "#misc-13", + "control": "misc-13", "title": "¿Puedo usar una VPN con Tor?", "description": "En términos generales, no recomendamos usar una VPN con Tor a menos que seas un usuario avanzado que sabe cómo configurar ambos de forma que no comprometa su privacidad. Puedes encontrar más información detallada sobre Tor + VPN en <a href=\"https://trac.torproject.org/projects/tor/wiki/doc/TorPlusVPN\">nuestro wiki</a>." }, - "miscellaneus-14": { - "id": "#miscellaneus-14", - "control": "miscellaneus-14", + "misc-14": { + "id": "#misc-14", + "control": "misc-14", "title": "¿Cómo puedo ofrecerme voluntario para Tor Project?", - "description": "Por favor, ¡vea nuestra <a href=\"https://www.torproject.org/getinvolved/volunteer.html.en\">página de voluntariado</a> para ver cómo involucrarse!" + "description": "Por favor, ¡mira nuestra <a href=\"https://www.torproject.org/getinvolved/volunteer.html.en\">página de voluntariado</a> para ver cómo involucrarte!" }, - "miscellaneus-15": { - "id": "#miscellaneus-15", - "control": "miscellaneus-15", - "title": "¿Cómo puedo donar a Tor Project?", - "description": "¡Gracias por su apoyo! Puede encontrar más información sobre donación en nuestras <a href=\"https://donate.torproject.org/donor-faq\">preguntas frecuentes del donante</a>." + "misc-15": { + "id": "#misc-15", + "control": "misc-15", + "title": "¿Cómo puedo donar al Proyecto Tor?", + "description": "¡Gracias por tu apoyo! Puedes encontrar más información sobre donación en nuestras <a href=\"https://donate.torproject.org/donor-faq\">en las FAQ del donante</a>." } }

1 0

[translation/support-miscellaneous] Update translations for support-miscellaneous
by translation＠torproject.org 13 Apr '18

13 Apr '18

commit adf7d5ebe2411beaf71589c05afb32b6160a0c02 Author: Translation commit bot <translation(a)torproject.org> Date: Fri Apr 13 15:50:32 2018 +0000 Update translations for support-miscellaneous --- es.json | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/es.json b/es.json index b6b6bf353..8f48ce9f0 100644 --- a/es.json +++ b/es.json @@ -27,18 +27,18 @@ "id": "#misc-5", "control": "misc-5", "title": "¡Los archivos de mi equipo han sido bloqueados, y alguien me está exigiendo que descargue el Navegador Tor para pagar un rescate por mis archivos!", - "description": "Lo sentimos mucho, pero ha sido infectado por malware. The Tor Project no ha creado este malware. Los autores de este malware le están urgiendo a que descargue el Navegador Tor presumiblemente para que contacte con ellos de forma anónima con el rescate que le están exigiendo. Si este es su primer contacto con el Navegador Tor, entendemos que pueda pensar que somos unos indeseables que facultamos a gente todavía más indeseable. Pero, por favor, considere que nuestro software es usado cada día por una amplia variedad de propósitos por activistas pro derechos humanos, periodistas, supervivientes de violencia doméstica, informantes, agentes de la ley, y muchos otros. Desafortunadamente, la protección que puede proporcionar nuestro software a estos grupos de personas también puede ser abusada por criminales y autores de malware. The Tor Project no apoya ni excusa el uso de nuestro software para propósitos maliciosos." + "description": "Lo sentimos mucho, pero has sido infectado por malware. El Proyecto Tor no ha creado este malware. Los autores de este malware te están urgiendo a que descargues el Navegador Tor presumiblemente para que contactes con ellos de forma anónima con el rescate que te están exigiendo. Si este es tu primer contacto con el Navegador Tor, entendemos que puedas pensar que somos unos indeseables que facultamos a gente todavía más indeseable. Pero, por favor, considera que nuestro software es usado cada día por una amplia variedad de propósitos por activistas pro derechos humanos, periodistas, supervivientes de violencia doméstica, informantes, agentes de la ley, y muchos otros. Desafortunadamente, la protección que puede proporcionar nuestro software a estos grupos de personas también puede ser usada por criminales y autores de malware. El proyecto Tor no apoya ni excusa el uso de nuestro software para propósitos maliciosos." }, "misc-6": { "id": "#misc-6", "control": "misc-6", - "title": "¿Tor conserva registros?", - "description": "Tor no conserva ningún registro que pueda identificar a un usuario concreto. Tomamos algunas mediciones seguras acerca de cómo funciona la red, que puede revisar en <a href=\"https://metrics.torproject.org/\">Tor Metrics</a>." + "title": "¿Conserva Tor registros?", + "description": "Tor no conserva ningún registro que pueda identificar a un usuario concreto. Tomamos algunas mediciones seguras de cómo funciona la red, que puedes revisar en <a href=\"https://metrics.torproject.org/\">Tor Metrics</a>." }, "misc-7": { "id": "#misc-7", "control": "misc-7", - "title": "¿Tor Project ofrece servicio de correo electrónico u otros servicios web protectores de la privacidad?", + "title": "¿Ofrece el Proyecto Tor servicio de correo electrónico u otros servicios web protectores de la privacidad?", "description": "No, no proporcionamos ningún servicio en línea. Puedes leer una lista de todos nuestros proyectos de software en la <a href=\"https://www.torproject.org/projects/projects.html.en\">página de nuestro proyecto</a>." }, "misc-8": { @@ -56,8 +56,8 @@ "misc-10": { "id": "#misc-10", "control": "misc-10", - "title": "¿Tor Project ofrece alojamiento?", - "description": "No, Tor Project no ofrece servicios de alojamiento." + "title": "¿Ofrece alojamiento el Proyecto Tor?", + "description": "No, el Proyecto Tor no ofrece servicios de alojamiento." }, "misc-11": { "id": "#misc-11", @@ -81,12 +81,12 @@ "id": "#misc-14", "control": "misc-14", "title": "¿Cómo puedo ofrecerme voluntario para Tor Project?", - "description": "Por favor, ¡vea nuestra <a href=\"https://www.torproject.org/getinvolved/volunteer.html.en\">página de voluntariado</a> para ver cómo involucrarse!" + "description": "Por favor, ¡mira nuestra <a href=\"https://www.torproject.org/getinvolved/volunteer.html.en\">página de voluntariado</a> para ver cómo involucrarte!" }, "misc-15": { "id": "#misc-15", "control": "misc-15", - "title": "¿Cómo puedo donar a Tor Project?", - "description": "¡Gracias por su apoyo! Puede encontrar más información sobre donación en nuestras <a href=\"https://donate.torproject.org/donor-faq\">preguntas frecuentes del donante</a>." + "title": "¿Cómo puedo donar al Proyecto Tor?", + "description": "¡Gracias por tu apoyo! Puedes encontrar más información sobre donación en nuestras <a href=\"https://donate.torproject.org/donor-faq\">en las FAQ del donante</a>." } }

1 0

[translation/support-miscellaneous] Update translations for support-miscellaneous
by translation＠torproject.org 13 Apr '18

13 Apr '18

commit 0ee420a7a0d4bc0fa1be6c541cffccf152a6539a Author: Translation commit bot <translation(a)torproject.org> Date: Fri Apr 13 15:20:35 2018 +0000 Update translations for support-miscellaneous --- es.json | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/es.json b/es.json index a1c9fa15a..b6b6bf353 100644 --- a/es.json +++ b/es.json @@ -8,14 +8,14 @@ "misc-2": { "id": "#misc-2", "control": "misc-2", - "title": "¿Por qué no evitan que gente indeseable haga maldades cuando usa Tor?", - "description": "Tor está diseñado para defender los derechos humanos y la privacidad evitando que cualquiera censure cosas, incluso nosotros. Aborrecemos que haya algunas personas que usen Tor para hacer cosas terribles, pero no podemos hacer nada para librarnos de ellos sin socavar también los activistas pro derechos humanos, periodistas, supervivientes de abuso, y otras personas que usan Tor para cosas buenas. Si queremos evitar que ciertas personas usen Tor, básicamente estaríamos añadiendo una puerta trasera al software, lo que dejaría vulnerables a nuestros usuarios a ataques de regímenes opresivos y otros adversarios." + "title": "¿Por qué no evitáis que indeseables haga maldades cuando usan Tor?", + "description": "Tor está diseñado para defender los derechos humanos y la privacidad evitando que cualquiera censure cosas, incluso nosotros. Odiamos que haya personas que usen Tor para hacer cosas terribles, pero no podemos hacer nada para librarnos de ellos sin socavar también los activistas pro derechos humanos, periodistas, supervivientes de abusos, y otras personas que usan Tor para cosas buenas. Si queremos evitar que ciertas personas usen Tor, básicamente estaríamos añadiendo una puerta trasera al software, lo que dejaría vulnerables a nuestros usuarios a ataques de regímenes opresivos y otros adversarios." }, "misc-3": { "id": "#misc-3", "control": "misc-3", "title": "¿Quién financia Tor?", - "description": "Tor está financiado por varios patrocinadores distintos, incluyendo agencias federales de EEUU, fundaciones privadas, y donantes individuales. Revise la lista completa de <a href=\"https://www.torproject.org/about/sponsors.html.en\">nuestros patrocinadores</a> y una serie de <a href=\"https://blog.torproject.org/category/tags/form-990\">artículos del blog</a> sobre nuestros informes financieros.Sentimos que hablar abiertamente sobre quienes nos financian y nuestro modelo de financiación es la mejor forma de mantener la confianza con nuestra comunidad. Siempre estamos buscando más diversidad para nuestras fuentes de financiación, especialmente desde fundaciones y personas a título individual." + "description": "Tor está financiado por varios patrocinadores distintos, incluyendo agencias federales de EEUU, fundaciones privadas, y donantes individuales. Revise la lista completa de <a href=\"https://www.torproject.org/about/sponsors.html.en\">nuestros patrocinadores</a> y una serie de <a href=\"https://blog.torproject.org/category/tags/form-990\">artículos del blog</a> sobre nuestros informes financieros.Creemos que hablar claramente sobre quienes nos financian y nuestro modelo de financiación es la mejor forma de mantener la confianza con nuestra comunidad. Siempre estamos buscando más diversidad para nuestras fuentes de financiación, especialmente desde fundaciones y personas a título individual." }, "misc-4": { "id": "#misc-4",

1 0

[tor-browser-build/master] Bug 25481: Enable rust support in Linux nightlies
by boklm＠torproject.org 13 Apr '18

13 Apr '18

commit 5be41831b52795ee87c57cdbd18da68bf73fe1cc Author: Georg Koppen <gk(a)torproject.org> Date: Fri Apr 13 14:29:20 2018 +0000 Bug 25481: Enable rust support in Linux nightlies We enable Rust support for tor in our Linux nightly builds as a first step to enabling it on all supported platforms. --- keyring/rust.gpg | Bin 0 -> 5311 bytes projects/cmake/config | 2 +- projects/rust/binaryen.patch | 22 +++++++++++++++ projects/rust/build | 38 ++++++++++++++++++++++++++ projects/rust/config | 62 +++++++++++++++++++++++++++++++++++++++++++ projects/tor/build | 8 ++++-- projects/tor/config | 4 +++ 7 files changed, 133 insertions(+), 3 deletions(-) diff --git a/keyring/rust.gpg b/keyring/rust.gpg new file mode 100644 index 0000000..e81f452 Binary files /dev/null and b/keyring/rust.gpg differ diff --git a/projects/cmake/config b/projects/cmake/config index 357370d..492b99c 100644 --- a/projects/cmake/config +++ b/projects/cmake/config @@ -1,5 +1,5 @@ # vim: filetype=yaml sw=2 -version: 2.8.12.2 +version: 3.4.3 git_url: https://cmake.org/cmake.git git_hash: 'v[% c("version") %]' tag_gpg_id: 1 diff --git a/projects/rust/binaryen.patch b/projects/rust/binaryen.patch new file mode 100644 index 0000000..fee9b6a --- /dev/null +++ b/projects/rust/binaryen.patch @@ -0,0 +1,22 @@ +From 820f26810baf35c1961763f6b0013e4f7e7380da Mon Sep 17 00:00:00 2001 +From: Georg Koppen <gk(a)torproject.org> +Date: Thu, 12 Apr 2018 13:10:13 +0000 +Subject: [PATCH] Use our gcc/c++ + + +diff --git a/src/librustc_binaryen/build.rs b/src/librustc_binaryen/build.rs +index f23ff3cee5..2496dd1be9 100644 +--- a/src/librustc_binaryen/build.rs ++++ b/src/librustc_binaryen/build.rs +@@ -26,6 +26,8 @@ fn main() { + + Config::new("../binaryen") + .define("BUILD_STATIC_LIB", "ON") ++ .define("CMAKE_C_COMPILER", "gcc") ++ .define("CMAKE_CXX_COMPILER", "c++") + .build_target("binaryen") + .build(); + +-- +2.16.3 + diff --git a/projects/rust/build b/projects/rust/build new file mode 100644 index 0000000..e20df55 --- /dev/null +++ b/projects/rust/build @@ -0,0 +1,38 @@ +#!/bin/bash +[% c('var/setarch') %] +[% c("var/set_default_env") -%] +distdir=/var/tmp/dist/[% project %] +mkdir -p /var/tmp/dist +tar -C /var/tmp/dist -xf [% c('input_files_by_name/cmake') %] +export PATH="/var/tmp/dist/cmake/bin:$PATH" +tar -C /var/tmp/dist -xf [% c('input_files_by_name/prev_rust') %] +cd /var/tmp/dist/rust-[% c('var/prev_version') %]-[% c('arch') %]-unknown-linux-gnu +./install.sh --prefix=$distdir-rust-old +export PATH="$distdir-rust-old/bin:$PATH" + +[% IF c("var/linux") %] + [% pc('gcc', 'var/setup', { compiler_tarfile => c('input_files_by_name/gcc') }) %] +[% END -%] + +cd $rootdir +mkdir /var/tmp/build +tar -C /var/tmp/build -xf [% c('input_files_by_name/rust') %] +cd /var/tmp/build/rustc-[% c('version') %]-src + +[% IF c("var/linux") %] + # binaryen hardcodes /usr/bin/cc and /usr/bin/c++ as the C and C++ compiler. + # But that is too old on Debian Wheezy which is why we need to patch it, so + # we can use our own GCC instead. + patch -p1 < $rootdir/binaryen.patch +[% END %] + +mkdir build +cd build +../configure --prefix=$distdir [% c("var/configure_opt") %] +make -j[% c("buildconf/num_procs") %] +make install +cd /var/tmp/dist +[% c('tar', { + tar_src => [ project ], + tar_args => '-czf ' _ dest_dir _ '/' _ c('filename'), + }) %] diff --git a/projects/rust/config b/projects/rust/config new file mode 100644 index 0000000..49fc3ab --- /dev/null +++ b/projects/rust/config @@ -0,0 +1,62 @@ +# vim: filetype=yaml sw=2 +filename: '[% project %]-[% c("version") %]-[% c("var/osname") %]-[% c("var/build_id") %].tar.gz' +version: 1.25.0 +var: + prev_version: 1.24.1 + container: + use_container: 1 + +targets: + linux: + var: + arch_deps: + - hardening-wrapper + - libssl-dev + - pkg-config + # We use + # `--enable-local-rust` to avoid downloading the required compiler during + # build time + # + # `--enable-vendor` to avoid downloading crates during build time and just + # use the ones which are shipped with the source + # + # `--enable-extended` to build not only rustc but cargo as well + # + # `--release-channel=stable` to just include stable features in the + # compiler + # + # `--sysconfdir=etc` to avoid install failures as |make install| wants to + # write to /etc otherwise + # + # the `target` triple to explicitly specify the architecture and platform + # for the compiler/std lib. Ideally, it should not be needed unless one is + # cross-compiling, but compiling `alloc_jemalloc` fails without that in a + # 32bit container. "--host=x86_64-unknown-linux-gnu" is used in its + # configure script in this case. + # `--set=` to explicitly specify the C compiler. We need to compile the + # bundled LLVM and it wants to use `cc`. However, we don't have that in + # our compiled GCC resulting in weird errors due to C and C++ compiler + # version mismatch. We avoid that with this configure option. We need to + # build our own GCC in the first place as 4.7.2 is too old to get all the + # Rust pieces compiled. + configure_opt: --enable-local-rust --enable-vendor --enable-extended --release-channel=stable --sysconfdir=etc --target=[% c("arch") %]-unknown-linux-gnu --set=target.[% c("arch") %]-unknown-linux-gnu.cc=gcc + +input_files: + - project: container-image + - project: cmake + name: cmake + - project: gcc + name: gcc + enable: '[% c("var/linux") %]' + - URL: 'https://static.rust-lang.org/dist/rustc-[% c("version") %]-src.tar.gz' + name: rust + sig_ext: asc + file_gpg_id: 1 + gpg_keyring: rust.gpg + - URL: 'https://static.rust-lang.org/dist/rust-[% c("var/prev_version") %]-[% c("arch") %]-unknown-linux-gnu.tar.xz' + name: prev_rust + sig_ext: asc + file_gpg_id: 1 + gpg_keyring: rust.gpg + - filename: binaryen.patch + enable: '[% c("var/linux") %]' diff --git a/projects/tor/build b/projects/tor/build index e52e43b..bf7db83 100644 --- a/projects/tor/build +++ b/projects/tor/build @@ -20,6 +20,10 @@ mkdir -p /var/tmp/build tar -C /var/tmp/dist -xf [% c('input_files_by_name/openssl') %] tar -C /var/tmp/dist -xf [% c('input_files_by_name/libevent') %] +[% IF c("var/linux") && c("var/nightly") %] + tar -C /var/tmp/dist -xf [% c('input_files_by_name/rust') %] + export PATH=/var/tmp/dist/rust/bin:$PATH +[% END %] tar -C /var/tmp/build -xf [% project %]-[% c('version') %].tar.gz libeventdir=/var/tmp/dist/libevent openssldir=/var/tmp/dist/openssl @@ -31,7 +35,6 @@ openssldir=/var/tmp/dist/openssl $gcclibs/*.dll $distdir/Tor/ [% END %] - [% IF c("var/linux") %] mkdir -p "$distdir/Debug/Tor" cp $openssldir/lib/libssl.so.1.0.0 "$distdir/Tor/" @@ -62,10 +65,11 @@ cd /var/tmp/build/[% project %]-[% c('version') %] echo '"[% c("abbrev", { abbrev_length => 16 }) %]"' > micro-revision.i ./autogen.sh find -type f -print0 | xargs -0 [% c("var/touch") %] -./configure --disable-asciidoc --with-libevent-dir="$libeventdir" --with-openssl-dir="$openssldir" \ +[% IF c("var/linux") && c("var/nightly") %]TOR_RUST_DEPENDENCIES=`pwd`/src/ext/rust/crates[% END %] ./configure --disable-asciidoc --with-libevent-dir="$libeventdir" --with-openssl-dir="$openssldir" \ [% IF c("var/asan") %]--enable-fragile-hardening[% END %] \ [% IF c("var/windows") %]--with-zlib-dir="$zlibdir"[% END %] \ [% IF c("var/osx") %]--enable-static-openssl[% END %] \ + [% IF c("var/linux") && c("var/nightly") %]--enable-rust[% END %] \ --prefix="$distdir" [% c("var/configure_opt") %] [% IF c("var/osx") || c("var/windows") -%] export LD_PRELOAD=[% c("var/faketime_path") %] diff --git a/projects/tor/config b/projects/tor/config index e650b6e..7029880 100644 --- a/projects/tor/config +++ b/projects/tor/config @@ -3,6 +3,7 @@ filename: '[% project %]-[% c("version") %]-[% c("var/osname") %]-[% c("var/buil version: 0.3.3.3-alpha git_hash: 'tor-[% c("version") %]' git_url: https://git.torproject.org/tor.git +git_submodule: 1 gpg_keyring: tor.gpg tag_gpg_id: 1 @@ -57,3 +58,6 @@ input_files: enable: '[% c("var/windows") %]' - name: '[% c("var/compiler") %]' project: '[% c("var/compiler") %]' + - name: rust + project: rust + enable: '[% c("var/linux") && c("var/nightly") %]'

1 0

[tor/master] Accept small hops backward in the monotonic timer.
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 3f514fe3b1d217c80edb1524976203bc535f9502 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Apr 12 13:11:35 2018 -0400 Accept small hops backward in the monotonic timer. --- src/common/token_bucket.c | 8 ++++++++ src/test/test_bwmgt.c | 7 ++++++- 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/src/common/token_bucket.c b/src/common/token_bucket.c index abb050183..6af298214 100644 --- a/src/common/token_bucket.c +++ b/src/common/token_bucket.c @@ -120,6 +120,14 @@ token_bucket_refill(token_bucket_t *bucket, uint32_t now_ts) { const uint32_t elapsed_ticks = (now_ts - bucket->last_refilled_at_ts); + if (elapsed_ticks > UINT32_MAX-(300*1000)) { + /* Either about 48 days have passed since the last refill, or the + * monotonic clock has somehow moved backwards. (We're looking at you, + * Windows.). We accept up to a 5 minute jump backwards as + * "unremarkable". + */ + return 0; + } const uint32_t elapsed_steps = elapsed_ticks / TICKS_PER_STEP; if (!elapsed_steps) { diff --git a/src/test/test_bwmgt.c b/src/test/test_bwmgt.c index 7bcfcf7fe..2428b1505 100644 --- a/src/test/test_bwmgt.c +++ b/src/test/test_bwmgt.c @@ -178,8 +178,13 @@ test_bwmgt_token_buf_refill(void *arg) tt_int_op(b.read_bucket, OP_GT, 8*KB-200); tt_int_op(b.read_bucket, OP_LT, 8*KB+200); - // a ridiculous amount of time passes + // We step a second backwards, and nothing happens. tt_int_op(0, OP_EQ, token_bucket_refill(&b, START_TS + SEC*64)); + tt_int_op(b.read_bucket, OP_GT, 8*KB-200); + tt_int_op(b.read_bucket, OP_LT, 8*KB+200); + + // A ridiculous amount of time passes. + tt_int_op(0, OP_EQ, token_bucket_refill(&b, INT32_MAX)); tt_int_op(b.read_bucket, OP_EQ, b.burst); done:

1 0