You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@friedger was kind enough to point out two existing bugs in the Clarity contracts:
An extra response wrapper in the 'complete-deposit-wrapper' function
A recalculated value that's already saved as a variable
2. Why It's Necessary
The first issue could (possibly?) lead to an exploit since it's possible the contract call fails since the sbtc-registry contracts updates with 'map-insert'.
The second issue is strictly optimization.
3. Execution Checklist
Remove the (ok ...) wrapper from the 'complete-deposit' contract-call
Reuse 'version-int' variable
The text was updated successfully, but these errors were encountered:
Chore - Clarity Bugs
1. Description
@friedger was kind enough to point out two existing bugs in the Clarity contracts:
2. Why It's Necessary
The first issue could (possibly?) lead to an exploit since it's possible the contract call fails since the sbtc-registry contracts updates with 'map-insert'.
The second issue is strictly optimization.
3. Execution Checklist
The text was updated successfully, but these errors were encountered: