Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Audit Fixes #13

Merged
merged 13 commits into from
Dec 9, 2024
Merged

Audit Fixes #13

merged 13 commits into from
Dec 9, 2024

Conversation

coffeexcoin
Copy link

  • [TRST-M-1][M-05]: Require specific deployment salt and initializer selector for account deployment
  • [M-04] make initial call payable
  • [M-02] - clean up hook contet when removing hook
  • [L-05] Use Ownable2Step
  • [L-04] - Return false instead of reverting for invalid validator address
  • [L-03] return false from runValidationHooks if hook data length is lower than the number of hooks being executed
  • [L-01] add fallback function
  • [H-01] increment nonce on executeTransactionFromOutside
  • [M-03] remove early return and check hookSuccess at the end of validation
  • [TRST-L-01] Ensure at least one K1 validator and owner are always present

@coffeexcoin coffeexcoin merged commit 85970fc into main Dec 9, 2024
1 check failed
@coffeexcoin coffeexcoin deleted the audit-remediation branch December 9, 2024 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant