Skip to content

Commit

Permalink
Bug: Range over constant SeqNums instead of Views.
Browse files Browse the repository at this point in the history
48316b6#r1745808451
Signed-off-by: Markus Alexander Kuppe <github.com@lemmster.de>
  • Loading branch information
lemmy committed Sep 6, 2024
1 parent cb85d20 commit 4473be6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pbft.tla
Original file line number Diff line number Diff line change
Expand Up @@ -602,7 +602,7 @@ GenerateO(V,i,v) ==
maxs == Max0({ppm.n: ppm \in ppms})
\* Apalache does not yet support integer ranges with non-constant bounds:
\* https://github.com/apalache-mc/apalache/blob/main/docs/src/apalache/known-issues.md#integer-ranges-with-non-constant-bounds
apa == { j \in Views : mins+1 <= j /\ j <= maxs} IN
apa == { j \in SeqNums : mins+1 <= j /\ j <= maxs} IN
{[v |-> v, p |-> i, n |-> sn, d |-> GetDigest(ppms,sn)] : sn \in apa }

\* Castro & Liskov S4.4:
Expand Down Expand Up @@ -677,7 +677,7 @@ Spec == Init /\ [][Next]_vars
\* Castro & Liskov S4.1:
\* The client waits for f+1 replies with valid signatures from different replicas, and with the same t and r, before accepting the result r.

Decided(t,r) ==
Decided(t,r) ==
Cardinality({rm.i: rm \in {m \in msgs.reply: m.t = t /\ m.r = r}}) >= F+1

OneReplyInv ==
Expand Down

0 comments on commit 4473be6

Please sign in to comment.