tor-commits April 2018

tor-commits@lists.torproject.org

17 participants
1638 discussions

[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 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] 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] Remove tell_event_loop_to_run_external_code() per review
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 4c03af48806ac03e5754a7531ee9e915fc2f6c8c Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Apr 13 12:11:22 2018 -0400 Remove tell_event_loop_to_run_external_code() per review (This function is no longer used.) --- src/or/main.c | 14 -------------- src/or/main.h | 1 - 2 files changed, 15 deletions(-) diff --git a/src/or/main.c b/src/or/main.c index 257af8b18..074746108 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -731,20 +731,6 @@ connection_should_read_from_linked_conn(connection_t *conn) return 0; } -/** If we called event_base_loop() and told it to never stop until it - * runs out of events, now we've changed our mind: tell it we want it to - * exit once the current round of callbacks is done, so that we can - * run external code, and then return to the main loop. */ -void -tell_event_loop_to_run_external_code(void) -{ - if (!called_loop_once) { - struct timeval tv = { 0, 0 }; - tor_libevent_exit_loop_after_delay(tor_libevent_get_base(), &tv); - called_loop_once = 1; /* hack to avoid adding more exit events */ - } -} - /** Event to run 'shutdown did not work callback'. */ static struct event *shutdown_did_not_work_event = NULL; diff --git a/src/or/main.h b/src/or/main.h index f01506fce..5ca435064 100644 --- a/src/or/main.h +++ b/src/or/main.h @@ -45,7 +45,6 @@ int connection_is_writing(connection_t *conn); MOCK_DECL(void,connection_stop_writing,(connection_t *conn)); MOCK_DECL(void,connection_start_writing,(connection_t *conn)); -void tell_event_loop_to_run_external_code(void); void tor_shutdown_event_loop_and_exit(int exitcode); int tor_event_loop_shutdown_is_pending(void);

1 0

[tor/master] Merge branch 'postloop_callbacks_2'
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 61d87dfa1509aaf209ff72d7c5e091d06b191040 Merge: b152d62ce 4c03af488 Author: Nick Mathewson <nickm(a)torproject.org> Date: Fri Apr 13 12:12:46 2018 -0400 Merge branch 'postloop_callbacks_2' changes/ticket25374 | 6 +++ src/common/compat_libevent.c | 115 +++++++++++++++++++++++++++++++++++++--- src/common/compat_libevent.h | 3 ++ src/or/connection_edge.c | 32 +++++++---- src/or/main.c | 66 ++++++++++------------- src/or/main.h | 1 - src/test/test_compat_libevent.c | 60 +++++++++++++++++++++ 7 files changed, 227 insertions(+), 56 deletions(-)

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] Remove TestingEnableTbEmptyEvent
by nickm＠torproject.org 13 Apr '18

13 Apr '18

commit 16f08de0fd85b9fe8ace9f4905190fa6dc27e4ea Author: Nick Mathewson <nickm(a)torproject.org> Date: Tue Apr 10 12:16:21 2018 -0400 Remove TestingEnableTbEmptyEvent This option was used for shadow testing previously, but is no longer used for anything. It interferes with refactoring our token buckets. --- changes/ticket25760 | 5 ++ doc/tor.1.txt | 6 -- src/or/config.c | 9 +-- src/or/connection.c | 131 -------------------------------------- src/or/connection.h | 7 -- src/or/control.c | 23 ------- src/or/control.h | 5 +- src/or/or.h | 10 --- src/test/test_controller_events.c | 75 ---------------------- src/test/test_options.c | 10 --- 10 files changed, 7 insertions(+), 274 deletions(-) diff --git a/changes/ticket25760 b/changes/ticket25760 new file mode 100644 index 000000000..504fd60de --- /dev/null +++ b/changes/ticket25760 @@ -0,0 +1,5 @@ + o Removed features: + - The TestingEnableTbEmptyEvent option has been removed. It was used + in testing simulations to measure how often connection buckets were + emptied, in order to improve our scheduling, but it has not + been actively used in years. Closes ticket 25760. diff --git a/doc/tor.1.txt b/doc/tor.1.txt index 5f31344f6..c044a765f 100644 --- a/doc/tor.1.txt +++ b/doc/tor.1.txt @@ -2896,7 +2896,6 @@ The following options are used for running a testing Tor network. TestingDirConnectionMaxStall 30 seconds TestingEnableConnBwEvent 1 TestingEnableCellStatsEvent 1 - TestingEnableTbEmptyEvent 1 [[TestingV3AuthInitialVotingInterval]] **TestingV3AuthInitialVotingInterval** __N__ **minutes**|**hours**:: Like V3AuthVotingInterval, but for initial voting interval before the first @@ -3034,11 +3033,6 @@ The following options are used for running a testing Tor network. events. Changing this requires that **TestingTorNetwork** is set. (Default: 0) -[[TestingEnableTbEmptyEvent]] **TestingEnableTbEmptyEvent** **0**|**1**:: - If this option is set, then Tor controllers may register for TB_EMPTY - events. Changing this requires that **TestingTorNetwork** is set. - (Default: 0) - [[TestingMinExitFlagThreshold]] **TestingMinExitFlagThreshold** __N__ **KBytes**|**MBytes**|**GBytes**|**TBytes**|**KBits**|**MBits**|**GBits**|**TBits**:: Sets a lower-bound for assigning an exit flag when running as an authority on a testing network. Overrides the usual default lower bound diff --git a/src/or/config.c b/src/or/config.c index 3c3fd46e6..71f8528b6 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -334,7 +334,7 @@ static config_var_t option_vars_[] = { V(DownloadExtraInfo, BOOL, "0"), V(TestingEnableConnBwEvent, BOOL, "0"), V(TestingEnableCellStatsEvent, BOOL, "0"), - V(TestingEnableTbEmptyEvent, BOOL, "0"), + OBSOLETE("TestingEnableTbEmptyEvent"), V(EnforceDistinctSubnets, BOOL, "1"), V(EntryNodes, ROUTERSET, NULL), V(EntryStatistics, BOOL, "0"), @@ -704,7 +704,6 @@ static const config_var_t testing_tor_network_defaults[] = { V(TestingDirConnectionMaxStall, INTERVAL, "30 seconds"), V(TestingEnableConnBwEvent, BOOL, "1"), V(TestingEnableCellStatsEvent, BOOL, "1"), - V(TestingEnableTbEmptyEvent, BOOL, "1"), VAR("___UsingTestNetworkDefaults", BOOL, UsingTestNetworkDefaults_, "1"), V(RendPostPeriod, INTERVAL, "2 minutes"), @@ -4499,12 +4498,6 @@ options_validate(or_options_t *old_options, or_options_t *options, "Tor networks!"); } - if (options->TestingEnableTbEmptyEvent && - !options->TestingTorNetwork && !options->UsingTestNetworkDefaults_) { - REJECT("TestingEnableTbEmptyEvent may only be changed in testing " - "Tor networks!"); - } - if (options->TestingTorNetwork) { log_warn(LD_CONFIG, "TestingTorNetwork is set. This will make your node " "almost unusable in the public Tor network, and is " diff --git a/src/or/connection.c b/src/or/connection.c index 5532551cf..e2fd196dd 100644 --- a/src/or/connection.c +++ b/src/or/connection.c @@ -3019,57 +3019,6 @@ record_num_bytes_transferred_impl(connection_t *conn, rep_hist_note_exit_bytes(conn->port, num_written, num_read); } -/** Helper: convert given tvnow time value to milliseconds since - * midnight. */ -static uint32_t -msec_since_midnight(const struct timeval *tvnow) -{ - return (uint32_t)(((tvnow->tv_sec % 86400L) * 1000L) + - ((uint32_t)tvnow->tv_usec / (uint32_t)1000L)); -} - -/** Helper: return the time in milliseconds since last_empty_time - * when a bucket ran empty that previously had tokens_before tokens - * now has tokens_after tokens after refilling at timestamp - * tvnow, capped at milliseconds_elapsed milliseconds since - * last refilling that bucket. Return 0 if the bucket has not been empty - * since the last refill or has not been refilled. */ -uint32_t -bucket_millis_empty(int tokens_before, uint32_t last_empty_time, - int tokens_after, int milliseconds_elapsed, - const struct timeval *tvnow) -{ - uint32_t result = 0, refilled; - if (tokens_before <= 0 && tokens_after > tokens_before) { - refilled = msec_since_midnight(tvnow); - result = (uint32_t)((refilled + 86400L * 1000L - last_empty_time) % - (86400L * 1000L)); - if (result > (uint32_t)milliseconds_elapsed) - result = (uint32_t)milliseconds_elapsed; - } - return result; -} - -/** Check if a bucket which had tokens_before tokens and which got - * tokens_removed tokens removed at timestamp tvnow has run - * out of tokens, and if so, note the milliseconds since midnight in - * timestamp_var for the next TB_EMPTY event. */ -void -connection_buckets_note_empty_ts(uint32_t *timestamp_var, - int tokens_before, size_t tokens_removed, - const struct timeval *tvnow) -{ - if (tokens_before > 0 && (uint32_t)tokens_before <= tokens_removed) - *timestamp_var = msec_since_midnight(tvnow); -} - -/** Last time at which the global or relay buckets were emptied in msec - * since midnight. */ -static uint32_t global_relayed_read_emptied = 0, - global_relayed_write_emptied = 0, - global_read_emptied = 0, - global_write_emptied = 0; - /** We just read num_read and wrote num_written bytes * onto conn. Decrement buckets appropriately. */ static void @@ -3094,30 +3043,6 @@ connection_buckets_decrement(connection_t *conn, time_t now, if (!connection_is_rate_limited(conn)) return; /* local IPs are free */ - /* If one or more of our token buckets ran dry just now, note the - * timestamp for TB_EMPTY events. */ - if (get_options()->TestingEnableTbEmptyEvent) { - struct timeval tvnow; - tor_gettimeofday_cached(&tvnow); - if (connection_counts_as_relayed_traffic(conn, now)) { - connection_buckets_note_empty_ts(&global_relayed_read_emptied, - global_relayed_read_bucket, num_read, &tvnow); - connection_buckets_note_empty_ts(&global_relayed_write_emptied, - global_relayed_write_bucket, num_written, &tvnow); - } - connection_buckets_note_empty_ts(&global_read_emptied, - global_read_bucket, num_read, &tvnow); - connection_buckets_note_empty_ts(&global_write_emptied, - global_write_bucket, num_written, &tvnow); - if (connection_speaks_cells(conn) && conn->state == OR_CONN_STATE_OPEN) { - or_connection_t *or_conn = TO_OR_CONN(conn); - connection_buckets_note_empty_ts(&or_conn->read_emptied_time, - or_conn->read_bucket, num_read, &tvnow); - connection_buckets_note_empty_ts(&or_conn->write_emptied_time, - or_conn->write_bucket, num_written, &tvnow); - } - } - if (connection_counts_as_relayed_traffic(conn, now)) { global_relayed_read_bucket -= (int)num_read; global_relayed_write_bucket -= (int)num_written; @@ -3238,12 +3163,6 @@ connection_bucket_refill(int milliseconds_elapsed, time_t now) smartlist_t *conns = get_connection_array(); int bandwidthrate, bandwidthburst, relayrate, relayburst; - int prev_global_read = global_read_bucket; - int prev_global_write = global_write_bucket; - int prev_relay_read = global_relayed_read_bucket; - int prev_relay_write = global_relayed_write_bucket; - struct timeval tvnow; /*< Only used if TB_EMPTY events are enabled. */ - bandwidthrate = (int)options->BandwidthRate; bandwidthburst = (int)options->BandwidthBurst; @@ -3278,32 +3197,6 @@ connection_bucket_refill(int milliseconds_elapsed, time_t now) milliseconds_elapsed, "global_relayed_write_bucket"); - /* If buckets were empty before and have now been refilled, tell any - * interested controllers. */ - if (get_options()->TestingEnableTbEmptyEvent) { - uint32_t global_read_empty_time, global_write_empty_time, - relay_read_empty_time, relay_write_empty_time; - tor_gettimeofday_cached(&tvnow); - global_read_empty_time = bucket_millis_empty(prev_global_read, - global_read_emptied, global_read_bucket, - milliseconds_elapsed, &tvnow); - global_write_empty_time = bucket_millis_empty(prev_global_write, - global_write_emptied, global_write_bucket, - milliseconds_elapsed, &tvnow); - control_event_tb_empty("GLOBAL", global_read_empty_time, - global_write_empty_time, milliseconds_elapsed); - relay_read_empty_time = bucket_millis_empty(prev_relay_read, - global_relayed_read_emptied, - global_relayed_read_bucket, - milliseconds_elapsed, &tvnow); - relay_write_empty_time = bucket_millis_empty(prev_relay_write, - global_relayed_write_emptied, - global_relayed_write_bucket, - milliseconds_elapsed, &tvnow); - control_event_tb_empty("RELAY", relay_read_empty_time, - relay_write_empty_time, milliseconds_elapsed); - } - /* refill the per-connection buckets */ SMARTLIST_FOREACH_BEGIN(conns, connection_t *, conn) { if (connection_speaks_cells(conn)) { @@ -3311,9 +3204,6 @@ connection_bucket_refill(int milliseconds_elapsed, time_t now) int orbandwidthrate = or_conn->bandwidthrate; int orbandwidthburst = or_conn->bandwidthburst; - int prev_conn_read = or_conn->read_bucket; - int prev_conn_write = or_conn->write_bucket; - if (connection_bucket_should_increase(or_conn->read_bucket, or_conn)) { connection_bucket_refill_helper(&or_conn->read_bucket, orbandwidthrate, @@ -3328,27 +3218,6 @@ connection_bucket_refill(int milliseconds_elapsed, time_t now) milliseconds_elapsed, "or_conn->write_bucket"); } - - /* If buckets were empty before and have now been refilled, tell any - * interested controllers. */ - if (get_options()->TestingEnableTbEmptyEvent) { - char *bucket; - uint32_t conn_read_empty_time, conn_write_empty_time; - tor_asprintf(&bucket, "ORCONN ID="U64_FORMAT, - U64_PRINTF_ARG(or_conn->base_.global_identifier)); - conn_read_empty_time = bucket_millis_empty(prev_conn_read, - or_conn->read_emptied_time, - or_conn->read_bucket, - milliseconds_elapsed, &tvnow); - conn_write_empty_time = bucket_millis_empty(prev_conn_write, - or_conn->write_emptied_time, - or_conn->write_bucket, - milliseconds_elapsed, &tvnow); - control_event_tb_empty(bucket, conn_read_empty_time, - conn_write_empty_time, - milliseconds_elapsed); - tor_free(bucket); - } } if (conn->read_blocked_on_bw == 1 /* marked to turn reading back on now */ diff --git a/src/or/connection.h b/src/or/connection.h index 6bc5a7cfd..930e5f81a 100644 --- a/src/or/connection.h +++ b/src/or/connection.h @@ -272,13 +272,6 @@ void connection_check_oos(int n_socks, int failed); STATIC void connection_free_minimal(connection_t *conn); /* Used only by connection.c and test*.c */ -uint32_t bucket_millis_empty(int tokens_before, uint32_t last_empty_time, - int tokens_after, int milliseconds_elapsed, - const struct timeval *tvnow); -void connection_buckets_note_empty_ts(uint32_t *timestamp_var, - int tokens_before, - size_t tokens_removed, - const struct timeval *tvnow); MOCK_DECL(STATIC int,connection_connect_sockaddr, (connection_t *conn, const struct sockaddr *sa, diff --git a/src/or/control.c b/src/or/control.c index 5a2fae64e..0539ddaca 100644 --- a/src/or/control.c +++ b/src/or/control.c @@ -1214,7 +1214,6 @@ static const struct control_event_t control_event_table[] = { { EVENT_CONF_CHANGED, "CONF_CHANGED"}, { EVENT_CONN_BW, "CONN_BW" }, { EVENT_CELL_STATS, "CELL_STATS" }, - { EVENT_TB_EMPTY, "TB_EMPTY" }, { EVENT_CIRC_BANDWIDTH_USED, "CIRC_BW" }, { EVENT_TRANSPORT_LAUNCHED, "TRANSPORT_LAUNCHED" }, { EVENT_HS_DESC, "HS_DESC" }, @@ -6077,28 +6076,6 @@ control_event_circuit_cell_stats(void) return 0; } -/** Tokens in bucket have been refilled: the read bucket was empty - * for read_empty_time millis, the write bucket was empty for - * write_empty_time millis, and buckets were last refilled - * milliseconds_elapsed millis ago. Only emit TB_EMPTY event if - * either read or write bucket have been empty before. */ -int -control_event_tb_empty(const char *bucket, uint32_t read_empty_time, - uint32_t write_empty_time, - int milliseconds_elapsed) -{ - if (get_options()->TestingEnableTbEmptyEvent && - EVENT_IS_INTERESTING(EVENT_TB_EMPTY) && - (read_empty_time > 0 || write_empty_time > 0)) { - send_control_event(EVENT_TB_EMPTY, - "650 TB_EMPTY %s READ=%d WRITTEN=%d " - "LAST=%d\r\n", - bucket, read_empty_time, write_empty_time, - milliseconds_elapsed); - } - return 0; -} - /* about 5 minutes worth. */ #define N_BW_EVENTS_TO_CACHE 300 /* Index into cached_bw_events to next write. */ diff --git a/src/or/control.h b/src/or/control.h index 28ffeaed8..2fd3c553f 100644 --- a/src/or/control.h +++ b/src/or/control.h @@ -59,9 +59,6 @@ int control_event_circ_bandwidth_used(void); int control_event_conn_bandwidth(connection_t *conn); int control_event_conn_bandwidth_used(void); int control_event_circuit_cell_stats(void); -int control_event_tb_empty(const char *bucket, uint32_t read_empty_time, - uint32_t write_empty_time, - int milliseconds_elapsed); void control_event_logmsg(int severity, uint32_t domain, const char *msg); int control_event_descriptors_changed(smartlist_t *routers); int control_event_address_mapped(const char *from, const char *to, @@ -194,7 +191,7 @@ void control_free_all(void); #define EVENT_CONF_CHANGED 0x0019 #define EVENT_CONN_BW 0x001A #define EVENT_CELL_STATS 0x001B -#define EVENT_TB_EMPTY 0x001C +/* UNUSED : 0x001C */ #define EVENT_CIRC_BANDWIDTH_USED 0x001D #define EVENT_TRANSPORT_LAUNCHED 0x0020 #define EVENT_HS_DESC 0x0021 diff --git a/src/or/or.h b/src/or/or.h index 25ad35175..bcce33755 100644 --- a/src/or/or.h +++ b/src/or/or.h @@ -1660,13 +1660,6 @@ typedef struct or_connection_t { * bandwidthburst. (OPEN ORs only) */ int write_bucket; /**< When this hits 0, stop writing. Like read_bucket. */ - /** Last emptied read token bucket in msec since midnight; only used if - * TB_EMPTY events are enabled. */ - uint32_t read_emptied_time; - /** Last emptied write token bucket in msec since midnight; only used if - * TB_EMPTY events are enabled. */ - uint32_t write_emptied_time; - /* * Count the number of bytes flushed out on this orconn, and the number of * bytes TLS actually sent - used for overhead estimation for scheduling. @@ -4427,9 +4420,6 @@ typedef struct { /** Enable CELL_STATS events. Only altered on testing networks. */ int TestingEnableCellStatsEvent; - /** Enable TB_EMPTY events. Only altered on testing networks. */ - int TestingEnableTbEmptyEvent; - /** If true, and we have GeoIP data, and we're a bridge, keep a per-country * count of how many client addresses have contacted us so that we can help * the bridge authority guess which countries have blocked access to us. */ diff --git a/src/test/test_controller_events.c b/src/test/test_controller_events.c index 901ad7ab3..e81aea8d6 100644 --- a/src/test/test_controller_events.c +++ b/src/test/test_controller_events.c @@ -12,79 +12,6 @@ #include "test.h" static void -help_test_bucket_note_empty(uint32_t expected_msec_since_midnight, - int tokens_before, size_t tokens_removed, - uint32_t msec_since_epoch) -{ - uint32_t timestamp_var = 0; - struct timeval tvnow; - tvnow.tv_sec = msec_since_epoch / 1000; - tvnow.tv_usec = (msec_since_epoch % 1000) * 1000; - connection_buckets_note_empty_ts(&timestamp_var, tokens_before, - tokens_removed, &tvnow); - tt_int_op(expected_msec_since_midnight, OP_EQ, timestamp_var); - - done: - ; -} - -static void -test_cntev_bucket_note_empty(void *arg) -{ - (void)arg; - - /* Two cases with nothing to note, because bucket was empty before; - * 86442200 == 1970-01-02 00:00:42.200000 */ - help_test_bucket_note_empty(0, 0, 0, 86442200); - help_test_bucket_note_empty(0, -100, 100, 86442200); - - /* Nothing to note, because bucket has not been emptied. */ - help_test_bucket_note_empty(0, 101, 100, 86442200); - - /* Bucket was emptied, note 42200 msec since midnight. */ - help_test_bucket_note_empty(42200, 101, 101, 86442200); - help_test_bucket_note_empty(42200, 101, 102, 86442200); -} - -static void -test_cntev_bucket_millis_empty(void *arg) -{ - struct timeval tvnow; - (void)arg; - - /* 1970-01-02 00:00:42.200000 */ - tvnow.tv_sec = 86400 + 42; - tvnow.tv_usec = 200000; - - /* Bucket has not been refilled. */ - tt_int_op(0, OP_EQ, bucket_millis_empty(0, 42120, 0, 100, &tvnow)); - tt_int_op(0, OP_EQ, bucket_millis_empty(-10, 42120, -10, 100, &tvnow)); - - /* Bucket was not empty. */ - tt_int_op(0, OP_EQ, bucket_millis_empty(10, 42120, 20, 100, &tvnow)); - - /* Bucket has been emptied 80 msec ago and has just been refilled. */ - tt_int_op(80, OP_EQ, bucket_millis_empty(-20, 42120, -10, 100, &tvnow)); - tt_int_op(80, OP_EQ, bucket_millis_empty(-10, 42120, 0, 100, &tvnow)); - tt_int_op(80, OP_EQ, bucket_millis_empty(0, 42120, 10, 100, &tvnow)); - - /* Bucket has been emptied 180 msec ago, last refill was 100 msec ago - * which was insufficient to make it positive, so cap msec at 100. */ - tt_int_op(100, OP_EQ, bucket_millis_empty(0, 42020, 1, 100, &tvnow)); - - /* 1970-01-02 00:00:00:050000 */ - tvnow.tv_sec = 86400; - tvnow.tv_usec = 50000; - - /* Last emptied 30 msec before midnight, tvnow is 50 msec after - * midnight, that's 80 msec in total. */ - tt_int_op(80, OP_EQ, bucket_millis_empty(0, 86400000 - 30, 1, 100, &tvnow)); - - done: - ; -} - -static void add_testing_cell_stats_entry(circuit_t *circ, uint8_t command, unsigned int waiting_time, unsigned int removed, unsigned int exitward) @@ -395,8 +322,6 @@ test_cntev_event_mask(void *arg) { #name, test_cntev_ ## name, flags, 0, NULL } struct testcase_t controller_event_tests[] = { - TEST(bucket_note_empty, TT_FORK), - TEST(bucket_millis_empty, TT_FORK), TEST(sum_up_cell_stats, TT_FORK), TEST(append_cell_stats, TT_FORK), TEST(format_cell_stats, TT_FORK), diff --git a/src/test/test_options.c b/src/test/test_options.c index eaf503439..0a04e8e40 100644 --- a/src/test/test_options.c +++ b/src/test/test_options.c @@ -4135,16 +4135,6 @@ test_options_validate__testing_options(void *ignored) free_options_test_data(tdata); tdata = get_options_test_data(TEST_OPTIONS_DEFAULT_VALUES "TestingEnableTbEmptyEvent 1\n" - ); - ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg); - tt_int_op(ret, OP_EQ, -1); - tt_str_op(msg, OP_EQ, "TestingEnableTbEmptyEvent may only be changed " - "in testing Tor networks!"); - tor_free(msg); - - free_options_test_data(tdata); - tdata = get_options_test_data(TEST_OPTIONS_DEFAULT_VALUES - "TestingEnableTbEmptyEvent 1\n" VALID_DIR_AUTH "TestingTorNetwork 1\n" "___UsingTestNetworkDefaults 0\n"

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

tor-commits April 2018