Skip to content

Commit 8cded97

Browse files
committed
fix(stm): update 'bincode' dependency to '2.0'
1 parent d91b039 commit 8cded97

File tree

5 files changed

+50
-26
lines changed

5 files changed

+50
-26
lines changed

Cargo.lock

Lines changed: 25 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

mithril-stm/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ num-traits = { version = "0.2.19", optional = true }
4545
rug = { version = "1.27.0", optional = true }
4646

4747
[dev-dependencies]
48-
bincode = "1.3.3"
48+
bincode = { version = "2.0.1", features = ["serde"] }
4949
criterion = { version = "0.6.0", features = ["html_reports"] }
5050
num-bigint = "0.4.6"
5151
num-rational = "0.4.2"

mithril-stm/src/bls_multi_signature/mod.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -223,12 +223,12 @@ mod tests {
223223
assert_eq!(vkpop, vkpop2);
224224

225225
// Now we test serde
226-
let encoded = bincode::serialize(&vk).unwrap();
226+
let encoded = bincode::serde::encode_to_vec(vk, bincode::config::legacy()).unwrap();
227227
assert_eq!(encoded, vk_bytes);
228-
let decoded: VerificationKey = bincode::deserialize(&encoded).unwrap();
228+
let (decoded,_) = bincode::serde::decode_from_slice::<VerificationKey,_>(&encoded, bincode::config::legacy()).unwrap();
229229
assert_eq!(vk, decoded);
230-
let encoded = bincode::serialize(&vkpop).unwrap();
231-
let decoded: VerificationKeyPoP = bincode::deserialize(&encoded).unwrap();
230+
let encoded = bincode::serde::encode_to_vec(vkpop, bincode::config::legacy()).unwrap();
231+
let (decoded,_) = bincode::serde::decode_from_slice::<VerificationKeyPoP,_>(&encoded, bincode::config::legacy()).unwrap();
232232
assert_eq!(vkpop, decoded);
233233
}
234234

@@ -241,12 +241,12 @@ mod tests {
241241
assert_eq!(sk, sk2);
242242

243243
// Now we test serde
244-
let encoded = bincode::serialize(&sk).unwrap();
245-
let decoded: SigningKey = bincode::deserialize(&encoded).unwrap();
244+
let encoded = bincode::serde::encode_to_vec(&sk, bincode::config::legacy()).unwrap();
245+
let (decoded,_) = bincode::serde::decode_from_slice::<SigningKey,_>(&encoded, bincode::config::legacy()).unwrap();
246246
assert_eq!(sk, decoded);
247247

248248
// See that it is consistent with raw serialisation
249-
let decoded_bytes: SigningKey = bincode::deserialize(&sk_bytes).unwrap();
249+
let (decoded_bytes,_) = bincode::serde::decode_from_slice::<SigningKey,_>(&sk_bytes, bincode::config::legacy()).unwrap();
250250
assert_eq!(sk, decoded_bytes);
251251
}
252252

mithril-stm/src/merkle_tree/tree.rs

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -260,16 +260,17 @@ mod tests {
260260
let deserialised = Path::from_bytes(&bytes).unwrap();
261261
assert!(t.to_commitment().check(&values[i], &deserialised).is_ok());
262262

263-
let encoded = bincode::serialize(&pf).unwrap();
264-
let decoded: Path<Blake2b<U32>> = bincode::deserialize(&encoded).unwrap();
263+
let encoded = bincode::serde::encode_to_vec(&pf, bincode::config::legacy()).unwrap();
264+
let (decoded,_) = bincode::serde::decode_from_slice::<Path<Blake2b<U32>>,_>(&encoded, bincode::config::legacy()).unwrap();
265265
assert!(t.to_commitment().check(&values[i], &decoded).is_ok());
266266
})
267267
}
268268

269269
#[test]
270270
fn test_bytes_tree_commitment((t, values) in arb_tree(5)) {
271-
let encoded = bincode::serialize(&t.to_commitment()).unwrap();
272-
let decoded: MerkleTreeCommitment::<Blake2b<U32>> = bincode::deserialize(&encoded).unwrap();
271+
let encoded = bincode::serde::encode_to_vec(t.to_commitment(), bincode::config::legacy()).unwrap();
272+
let (decoded,_) = bincode::serde::decode_from_slice::<MerkleTreeCommitment<Blake2b<U32>>,_>(&encoded, bincode::config::legacy()).unwrap();
273+
273274
let tree_commitment = MerkleTree::<Blake2b<U32>>::create(&values).to_commitment();
274275
assert_eq!(tree_commitment.root, decoded.root);
275276
}
@@ -281,15 +282,15 @@ mod tests {
281282
let tree = MerkleTree::<Blake2b<U32>>::create(&values);
282283
assert_eq!(tree.nodes, deserialised.nodes);
283284

284-
let encoded = bincode::serialize(&t).unwrap();
285-
let decoded: MerkleTree::<Blake2b<U32>> = bincode::deserialize(&encoded).unwrap();
285+
let encoded = bincode::serde::encode_to_vec(&t, bincode::config::legacy()).unwrap();
286+
let (decoded,_) = bincode::serde::decode_from_slice::<MerkleTree<Blake2b<U32>>,_>(&encoded, bincode::config::legacy()).unwrap();
286287
assert_eq!(tree.nodes, decoded.nodes);
287288
}
288289

289290
#[test]
290291
fn test_bytes_tree_commitment_batch_compat((t, values) in arb_tree(5)) {
291-
let encoded = bincode::serialize(&t.to_commitment_batch_compat()).unwrap();
292-
let decoded: MerkleTreeCommitmentBatchCompat::<Blake2b<U32>> = bincode::deserialize(&encoded).unwrap();
292+
let encoded = bincode::serde::encode_to_vec(t.to_commitment_batch_compat(), bincode::config::legacy()).unwrap();
293+
let (decoded,_) = bincode::serde::decode_from_slice::<MerkleTreeCommitmentBatchCompat<Blake2b<U32>>,_>(&encoded, bincode::config::legacy()).unwrap();
293294
let tree_commitment = MerkleTree::<Blake2b<U32>>::create(&values).to_commitment_batch_compat();
294295
assert_eq!(tree_commitment.root, decoded.root);
295296
assert_eq!(tree_commitment.get_nr_leaves(), decoded.get_nr_leaves());
@@ -379,8 +380,8 @@ mod tests {
379380
let deserialized = BatchPath::from_bytes(bytes).unwrap();
380381
assert!(t.to_commitment_batch_compat().check(&batch_values, &deserialized).is_ok());
381382

382-
let encoded = bincode::serialize(&bp).unwrap();
383-
let decoded: BatchPath<Blake2b<U32>> = bincode::deserialize(&encoded).unwrap();
383+
let encoded = bincode::serde::encode_to_vec(&bp, bincode::config::legacy()).unwrap();
384+
let (decoded,_) = bincode::serde::decode_from_slice::<BatchPath<Blake2b<U32>>,_>(&encoded, bincode::config::legacy()).unwrap();
384385
assert!(t.to_commitment_batch_compat().check(&batch_values, &decoded).is_ok());
385386
}
386387
}

mithril-stm/src/stm.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1195,8 +1195,8 @@ mod tests {
11951195
let bytes = initializer.to_bytes();
11961196
assert!(StmInitializer::from_bytes(&bytes).is_ok());
11971197

1198-
let bytes = bincode::serialize(&initializer).unwrap();
1199-
assert!(bincode::deserialize::<StmInitializer>(&bytes).is_ok())
1198+
let bytes = bincode::serde::encode_to_vec(&initializer, bincode::config::legacy()).unwrap();
1199+
assert!(bincode::serde::decode_from_slice::<StmInitializer,_>(&bytes, bincode::config::legacy()).is_ok())
12001200
}
12011201

12021202
#[test]
@@ -1211,8 +1211,8 @@ mod tests {
12111211
let sig_deser = StmSig::from_bytes::<D>(&bytes).unwrap();
12121212
assert!(sig_deser.verify(&params, &ps[0].get_vk(), &ps[0].get_stake(), &avk, &msg).is_ok());
12131213

1214-
let encoded = bincode::serialize(&sig).unwrap();
1215-
let decoded: StmSig = bincode::deserialize(&encoded).unwrap();
1214+
let encoded = bincode::serde::encode_to_vec(&sig, bincode::config::legacy()).unwrap();
1215+
let (decoded,_) = bincode::serde::decode_from_slice::<StmSig,_>(&encoded, bincode::config::legacy()).unwrap();
12161216
assert!(decoded.verify(&params, &ps[0].get_vk(), &ps[0].get_stake(), &avk, &msg).is_ok());
12171217
}
12181218
}
@@ -1232,8 +1232,8 @@ mod tests {
12321232
let aggr2 = StmAggrSig::from_bytes(&bytes).unwrap();
12331233
assert!(aggr2.verify(&msg, &clerk.compute_avk(), &params).is_ok());
12341234

1235-
let encoded = bincode::serialize(&aggr).unwrap();
1236-
let decoded: StmAggrSig::<D> = bincode::deserialize(&encoded).unwrap();
1235+
let encoded = bincode::serde::encode_to_vec(&aggr, bincode::config::legacy()).unwrap();
1236+
let (decoded,_) = bincode::serde::decode_from_slice::<StmAggrSig<D>,_>(&encoded, bincode::config::legacy()).unwrap();
12371237
assert!(decoded.verify(&msg, &clerk.compute_avk(), &params).is_ok());
12381238
}
12391239
}

0 commit comments

Comments
 (0)