summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexander Færøy <ahf@torproject.org>2017-02-01 18:58:06 +0000
committerAlexander Færøy <ahf@torproject.org>2017-02-01 19:01:07 +0000
commit7eb7af08d942766cb8b96d2460f0b2946cf4403c (patch)
tree555b67c65b5a6ede463198b1aab3958250f14e39
parenta5aec6ac3726d0202be7caa64acf1f02d2eac653 (diff)
downloadtor-7eb7af08d942766cb8b96d2460f0b2946cf4403c.tar.gz
tor-7eb7af08d942766cb8b96d2460f0b2946cf4403c.zip
Add checks for expected log messages in test_hs_intropoint.
This patch adds checks for expected log messages for failure cases of different ill-formed ESTABLISH_INTRO cell's. See: https://bugs.torproject.org/21266
-rw-r--r--src/test/test_hs_intropoint.c14
1 files changed, 13 insertions, 1 deletions
diff --git a/src/test/test_hs_intropoint.c b/src/test/test_hs_intropoint.c
index 770fff42d2..ea12aeb2da 100644
--- a/src/test/test_hs_intropoint.c
+++ b/src/test/test_hs_intropoint.c
@@ -146,8 +146,11 @@ test_establish_intro_wrong_purpose(void *arg)
establish_intro_cell);
tt_int_op(cell_len, >, 0);
- /* Receive the cell */
+ /* Receive the cell. Should fail. */
+ setup_full_capture_of_logs(LOG_INFO);
retval = hs_intro_received_establish_intro(intro_circ, cell_body, cell_len);
+ expect_log_msg_containing("Rejecting ESTABLISH_INTRO on non-OR circuit.");
+ teardown_capture_of_logs();
tt_int_op(retval, ==, -1);
done:
@@ -180,7 +183,10 @@ test_establish_intro_wrong_keytype(void *arg)
helper_prepare_circ_for_intro(intro_circ, circuit_key_material);
/* Receive the cell. Should fail. */
+ setup_full_capture_of_logs(LOG_INFO);
retval = hs_intro_received_establish_intro(intro_circ, (uint8_t*)"", 0);
+ expect_log_msg_containing("Empty ESTABLISH_INTRO cell.");
+ teardown_capture_of_logs();
tt_int_op(retval, ==, -1);
done:
@@ -217,7 +223,10 @@ test_establish_intro_wrong_keytype2(void *arg)
cell_body[0] = 42;
/* Receive the cell. Should fail. */
+ setup_full_capture_of_logs(LOG_INFO);
retval = hs_intro_received_establish_intro(intro_circ, cell_body, cell_len);
+ expect_log_msg_containing("Unrecognized AUTH_KEY_TYPE 42.");
+ teardown_capture_of_logs();
tt_int_op(retval, ==, -1);
done:
@@ -413,7 +422,10 @@ test_establish_intro_wrong_sig(void *arg)
cell_body[cell_len-1]++;
/* Receive the cell. Should fail. */
+ setup_full_capture_of_logs(LOG_INFO);
retval = hs_intro_received_establish_intro(intro_circ, cell_body, cell_len);
+ expect_log_msg_containing("Failed to verify ESTABLISH_INTRO cell.");
+ teardown_capture_of_logs();
tt_int_op(retval, ==, -1);
done: