commit e9fdec2b1d5cc4055516d08a53663002b3722529 Author: Nick Mathewson nickm@torproject.org Date: Thu Sep 8 15:13:53 2016 -0400

capture and detect expected BUG messages in shared-random tests --- src/or/shared_random.c | 2 +- src/test/test_shared_random.c | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/or/shared_random.c b/src/or/shared_random.c index e672a41..5f6b03f 100644 --- a/src/or/shared_random.c +++ b/src/or/shared_random.c @@ -201,7 +201,7 @@ verify_commit_and_reveal(const sr_commit_t *commit) if (fast_memneq(received_hashed_reveal, commit->hashed_reveal, sizeof(received_hashed_reveal))) { log_warn(LD_BUG, "SR: Received reveal value from authority %s " - "does't match the commit value.", + "doesn't match the commit value.", sr_commit_get_rsa_fpr(commit)); goto invalid; } diff --git a/src/test/test_shared_random.c b/src/test/test_shared_random.c index 8210ba2..19f2e2d 100644 --- a/src/test/test_shared_random.c +++ b/src/test/test_shared_random.c @@ -14,6 +14,7 @@ #include "router.h" #include "routerparse.h" #include "networkstatus.h" +#include "log_test_helpers.h"

static authority_cert_t *mock_cert;

@@ -326,14 +327,20 @@ test_sr_commit(void *arg)

/* Timestamp MUST match. */ test_commit.commit_ts = test_commit.reveal_ts - 42; + setup_full_capture_of_logs(LOG_WARN); tt_int_op(-1, ==, verify_commit_and_reveal(&test_commit)); + expect_log_msg_containing("doesn't match reveal timestamp"); + teardown_capture_of_logs(); memcpy(&test_commit, our_commit, sizeof(test_commit)); tt_int_op(0, ==, verify_commit_and_reveal(&test_commit));

/* Hashed reveal must match the H(encoded_reveal). */ memset(test_commit.hashed_reveal, 'X', sizeof(test_commit.hashed_reveal)); + setup_full_capture_of_logs(LOG_WARN); tt_int_op(-1, ==, verify_commit_and_reveal(&test_commit)); + expect_single_log_msg_containing("doesn't match the commit value"); + teardown_capture_of_logs(); memcpy(&test_commit, our_commit, sizeof(test_commit)); tt_int_op(0, ==, verify_commit_and_reveal(&test_commit)); } @@ -357,6 +364,7 @@ test_sr_commit(void *arg) }

done: + teardown_capture_of_logs(); SMARTLIST_FOREACH(args, char *, cp, tor_free(cp)); smartlist_free(args); sr_commit_free(our_commit); @@ -1164,8 +1172,14 @@ test_keep_commit(void *arg) memcpy(place_holder.hashed_reveal, commit->hashed_reveal, sizeof(place_holder.hashed_reveal)); memset(commit->hashed_reveal, 0, sizeof(commit->hashed_reveal)); + setup_full_capture_of_logs(LOG_WARN); tt_int_op(should_keep_commit(commit, commit->rsa_identity, SR_PHASE_REVEAL), ==, 0); + expect_log_msg_containing("doesn't match the commit value."); + expect_log_msg_containing("has an invalid reveal value."); + assert_log_predicate(mock_saved_log_n_entries() == 2, + "expected 2 log entries"); + teardown_capture_of_logs(); memcpy(commit->hashed_reveal, place_holder.hashed_reveal, sizeof(commit->hashed_reveal)); } @@ -1178,6 +1192,7 @@ test_keep_commit(void *arg) SR_PHASE_REVEAL), ==, 0);

done: + teardown_capture_of_logs(); sr_commit_free(commit); sr_commit_free(dup_commit); crypto_pk_free(k);