Skip to content

Commit 31ff7ca

Browse files
authored
Merge pull request #203 from dalek-cryptography/oleg/cleanup
Versions for RP and IPP domain separators + cleaner test
2 parents 4e58a6a + 7320c15 commit 31ff7ca

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/range_proof/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -690,6 +690,7 @@ mod tests {
690690
fn detect_dishonest_dealer_during_aggregation() {
691691
use self::dealer::*;
692692
use self::party::*;
693+
use errors::MPCError;
693694

694695
// Simulate one party
695696
let m = 1;
@@ -724,7 +725,6 @@ mod tests {
724725

725726
let maybe_share0 = party0.apply_challenge(&poly_challenge);
726727

727-
// XXX when we have error types, check finer info than "was error"
728-
assert!(maybe_share0.is_err());
728+
assert!(maybe_share0.unwrap_err() == MPCError::MaliciousDealer);
729729
}
730730
}

src/transcript.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ fn le_u64(value: u64) -> [u8; 8] {
2626

2727
impl TranscriptProtocol for Transcript {
2828
fn rangeproof_domain_sep(&mut self, n: u64, m: u64) {
29-
self.commit_bytes(b"dom-sep", b"rangeproof");
29+
self.commit_bytes(b"dom-sep", b"rangeproof v1");
3030
self.commit_bytes(b"n", &le_u64(n));
3131
self.commit_bytes(b"m", &le_u64(m));
3232
}
3333

3434
fn innerproduct_domain_sep(&mut self, n: u64) {
35-
self.commit_bytes(b"dom-sep", b"ipp");
35+
self.commit_bytes(b"dom-sep", b"ipp v1");
3636
self.commit_bytes(b"n", &le_u64(n));
3737
}
3838

0 commit comments

Comments
 (0)