summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2019-02-19 11:21:51 -0500
committerNick Mathewson <nickm@torproject.org>2019-02-19 11:21:51 -0500
commit5c87add923d14c90db8c84cca843754e7715df9e (patch)
tree50434c24c2f60c88f73c73ccd77e952911aa65ad
parent94f7e53d0416d80bacc5e2cf86e97ab07cfb09f2 (diff)
parentf5a6d4c6ea78e73e8f27292fac6d4153e8e84aa0 (diff)
downloadtor-5c87add923d14c90db8c84cca843754e7715df9e.tar.gz
tor-5c87add923d14c90db8c84cca843754e7715df9e.zip
Merge remote-tracking branch 'tor-github/pr/701' into maint-0.4.0
-rw-r--r--changes/bug292985
-rw-r--r--src/test/test_circuitpadding.c8
2 files changed, 10 insertions, 3 deletions
diff --git a/changes/bug29298 b/changes/bug29298
new file mode 100644
index 0000000000..df12db77d7
--- /dev/null
+++ b/changes/bug29298
@@ -0,0 +1,5 @@
+ o Minor bugfixes (testing, circuit padding):
+ - Disabled unstable circuit padding unittest that was causing intermittent
+ test failures because of ill-defined small histogram. Such histograms
+ will be allowed again after 29298 is implemented. Fixes second case of
+ bug 29122; bugfix on 0.4.0.1-alpha. \ No newline at end of file
diff --git a/src/test/test_circuitpadding.c b/src/test/test_circuitpadding.c
index 12a07fa957..eee1edc50c 100644
--- a/src/test/test_circuitpadding.c
+++ b/src/test/test_circuitpadding.c
@@ -46,7 +46,6 @@ void test_circuitpadding_conditions(void *arg);
void test_circuitpadding_serialize(void *arg);
void test_circuitpadding_rtt(void *arg);
void test_circuitpadding_tokens(void *arg);
-void test_circuitpadding_circuitsetup_machine(void *arg);
static void
simulate_single_hop_extend(circuit_t *client, circuit_t *mid_relay,
@@ -1809,6 +1808,8 @@ test_circuitpadding_conditions(void *arg)
return;
}
+/** Disabled unstable test until #29298 is implemented (see #29122) */
+#if 0
void
test_circuitpadding_circuitsetup_machine(void *arg)
{
@@ -2054,6 +2055,7 @@ test_circuitpadding_circuitsetup_machine(void *arg)
return;
}
+#endif
/** Helper function: Initializes a padding machine where every state uses the
* uniform probability distribution. */
@@ -2340,11 +2342,11 @@ test_circuitpadding_global_rate_limiting(void *arg)
{ #name, test_##name, (flags), NULL, NULL }
struct testcase_t circuitpadding_tests[] = {
- //TEST_CIRCUITPADDING(circuitpadding_circuitsetup_machine, 0),
TEST_CIRCUITPADDING(circuitpadding_tokens, TT_FORK),
TEST_CIRCUITPADDING(circuitpadding_negotiation, TT_FORK),
TEST_CIRCUITPADDING(circuitpadding_wronghop, TT_FORK),
- TEST_CIRCUITPADDING(circuitpadding_circuitsetup_machine, TT_FORK),
+ /** Disabled unstable test until #29298 is implemented (see #29122) */
+ // TEST_CIRCUITPADDING(circuitpadding_circuitsetup_machine, TT_FORK),
TEST_CIRCUITPADDING(circuitpadding_conditions, TT_FORK),
TEST_CIRCUITPADDING(circuitpadding_rtt, TT_FORK),
TEST_CIRCUITPADDING(circuitpadding_sample_distribution, TT_FORK),