Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Statemind audit results #217

Merged
merged 70 commits into from
Nov 29, 2024
Merged

Statemind audit results #217

merged 70 commits into from
Nov 29, 2024

Conversation

bulbozaur
Copy link
Contributor

No description provided.

bulbozaur and others added 30 commits November 20, 2024 11:58
Merge develop into audits/statemind-responses
Audit fix: remove Escrow.requestWithdrawals()
bulbozaur and others added 26 commits November 27, 2024 14:08
…-status-check

Audit fix: Check for proposal.status in ExecutableProposals._isProposalMarkedCancelled() is required only in one case
Add missing getter for vetoer locked unstETH Ids
Audit fix: Gas optimizations and code consistancy
Audit fix: Proposer is not saved proposal data
…Queue-functions-improvements

Audit fix: The WithdrawalsBatchesQueue functions improvements
Audit fix: Event emission for unfinalized records
…ed-condition

Fix the check in the addSealableWithdrawalBlocker()
…straints

Audit Fix: Add `EmergencyProtectedTimelock.MIN_EXECUTION_DELAY()` sanity check
@bulbozaur bulbozaur requested a review from Psirex November 29, 2024 09:24
@bulbozaur bulbozaur merged commit 7ca533f into develop Nov 29, 2024
2 checks passed
@bulbozaur bulbozaur deleted the audits/statemind-responses branch November 29, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants