Skip to content

Commit

Permalink
Report for issue #429 updated by SovaSlava
Browse files Browse the repository at this point in the history
  • Loading branch information
code423n4 committed Sep 7, 2023
1 parent 3a176bf commit d6e614d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion data/SovaSlava-Q.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,8 @@ https://github.com/code-423n4/2023-09-ondo/blob/47d34d6d4a5303af5f46e907ac2292e6

Q5 - Approver can't revoke his vote
There is not opportunity revoke vote for tx in DestinationBridge.sol
https://github.com/code-423n4/2023-09-ondo/blob/47d34d6d4a5303af5f46e907ac2292e6a7745f6c/contracts/bridge/DestinationBridge.sol#L197
https://github.com/code-423n4/2023-09-ondo/blob/47d34d6d4a5303af5f46e907ac2292e6a7745f6c/contracts/bridge/DestinationBridge.sol#L197

Q6 - Insufficient verification of threshold amount in setThresholds().
Function dont checks, that chainToThresholds[srcChain][i - 1].amount is equal to amounts[i]. If these values are equal, should be revert error - ThresholdsNotInAscendingOrder().
https://github.com/code-423n4/2023-09-ondo/blob/47d34d6d4a5303af5f46e907ac2292e6a7745f6c/contracts/bridge/DestinationBridge.sol#L270

0 comments on commit d6e614d

Please sign in to comment.