#30092: Add a probability-to-apply field for circuitpadidng machines -------------------------------------------------+------------------------- Reporter: mikeperry | Owner: (none) Type: enhancement | Status: | needs_review Priority: Medium | Milestone: Tor: | 0.4.1.x-final Component: Core Tor/Tor | Version: Severity: Normal | Resolution: Keywords: wtf-pad, tor-relay, tor-cell, | Actual Points: 0.5 padding, 041-proposed | Parent ID: #28634 | Points: 2 Reviewer: asn | Sponsor: | Sponsor2-can -------------------------------------------------+-------------------------
Comment (by asn): OK this LGTM. I think a test would be nice tho since it has grown to non-trivial complexity after the last commit. Here is an easy way to test this: - Put the body of `if (machine->conditions.apply_with_probability > 0) {` into its own function which is gonna be unittested. - Create a mock machine and a mock circuit. - Call the new function a few times and check that `circ->padding_apply_coin_tossed` is behaving properly. I think this can be done without mocking `crypto_rand_double()` but you could also mock it so that it returns predictable stuff to make it more easy. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30092#comment:5> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs