fix possible round overflow (#497)

MystenLabs · Jul 15, 2022 · daa44a3 · daa44a3
1 parent 1474264
commit daa44a3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/narwhal/consensus/src/consensus.rs b/narwhal/consensus/src/consensus.rs
@@ -91,7 +91,7 @@ impl<PublicKey: VerifyingKey> ConsensusState<PublicKey> {
         gc_depth: Round,
     ) -> Dag<PublicKey> {
         let mut dag: Dag<PublicKey> = HashMap::new();
-        let min_round = last_committed_round - gc_depth;
+        let min_round = last_committed_round.saturating_sub(gc_depth);
         let cert_map = cert_store
             .iter(Some(Box::new(move |(_dig, cert)| {
                 cert.header.round > min_round